我一直在阅读
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);
- }
- }