Javaの最終フィールドを初期化する

Question

私は2つのコンストラクタのうちの1つを使用してインスタンス化できる多くの最終メンバーを持つクラスを持っています。コンストラクタは、第3のコンストラクタに格納されているコードを共有します。

// SubTypeOne and SubTypeTwo both extend SuperType 

public class MyClass { 
    private final SomeType one; 
    private final SuperType two; 


    private MyClass(SomeType commonArg) { 
     one = commonArg; 
    } 

    public MyClass(SomeType commonArg, int intIn) { 
     this(commonArg); 

     two = new SubTypeOne(intIn); 
    } 

    public MyClass(SomeType commonArg, String stringIn) { 
     this(commonArg); 

     two = new SubTypeTwo(stringIn); 
    } 

問題は、このコードはコンパイルできないということです：Variable 'two' might not have been initialized.誰かが、おそらくMyClassの内側からの最初のコンストラクタを呼び出すことができますし、新しいオブジェクトには、「2」のフィールドが設定されていないだろう。

この場合、コンストラクタ間でコードを共有するにはどうすればよいでしょうか？通常はヘルパーメソッドを使用しますが、共有コードはコンストラクタからのみ実行できる最終変数を設定できる必要があります。

Answer 1

これはいかがですか？あなたがMyClass(SomeType oneIn)と呼ばれていた場合、twoが初期化されていないため、このエラーが出る

public class MyClass { 

    private final SomeType one; 
    private final SuperType two; 

    public MyClass (SomeType commonArg, int intIn) { 
     this(commonArg, new SubTypeOne(intIn)); 
    } 

    public MyClass (SomeType commonArg, String stringIn) { 
     this(commonArg, new SubTypeTwo(stringIn)); 
    } 

    private MyClass (SomeType commonArg, SuperType twoIn) { 
     one = commonArg; 
     two = twoIn; 
    } 
} 

Answer 2

「2」が初期化されていることを確認するだけです。最初のコンストラクタでは、ちょうど追加：

two = null; 

をあなただけの最初のコンストラクタが呼び出された場合に、それを与えるしたいのですが、いくつかの他の値がありますしない限り。

Answer 3

（変更された質問のために更新）。

Answer 4

すべての最終変数をすべてのコンストラクタで初期化する必要があります。私がやることは、すべての変数を初期化し、他のすべてのコンストラクタがそれを呼び出して、nullまたは値が与えられていないフィールドがある場合は何らかのデフォルト値を渡すようなコンストラクタを1つ持つことです。

例：

public class MyClass { 
    private final SomeType one; 
    private final SuperType two; 

    //constructor that initializes all variables 
    public MyClas(SomeType _one, SuperType _two) { 
     one = _one; 
     two = _two; 
    } 

    private MyClass(SomeType _one) { 
     this(_one, null); 
    } 

    public MyClass(SomeType _one, SubTypeOne _two) { 
     this(_one, _two); 
    } 

    public MyClass(SomeType _one, SubTypeTwo _two) { 
     this(_one, _two); 
    } 
}