我一直在阅读
Factory模式,并且发现了一些文章,建议将Factory模式与依赖注入结合使用,以最大限度地提高可重用性和可测试性.虽然我还没有找到这个Factory-DI混合的任何具体例子,但我将尝试给出一些我的解释的代码示例.但是,我的问题是关于这种方法如何提高可测试性.
我的解释:
所以我们有一个Widget类:
public class Widget { // blah }
我们想要包含一个WidgetFactory来控制Widgets的构造:
public interface WidgetFactory { public abstract static Widget getWidget(); } public class StandardWidgetFactory implements WidgetFactory { @Override public final static Widget getWidget() { // Creates normal Widgets } } public class TestWidgetFactory implements WidgetFactory { @Override public final static Widget getWidget() { // Creates test/mock Widgets for unit testing purposes } }
虽然这个例子使用的是Spring DI(这是我唯一经验过的API),但是如果我们谈论的是Guice或任何其他的IoC框架并不重要;这里的想法是,我们现在要在运行时注入正确的WidgetFactory实现,具体取决于我们是在测试代码还是正常运行.在Spring中,bean配置可能如下所示:
<bean id="widget-factory" class="org.me.myproject.StandardWidgetFactory"/> <bean id="test-widget-factory" class="org.me.myproject.TestWidgetFactory"/> <bean id="injected-factory" ref="${valueWillBeStdOrTestDependingOnEnvProp}"/>
然后,在代码中:
WidgetFactory wf = applicationContext.getBean("injected-factory"); Widget w = wf.getWidget();
这样,环境(部署级别)变量(可能在.properties文件中定义)决定Spring DI是否会注入StandardWidgetFactory或TestWidgetFactory.
我做得对吗?!?这似乎是一个非常多的基础设施只是为我的Widget获得良好的可测试性.并不是说我反对它,但这对我来说只是过度工程化.
我的挂断:
我问这个的原因是因为我将在其他包中有其他对象,其中包含在其中使用Widget对象的方法.也许是这样的:
public class Fizz { public void doSomething() { WidgetFactory wf = applicationContext.getBean("injected-factory"); Widget widget = wf.getWidget(); int foo = widget.calculatePremable(this.rippleFactor); doSomethingElse(foo); } }
如果没有这个看似过度设计的庞大设置,我就无法将“模拟小部件”注入到Fizz :: doSomething()的单元测试中.
所以我被撕裂了:一方面,我觉得我正在过度思考 – 我可能正在做的事情(如果我的解释不正确).另一方面,我没有看到任何干净的方式绕过它.
作为一个切向问题,这也引起了我的另一个巨大关注:如果我的解释是正确的(甚至有些正确),那么这是否意味着我们需要每个对象的工厂?!?
这听起来像过度工程!什么是截止?什么是沙子中的线条划分何时使用工厂,什么时候不用?!感谢您对冗长问题的任何帮助和道歉.这让我头晕目眩.
解决方法
DI / IoC有助于测试,因为您可以轻松地决定使用哪种实现,而无需修改使用它的代码.这意味着您可以注入已知的实现来执行特定功能,例如,模拟Web服务故障,保证对函数的良好(或不良)输入等.
工厂不需要使DI / IoC工作.是否需要工厂完全取决于使用细节.
public class Fizz { @Inject // Guice,new JEE,etc. or @Autowired // Spring,or private Widget widget; public void doSomething() { int foo = widget.calculatePremable(this.rippleFactor); doSomethingElse(foo); } }