从构造函数调用方法

Question

请原谅任何小的语法错误或诸如此类的问题，我正在使用 Jitsi 模块遇到这种情况，并且对 Java 不太熟悉，想确认发生了什么以及为什么以及如何修复它。

 public abstract class A
{
  public A()
  {
    this.load();
  }

  protected void load()
  {

  }
}

public class B extends A
{
  private String testString = null; 

  public B()
  {
    super();
  }

  @Override
  protected void load()
  {
    testString = "test";
  }
}

应用程序在使用按名称加载类的方法创建类 B 的实例时这样做：

• 在 B 类中调用重写的 load()
• 初始化变量（根据调试器调用“private string testString = null”），将它们清空。

这是预期的 Java 行为吗？什么可能导致这种情况？它是在 1.7 JDK 上运行的 Java 1.6 应用程序。

Answer 1

这是预期的 Java 行为吗？

是的。

是什么原因造成的？

您在非最终超类构造函数中调用非最终覆盖方法。

让我们一步一步来看看会发生什么：

• 您创建了一个B 的实例。
• B() 调用超类构造函数 - A()，初始化超类成员。
• A() 现在调用一个非 final 方法，该方法在 B 类中被覆盖，作为初始化的一部分。
• 由于上下文中的实例属于B 类，所以调用的方法load() 属于B 类。
• load() 初始化 B 类实例字段 - testString。
• 超类构造函数完成工作并返回（假设构造函数链接到Object类已经完成）
• B() 构造函数开始进一步执行，初始化它自己的成员。
• 现在，作为初始化过程的一部分，B 会覆盖testString 中先前写入的值，并将其重新初始化为null。

道德：永远不要在其构造函数中调用非最终类的非最终公共方法。

Answer 2

这是构造时初始化的常见问题模式，经常出现在基础架构代码和自制 DAO 中。

对“null”的赋值是不需要的，可以删除。

如果这还不足以作为一个快速补丁，那么： 将所有后期构造 init 移动到一个单独的方法中，并将其全部包装在一个“静态方法”伪构造函数中。

如果你在做 DAO 的事情，区分“加载”和“创建”真的很好，因为它们是完全不同的实例化。为这些定义单独的“静态构造函数”方法，也许还有单独的内部初始化。

abstract public class A {
    protected void initAfterCreate() {}
}

public class B {

    @Override
    protected void initAfterCreate() {
        this.testString = "test";
    }

    // static constructors;
    //     --        
    static public B createB() {
        B result = new B();
        result.initAfterCreate();
    }
}

演示 DAO 的加载/创建分离：

public class Order {
    protected int id;
    protected boolean dbExists;

    static public load (int id) {
        Order result = new Order( id, true);
        // populate from SQL query..
        return result;
    }
    static public create() {
        // allocate a key.
        int id = KeyAlloc.allocate( "Order");
        Order result = new Order( id, false);
    }

    // internal constructor;  not for external access.
    //
    protected Order (int id, boolean dbExists) {
        this.id = id;
        this.dbExists = dbExists;
    }
}