コンストラクタ内のオーバーライド可能なメソッド呼び出しの何が問題なのでしょうか? 質問する

コンストラクタからオーバーライド可能なメソッドを呼び出す場合

簡単に言えば、これは多くのバグの可能性を不必要に開くため間違っています。@Overrideが呼び出されると、オブジェクトの状態が矛盾したり不完全になったりする可能性があります。

Effective Java 2nd Edition の項目 17: 継承を考慮して設計および文書化する、または継承を禁止する からの引用:

継承を可能にするために、クラスが従わなければならない制限がさらにいくつかあります。コンストラクターは、直接的または間接的にオーバーライド可能なメソッドを呼び出してはなりません。この規則に違反すると、プログラムが失敗します。スーパークラスのコンストラクターはサブクラスのコンストラクターの前に実行されるため、サブクラスのオーバーライド メソッドはサブクラスのコンストラクターが実行される前に呼び出されます。オーバーライド メソッドがサブクラスのコンストラクターによって実行される初期化に依存している場合、メソッドは期待どおりに動作しません。

以下に例を挙げて説明します。

public class ConstructorCallsOverride {
    public static void main(String[] args) {

        abstract class Base {
            Base() {
                overrideMe();
            }
            abstract void overrideMe(); 
        }

        class Child extends Base {

            final int x;

            Child(int x) {
                this.x = x;
            }

            @Override
            void overrideMe() {
                System.out.println(x);
            }
        }
        new Child(42); // prints "0"
    }
}

public class Telescope {
    final String name;
    final int levels;
    final boolean isAdjustable;

    public Telescope(String name) {
        this(name, 5);
    }
    public Telescope(String name, int levels) {
        this(name, levels, false);
    }
    public Telescope(String name, int levels, boolean isAdjustable) {       
        this.name = name;
        this.levels = levels;
        this.isAdjustable = isAdjustable;
    }
}

new Telescope("X/1999");
new Telescope("X/1999", 13);
new Telescope("X/1999", 13, true);

Telescope telly = new Telescope.Builder("X/1999").setAdjustable(true).build();