java - 从 Java 构造函数调用实例方法是好是坏？

Question

有几种不同的方法可以初始化复杂对象(注入(inject)依赖项和注入(inject)成员的必需设置)，这些方法看起来都很合理，但各有优缺点。我举一个具体的例子:

final class MyClass {
  private final Dependency dependency;
  @Inject public MyClass(Dependency dependency) {
    this.dependency = dependency;
    dependency.addHandler(new Handler() {
      @Override void handle(int foo) { MyClass.this.doSomething(foo); }
    });
    doSomething(0);
  }
  private void doSomething(int foo) { dependency.doSomethingElse(foo+1); }
}

如您所见，构造函数做了 3 件事，包括调用实例方法。有人告诉我，从构造函数调用实例方法是不安全的，因为它绕过了编译器对未初始化成员的检查。 IE。我本可以在设置 this.dependency 之前调用 doSomething(0)，这会编译但不起作用。重构它的最佳方法是什么？

1. 将 doSomething 设为静态并显式传递依赖关系？在我的实际案例中，我有三个实例方法和三个成员字段，它们都相互依赖，所以这似乎是很多额外的样板来使所有这三个都是静态的。

2. 将 addHandler 和 doSomething 移动到 @Inject public void init() 方法中。虽然与 Guice 一起使用将是透明的，但它需要任何手动构造以确保调用 init() 否则如果有人忘记该对象将无法完全发挥作用。此外，这会暴露更多的 API，这两个想法似乎都是坏主意。

3. 包装一个嵌套类以保持依赖性，以确保它在不暴露额外 API 的情况下正常运行:

   class DependencyManager {
     private final Dependency dependency;
     public DependecyManager(Dependency dependency) { ... }
     public doSomething(int foo) { ... }
   }
   @Inject public MyClass(Dependency dependency) {
     DependencyManager manager = new DependencyManager(dependency);
     manager.doSomething(0);
   }

   这会从所有构造函数中提取实例方法，但会生成一个额外的类层，当我已经有了内部类和匿名类(例如那个处理程序)时，它可能会变得困惑——当我尝试这个时，我被告知要移动 DependencyManager 到一个单独的文件，这也是令人反感的，因为它现在是多个文件来做一件事情。

那么处理这种情况的首选方法是什么？

Answer 1

Effective Java 中的 Josh Bloch 建议使用静态工厂方法，尽管我找不到针对这种情况的任何论据。然而，Java Concurrency in Practice 中也有类似的情况。 ，专门用于防止从构造函数中泄漏对 this 的引用。应用于这种情况，它看起来像:

final class MyClass {
  private final Dependency dependency;

  private MyClass(Dependency dependency) {
    this.dependency = dependency;
  }

  public static createInstance(Dependency dependency) {
    MyClass instance = new MyClass(dependency);
    dependency.addHandler(new Handler() {
      @Override void handle(int foo) { instance.doSomething(foo); }
    });
    instance.doSomething(0);
    return instance;
  }
  ...
}

但是，这可能不适用于您使用的 DI 注释。