从构造函数调用方法

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行为吗？

是.

什么可能导致这个？

您在非final超类构造函数中调用非最终重写方法.

让我们一步一步看看会发生什么:

• 您创建一个实例B.
• B()调用超类构造函数 - A()来初始化超类成员.
• A()现在调用一个在B类中重写的非final方法,作为初始化的一部分.
• 由于上下文中的实例是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;
    }
}