コンストラクターまたは宣言でクラスフィールドを初期化しますか？

私は最近C＃とJavaでプログラミングをしていて、クラスフィールドを初期化するのに最適な場所がどこか知りたいです。

宣言時に行う必要がありますか？：

public class Dice
{
    private int topFace = 1;
    private Random myRand = new Random();

    public void Roll()
    {
       // ......
    }
}

またはコンストラクタで？：

public class Dice
{
    private int topFace;
    private Random myRand;

    public Dice()
    {
        topFace = 1;
        myRand = new Random();
    }

    public void Roll()
    {
        // .....
    }
}

ベテランの何人かがベストプラクティスだと思うのは本当に気になります。一貫性を保ち、1つのアプローチに固執したい。

私のルール：

1. 宣言内のデフォルト値で初期化しないでください（null、false、0、0.0...）。
2. フィールドの値を変更するコンストラクターパラメーターがない場合は、宣言で初期化を優先します。
3. コンストラクター・パラメーターのためにフィールドの値が変更された場合は、コンストラクターに初期化を入れてください。
4. 実践に一貫性を持たせる（最も重要なルール）。

C＃では問題ではありません。あなたが与える2つのコードサンプルは完全に同等です。最初の例では、C＃コンパイラー（またはCLRですか？）は空のコンストラクターを構築し、それらがコンストラクター内にあるかのように変数を初期化します（Jon Skeetが以下のコメントで説明する微妙なニュアンスがあります）。すでにコンストラクタがある場合は、「上」の初期化はその上部に移動されます。

ベストプラクティスの観点では、前者は後者に比べてエラーが発生しにくいため、誰かが簡単に別のコンストラクタを追加して、連鎖することを忘れる可能性があります。

C＃のセマンティクスは、ここではJavaと少し異なります。C＃では、宣言での割り当ては、スーパークラスコンストラクターを呼び出す前に実行されます。Javaでは、この直後に「this」を使用できるようになり（特に匿名の内部クラスに役立ちます）、2つの形式のセマンティクスが実際に一致することを意味します。

可能であれば、フィールドをfinalにします。

一つ注意点があると思います。私はかつてそのようなエラーを犯しました：派生クラスの内部で、抽象基本クラスから継承されたフィールドを「宣言時に初期化」しようとしました。その結果、「ベース」と新しく宣言されたフィールドの2つのフィールドセットが存在し、デバッグにかなりの時間がかかりました。

レッスン：継承されたフィールドを初期化するには、コンストラクター内で行います。

あなたの例のタイプを想定すると、コンストラクタのフィールドを初期化することを間違いなく好む。例外的なケースは次のとおりです。

• 静的クラス/メソッドのフィールド
• static / final / et alと入力されたフィールド

私は常にクラスの一番上にあるフィールドリストを目次（ここに含まれるものであり、使用方法ではない）と考え、コンストラクターを紹介と考えています。もちろんメソッドは章です。

私があなたに言った場合、それは依存しますか？

私は一般的にすべてを初期化し、一貫した方法でそれを行います。はい、それは過度に明示的ですが、保守も少し簡単です。

パフォーマンスが心配な場合は、やらなければならないことだけを初期化し、費用対効果が最も高い領域に配置します。

リアルタイムシステムでは、変数と定数のどちらが必要かについて疑問を投げかけます。

C ++では、どちらの場所でも初期化を行わずにInit（）関数に移動することがよくあります。どうして？まあ、C ++では、オブジェクトの構築中に例外をスローする可能性のあるものを初期化している場合、メモリリークが発生します。

多くのさまざまな状況があります。

空のリストが必要です

状況は明らかです。リストを準備し、誰かがリストにアイテムを追加したときに例外がスローされないようにするだけです。

public class CsvFile
{
    private List<CsvRow> lines = new List<CsvRow>();

    public CsvFile()
    {
    }
}

私は価値を知っています

デフォルトで必要な値を正確に知っているか、他のロジックを使用する必要があります。

public class AdminTeam
{
    private List<string> usernames;

    public AdminTeam()
    {
         usernames = new List<string>() {"usernameA", "usernameB"};
    }
}

または

public class AdminTeam
{
    private List<string> usernames;

    public AdminTeam()
    {
         usernames = GetDefaultUsers(2);
    }
}

可能な値を持つ空のリスト

時にはデフォルトで空のリストを期待しますが、別のコンストラクタを介して値を追加する可能性があります。

public class AdminTeam
{
    private List<string> usernames = new List<string>();

    public AdminTeam()
    {
    }

    public AdminTeam(List<string> admins)
    {
         admins.ForEach(x => usernames.Add(x));
    }
}

