コンストラクタ

Question

で可能な重複を初期化対宣言の初期化：
Should I initialize variable within constructor or outside constructor

を私はより良い練習と理由である、と思いまして。宣言時にクラスフィールドを初期化するか、コンストラクタでクラスフィールドを初期化する必要がありますか？それが単純な1行の初期化であるとすれば、

class Dude 
{ 
    String name = "El duderino"; 

    Dude() { 
     // irrelevant code 
    } 
} 

対

class Dude 
{ 
    String name; 

    Dude() { 
     name = "El duderino"; 

     // irrelevant code 
    } 
} 

編集：私はスタイルのいずれかが例外をスローする可能性がある初期化コードを実行する場合のように、他の上で好ましいであろう状況を認識してい。私がここで話しているのは、両方のスタイルが完全に同等である場合です。どちらの方法も同じタスクを達成します。どちらを使うべきですか？

Answer 1

メンバーはのみ可能であればアクセサー（ "セッター"メソッド）を介して設定する場合は、私は最初のスタイルを好む。それは、初期化された値が構築時のデフォルト値であるというヒントを提供します。

メンバを構築中に指定できる場合は、一般に、より少ないパラメータでコンストラクタから適切なコンストラクタにデフォルト値を渡します。たとえば、

final class Dude { 

    private final String name; 

    Dude() { 
    this("El Duderino"); 
    } 

    Dude(String name) { 
    this.name = name; 
    } 

} 

Answer 2

通常、最初の変数は静的変数を初期化するために使用され、その目的にのみ使用されます。

この場合、2番目の方法を使用する必要があります。

私が間違っている場合は、私に修正してください。

Answer 3

一貫性のために、コンストラクタ内で変数を宣言することをお勧めします。変数にはループやif-else文のようなものが必要になることがあります。これを初期化するには、メソッドの内部に操作を置かずに宣言で行うことはできません。

このルールの例外はスタティック変数で、コンストラクタの外側で宣言する必要があります。

Answer 4

単一行の宣言に複雑な初期化ロジックを含めることはできません。

あなたのように変数を初期化した場合：

class AnotherClass 
{ 
    MyClass anObject = new MyClass(); //MyClass() throws a checked exception. 
} 

そして、あなたは、単一の行に初期値を提供できないことがわかります。あなたはかなり明らかにコンストラクタ（または非static初期化ブロック内）内部に入るブロック、このようなコードを配置する必要があります：

コンストラクタを使用：

class AnotherClass 
{ 
    MyClass anObject; 

    AnotherClass() 
    { 
     try{this.anObject = new MyClass();}catch(SomeException e){/*handle exception.*/} 
    } 
} 

初期化ブロックを使用します：私は宣言および初期化が互いに分離されているように、後者は、あまり理解コードになり、そして実行時に差がないものの、初期化（開発者がコード化されたコンストラクタで発生していないことが判明

class AnotherClass 
{ 
    MyClass anObject; 

    { 
     try{this.anObject = new MyClass();}catch(SomeException e){/*handle exception.*/} 
    } 
} 

）。

フィールドの初期化に関連する他の複雑なルーチンについても同じことが言えます。あなたがArrayやCollectionを初期化し、いくつかのデフォルト値を配列/コレクションの内容を設定する場合たとえば、あなたは、コンストラクタ内でそうする必要があります。

class AnotherClass 
{ 
    Integer[] integers; 

    AnotherClass() 
    { 
     this.integers = new Integer[10]; 
     for(Integer integer: integers) 
     { 
      integer = Integer.MIN_VALUE; 
     } 
    } 
}