java - 工厂模式 : Validating input when creating a bean (Guice/Guava)

Question

我正在编写一个新应用，使用 Guice 进行依赖注入(inject)，并使用 Guava 的先决条件进行属性验证。

我正在使用工厂模式根据外部输入创建 bean。问题是，验证输入的首选方式是什么？(在可维护性、清晰度等方面)

让我们假设这些类:

Bean.java

public interface Bean {

    public Object getFoo();

}

BeanImpl.java

public class BeanImpl implements Bean {

    private final Object foo;

    public BeanImpl(Object param) {
        foo = param;
    }

    @Override
    public String getFoo() {
        return foo;
    }

}

我想检查“param”是否包含“foo”的有效值:

Preconditions.checkArgument(SomeValidator.isValid(param), "Bad param: %s!", param);

我应该在哪里做，为什么？

• 在 BeanImpl 的构造函数中？ (我不确定为什么，但在构造函数中添加检查感觉是个坏主意。)
• 在调用 BeanFactory.create(param) 之前？ (听起来像是可怕的代码重复。)
• 通过一些我不知道的 Guice 机制？

Answer 1

如果用特定的 foo 值构造一个 BeanImpl 是无效的，那么在构造函数中抛出一个 IllegalArgumentException 详细说明错误.

public class BeanImpl {
  ...
  public BeanImpl(Object param) {
    if (param == null) {
      throw new IllegalArgumentException("Param cannot be null");
    }
    foo = param;
  }
  ...
}

根据早期对象的“状态和行为组合”定义，这强制您不要用无效状态污染您的对象。

就工厂而言，在构造对象之前检查值是一种便利，但工厂的真正职责并不扩展到对象的职责中。工厂模式有助于确保对象以有意义的方式与其他对象相对应，但它不应该完成对象的工作以确保对象的内部状态正确。

让一个对象(工厂)确保另一个对象的状态正确是错误的行为。您将行为(验证状态)放在工厂中，但状态在对象中。因此，您现在拥有分散连接的代码，这些代码应该紧密耦合。这意味着现在您将必须有一个工厂来制作对象，而不是有一个工厂在有意义的时候制作对象，而在没有意义的时候直接制作对象。

这经常出现在单元测试中。如果你没有把验证码放在工厂里，你可以独立对Object进行单元测试；测试错误的参数。然而，如果你把验证码放在工厂里，你将无法在没有工厂的情况下构造“有效”的对象，这是你得到的第一个提示，你不自然地耦合了两个不应该紧密耦合在一起的对象.

当然也有异常(exception)；但是当人们期望数据是正确的，但收集数据的方式没有提供验证时，它们往往会出现。例如，从网络套接字收集的结构化记录实际上可能包含概念上内部不一致的数据；但是，由于处理的性质，无效输入会被处理、记录和丢弃。