Multifactory with parameters

@rp The problem with setting ID after creating the object is that the object is in invalid state. You can render it but having no ID it will crash. So it's a workaround rather than proper solution to the problem.

I would go with rewriting the interface into usual class. It doesn't pollute presenters either and you have freedom to express yourself. It's a little bit more writing but my experience says it's worth it.

final class MultiFactory
{
	public function __construct(
		private DummyService1 $ds1,
		private DummyService2 $ds2,
	) {}

    public function createFooControl(int $id): FooControl
	{
		return new FooControl($id, $this->ds1, $this->ds2);
	}

 	public function createBarControl(int $id): BarControl
	{
		return new BarControl($id);
	}
}

Last edited by dakur (2023-07-27 10:53)

@MarekBartoš
Let me describe an issue that I experience using your solution.

I move $id parameter into constructor:

class FooControl extends \Nette\Application\UI\Control
{
	public function __construct(
		private DummyService1 $ds1,
		private DummyService2 $ds2,
		private int $id;	// Great, I do not need a setter any more
	) {}

Let's create FooControl:

public function createFooControl(int $id): FooControl
{
	return $this->fooControlFactory->create($id);
}

I receive an Nette\DI\ServiceCreationException:
Service of type FooControl: Parameter $id in FooControl::__construct() has no class type or default value, so its value must be specified.

I could assign a value to $id in constructor to make it work:

class FooControl extends \Nette\Application\UI\Control
{
	public function __construct(
		private DummyService1 $ds1,
		private DummyService2 $ds2,

		// Could be any value as it is overwritten by the caller,
		// i.e. by fooControlFactory->create($id)
		private int $id = 1;	// Ugly, but works
	) {}

However, that is far from ideal. What do you think?

Edit:
There maybe a problem with my config.neon:

- DummyService1
- DummyService2
- FooControl	# how do I say that $id should not be injected

When ContainerBuilder tries to build a container it bumps into FooControl. It looks at its constructor and finds out there are three parameters he should inject:

• DummyService1
• DummyService2
• $id

It can find and inject the first two. Third one is unknown. That might be the source of the problem.

Last edited by rp (2023-07-28 12:57)

You are supposed to create a factory which creates control and register it as a service. You are registering control instead of factory. Check my example, I updated it a bit.

@MarekBartoš
I see where the misunderstanding is coming from.

Constructor of FooControl in my example:

public function __construct(
	private DummyService1 $ds1,
	private DummyService2 $ds2,
) {}

Constructor of FooControl in your example:

public function __construct(int $id) {}

Your example omits Dummy services, therefore FooControl does not have to be registered in config.neon.

My FooControl requires Dummy services to be injected so I register it in config.neon (as well as FooControlFactory) and receive Nette\DI\ServiceCreationException. I am almost sure that I receive that exception because DI tries to inject $id parameter that cannot be found.

One way around it is @dakur suggestion to have services injected in MultiFactory constructor and pass them to factory methods:

public function __construct(
	private DummyService1 $ds1,
	private DummyService2 $ds2,
) {}

public function createFooControl(int $id): FooControl
{
	return new FooControl($id, $this->ds1, $this->ds2);
}

At this point I should let you know that I'm more than happy with my solution and there is no need, unless you want to discuss it in more detail, to spend time on it. Thank you @dakur and @MarekBartoš for your time and your suggestions.

@MarekBartoš

Yes, you are correct!

The mistake was in registering FooControl in DI. That threw that Exception. I thought it has to be registered otherwise nothing will be injected. But it will be automatically, exactly as you said.

So your suggestion really works and the advatage is that I can pass $id in constructor.

Presenter.php

$foo = $this->multiFactory->createFooControl($this->id);

MultiFactory.php

class MultiFactory
{
	public function __construct(
		private FooControlFactory $fooControlFactory
	){}
	public function createFooControl(int $id): FooControl
	{
		return $this->fooControlFactory->create($id);
	}
}

FooControlFactory.php

interface FooControlFactory
{
	// When Nette generates class from this interface
	// it looks at FooControl constructor and passes all
	// present dependencies that are also registered in config.neon.
	// On top of that it passes all params from create method
	// i.e. $id parameter
	public function create(int $id): FooControl;
}

FooControl.php

class FooControl extends Nette\Application\UI\Control
{
	public function __construct(
		// this is passed by create($id)
		private $id,

		// these will be injected automatically
		// no need to register FooControl in DI
		private DummyService1 $ds1,
		private DummyService2 $ds2,
	) {}
}

config.neon

- DummyService1
- DummyService2
- FooControlFactory
- MultiFactory

Thanks @MarekBartoš for your time and effort.