java - 构造函数与 setter 注入(inject)

Question

我目前正在设计一个 API，希望允许通过各种方法进行配置。一种方法是通过 XML 配置模式，另一种方法是通过我希望与 Spring 很好地配合使用的 API。

我的 XML 模式解析代码以前是隐藏的，因此唯一关心的是它是否能正常工作，但现在我希望构建一个公共(public) API，而且我非常关心最佳实践。

似乎很多人喜欢带有默认零参数构造函数的 javabean 类型 PoJo，然后是 setter 注入(inject)。我试图解决的问题是一些 setter 方法的实现依赖于在它们之前按顺序调用的其他 setter 方法。

我可以编写允许自己在许多命令中被调用的 anal setter，但这不会解决用户忘记设置适当的 setter 从而导致 bean 处于不完整状态的问题。

我能想到的唯一解决方案是忘记对象 'beans' 并通过构造函数注入(inject)强制执行所需的参数。

这方面的一个例子是基于父组件 id 的组件 id 的默认设置。

我的界面

public interface IMyIdentityInterface {
    public String getId();
    /* A null value should create a unique meaningful default */
    public void setId(String id);
    public IMyIdentityInterface getParent();
    public void setParent(IMyIdentityInterface parent);
}

接口(interface)的基本实现:

public abstract class MyIdentityBaseClass implements IMyIdentityInterface {
   private String _id;
   private IMyIdentityInterface _parent;

   public MyIdentityBaseClass () {}

   @Override
   public String getId() {
      return _id;
   }

   /**
    * If the id is null, then use the id of the parent component
    * appended with a lower-cased simple name of the current impl
    * class along with a counter suffix to enforce uniqueness
    */
   @Override
   public void setId(String id) {
      if (id == null) {
          IMyIdentityInterface parent = getParent();
          if (parent == null) {
              // this may be the top level component or it may be that
              // the user called setId() before setParent(..)
          } else {
              _id = Helpers.makeIdFromParent(parent,getClass());
          }
      } else {
          _id = id;
      }
   }

   @Override
   public IMyIdentityInterface getParent() {
      return _parent;
   }

   @Override
   public void setParent(IMyIdentityInterface parent) {
      _parent = parent;
   }

}

除了顶级组件之外，框架中的每个组件都有一个父组件。使用 setter 类型的注入(inject)，然后 setter 将根据调用 setter 的顺序具有不同的行为。

在这种情况下，您是否同意，构造函数采用对父级的引用并从接口(interface)中完全删除父级 setter 方法更好？如果我希望能够使用 IoC 容器配置这些组件，这会被认为是不好的做法吗？

克里斯

Answer 1

我真的不认为问题是 setter 与构造函数注入(inject)。问题是你使用 null 来表示太多的东西......

至于:/* 空值应该创建一个唯一有意义的默认值 */

将它用作 setId() 方法中的“神奇”值也确实看起来像是一种糟糕的代码味道。如果您从不调用 setId()，则 getId() 返回 null，但如果您调用 setId(null)然后 getId() 返回一些生成的值？

这样的事情似乎更有意义:

public abstract class MyIdentityBaseClass implements IMyIdentityInterface {
    private String _id;
    private IMyIdentityInterface _parent;

    public MyIdentityBaseClass () {}

    @Override
    public String getId() {
        return _id;
    }

    @Override
    public void setId(String id) {
        _id = id;
    }

    @Override
    public IMyIdentityInterface getParent() {
        return _parent;
    }

    @Override
    public void setParent(IMyIdentityInterface parent) {
        _parent = parent;
        if (_id == null) {
            // if id isn't already set, set it to the generated id.
            _id = Helpers.makeIdFromParent(parent, getClass());
        }
    }

}

如果您的模型确实需要父级随意使用构造函数注入(inject)，那就是它的用途。但是意识到你仍然需要为顶级项目提供默认构造函数......然后是什么来阻止某人对非顶级项目使用默认构造函数然后调用 setParent()稍后再说。我会考虑创建某种容器/组件架构，并为顶级项目创建一个(或多个)特殊类。 (您可能已经有了这个，因为您当时只给出了一个示例类和一个抽象类。)

顺便说一句 - 此外，IInterface 和 _name 约定不是 Java 命名约定。如果这是其他人应该使用的公开 API，则可能会有一些提示。