I'm trying to design a set of factory classes for our system, where some objects created by the factory also need to be initialized before they can be used properly.
Example:
$foobar = new Foobar();
$foobar->init( $qux, ... );
// $foobar ready for usage
For the same of example, lets say that the $qux
object is the only dependency that Foobar
needs. What I'd like to get to is:
$foobar = Foo_Factory( 'bar' );
In order to avoid the need to pass along the $qux
object across the whole system and pass it to the factory class as another parameter, I'd like to perform initialization of Foobar
directly in the factory class:
class Foo_Factory {
public static function getFoo( $type ) {
// some processing here
$foo_name = 'Foo' . $type;
$foo = new $foo_name();
$foo->init( $qux );
return $foo;
}
}
There are few solutions that come to mind, but none of them is ideal:
$qux
to the factory class, and let it store a reference to $qux
in a private static variable. The system can set $qux
at the start, and the factory class can prevent any future changes (for security reasons).$qux
is problematic during unit testing (e.g. it happily lives survives between individual tests due to its static state).$qux
. This might be a bit cleaner way to do this than option #1 (although we move the static problem from the factory class to the context class).$qux
to any object which uses the factory class, and let that object pass it along to the factory class as another parameter: Foo_Factory::getFoo($type, $qux);
.$qux
along the system, pass an instance of the factory class instead (i.e. in this case it would not be static, but instantiable).What would you recommend please? Any of the four alternatives mentioned above, or is there a better way to do this please?
Note: I don't want to get into a static is evil
flamewar here, just trying to come up with the best solution.
I'd go with Dependency Injection all the way. But, instead of passing $qux along everywhere, just register it in the Dependency Injector Container and let the container sort it out. In Symfony Component speak:
// Create DI container
$container = new sfServiceContainerBuilder();
// Register Qux
$container->setService('qux', $qux);
// Or, to have the DI instanciate it
// $container->register('qux', 'QuxClass');
// Register Foobar
$container->register('foobar', 'Foobar')
->addArgument(new sfServiceReference('qux'));
// Alternative method, using the current init($qux) method
// Look! No factory required!
$container->register('altFoobar', 'Foobar')
->addMethodCall('init', array(new sfServiceReference('qux')));