Javaでは、宣言付きの初期化子は、使用されるコンストラクター（複数ある場合）またはコンストラクターのパラメーター（引数がある場合）に関係なく、フィールドは常に同じ方法で初期化されることを意味しますが、コンストラクターは後で値を変更します（最終でない場合）。したがって、宣言で初期化子を使用すると、初期化された値は、使用されるコンストラクターやコンストラクターに渡されるパラメーターに関係なく、すべての場合にフィールドが持つ値であることがリーダーに示唆されます。したがって、構築されたすべてのオブジェクトの値が同じである場合に限り、常に初期化子を宣言とともに使用します。

C＃の設計では、インライン初期化が推奨されます。そうでない場合は、言語ではありません。コード内の異なる場所間の相互参照を回避できる場合はいつでも、通常はより良い方法です。

静的フィールドの初期化と整合性の問題もあります。これは、最高のパフォーマンスを得るためにインラインにする必要があります。コンストラクター設計のフレームワーク設計ガイドラインは次のように述べています。

ランタイムは明示的に定義された静的コンストラクタを持たない型のパフォーマンスを最適化できるため、静的コンストラクタを明示的に使用するのではなく、インラインで静的フィールドを初期化することを検討してください。

この文脈での「考慮」とは、そうするべきである正当な理由がない限り、そうすることを意味します。静的初期化子フィールドの場合、正当な理由は、初期化が複雑すぎてインラインでコーディングできない場合です。

一貫性を保つことが重要ですが、これは自分自身に問うべき質問です。「他に何かコンストラクタはありますか？」

通常は、クラス自体が変数のハウジングとして機能する以外は何もしないデータ転送のモデルを作成しています。

これらのシナリオでは、通常、メソッドやコンストラクターはありません。特に宣言に合わせてリストを初期化できるので、リストを初期化するためだけにコンストラクタを作成するのはばかげたことでしょう。

他の多くの人が言ったように、それはあなたの使用法に依存します。シンプルに保ち、必要のないものを追加しないでください。

複数のコンストラクターがある状況を考えてみましょう。初期化はコンストラクターごとに異なりますか？それらが同じになる場合、なぜ各コンストラクタに対して繰り返すのですか？これはkokosステートメントと一致していますが、パラメーターに関連していない可能性があります。たとえば、オブジェクトがどのように作成されたかを示すフラグを保持したいとします。次に、そのフラグは、コンストラクターのパラメーターに関係なく、コンストラクターごとに異なる方法で初期化されます。一方、各コンストラクターに対して同じ初期化を繰り返すと、一部のコンストラクターでは初期化パラメーターを（意図せずに）変更する可能性がありますが、他のコンストラクターでは変更しない可能性があります。したがって、ここでの基本的な概念は、共通のコードには共通の場所があり、異なる場所で繰り返されないようにする必要があるということです。

宣言で値を設定すると、パフォーマンスが少し向上します。コンストラクタで設定すると、実際には2回設定されます（最初はデフォルト値に設定され、次にctorでリセットされます）。

私は通常、コンストラクターを試して、依存関係を取得し、関連するインスタンスメンバーを初期化するだけです。これにより、クラスを単体テストする場合の作業が楽になります。

インスタンス変数に割り当てる値が、コンストラクターに渡すパラメーターの影響を受けない場合は、宣言時に割り当てます。

ベストプラクティスに関する質問への直接の回答ではありませんが、重要で関連する再確認ポイントは、ジェネリッククラス定義の場合、コンパイラーに任せてデフォルト値で初期化するか、フィールドを初期化するために特別なメソッドを使用する必要があるということです。デフォルト値に変更します（コードを読みやすくするために絶対に必要な場合）。

class MyGeneric<T>
{
    T data;
    //T data = ""; // <-- ERROR
    //T data = 0; // <-- ERROR
    //T data = null; // <-- ERROR        

    public MyGeneric()
    {
        // All of the above errors would be errors here in constructor as well
    }
}

また、ジェネリックフィールドをデフォルト値に初期化する特別なメソッドは次のとおりです。

class MyGeneric<T>
{
    T data = default(T);

    public MyGeneric()
    {           
        // The same method can be used here in constructor
    }
}

ロジックやエラー処理が必要ない場合：

• 宣言時にクラスフィールドを初期化する

ロジックまたはエラー処理が必要な場合：

• コンストラクターでクラスフィールドを初期化する

これは、初期化値が使用可能であり、初期化を1行に置くことができる場合にうまく機能します。ただし、この形式の初期化は単純であるため、制限があります。初期化に何らかのロジック（たとえば、エラー処理や複雑な配列を満たすためのforループなど）が必要な場合、単純な割り当てでは不十分です。インスタンス変数はコンストラクターで初期化でき、エラー処理やその他のロジックを使用できます。

https://docs.oracle.com/javase/tutorial/java/javaOO/initial.htmlから。