从构造函数调用方法

2022-09-02 00:11:08

请原谅任何小的语法错误或其他什么,我在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()
  • 初始化变量(根据调试器调用“私有字符串 testString = null”),将它们清空。

这是预期的 Java 行为吗?这是什么原因造成的?这是一个在1.7 JDK上运行的Java 1.6应用程序。


答案 1

这是预期的 Java 行为吗?

是的。

这是什么原因造成的?

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

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

  • 创建 的实例。B
  • B()调用超类构造函数 - ,以初始化超类成员。A()
  • A()现在调用一个非 final 方法,该方法在类中被重写,作为初始化的一部分。B
  • 由于上下文中的实例是类的,因此调用的方法是类的。Bload()B
  • load()初始化类实例字段 - 。BtestString
  • 超类构造函数完成作业并返回(假设构造函数的链接直到类完成)Object
  • 构造函数开始进一步执行,初始化它自己的成员。B()
  • 现在,作为初始化过程的一部分,覆盖 中先前写入的值,并将其重新初始化为 。BtestStringnull

道德:切勿在其构造函数中调用非最终类的非最终公共方法。


答案 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;
    }
}