1つのコンストラクタが別のコンストラクタを実装すると、利点や欠点はありますか？

Question

私はこのようなクラスがある場合：私は、クラスをファクタし直すかの段階で

public class Foo 
{ 
    public IEnumerable<Bar> Bars { get; set; } 

    public Foo() 
    { 
     Bars = new List<Bar>(); 
    } 
} 

をし、このような最初のものを実装して、二次のコンストラクタを追加：

public class Foo 
{ 
    public IEnumerable<Bar> Bars { get; set; } 

    // some more properties were added 

    public Foo() 
    { 
     Bars = new List<Bar>(); 
    } 

    public Foo(string parameter): this() 
    { 
     .... some code here 
    } 
} 

を私も持っている可能性があり

public class Foo 
{ 
    public IEnumerable<Bar> Bars { get; set; } 

    // some more properties were added too 

    public Foo() 
    { 
     InitilizeFoo(); 
    } 

    public Foo(string parameter) 
    { 
     InitilizeFoo(); 
     .... some code here 
    } 

    private void InitializeFoo() 
    { 
     Bars = new List<Bar>(); 
    } 
} 

このシナリオではどちらのアプローチも有効ですが、どちらか一方を使用するとメリットか欠点がありますか？

は、より効率的なconstrcutorsを継承し、そのコードを高速に実行することや、私が代わりに第2の実施の効率化を知らない欠点がありますか。

Answer 1

1つのコンストラクタは、別のコンストラクタを呼び出す持つの重要な利点の一つは、あなたが道、あなたが非コンストラクタメソッドを呼び出すことによってそれを行うことはできません、読み取り専用のフィールドを設定することができるということです。例えば

：

public class Foo 
{ 
    private readonly int myNumber; 

    public Foo() : this(42) 
    { 
    } 

    public Foo(int num) 
    { 
     myNumber = num; 
    } 
} 

パフォーマンスが賢明、それは別のメソッドを呼び出すよりも、別のコンストラクタを呼び出すために、おそらく何も多かれ少なかれ、効率的ではないが、コンストラクタは、別の呼び出しすることが、私の意見では、より読みやすいですコンストラクタによって呼び出される点のみを持つ別のプライベートメソッドを呼び出すよりも、

別のメソッドを持つとき、当然のことながら、状況になる可能性があります理にかなっている、そしてそれは確かにそれ自体が「間違っている」ではありません。チェーニングコンストラクタはほとんどの用途で多くの方が読みやすく、パフォーマンスに負の影響はありません。

UPDATE：私は（プライベート初期化メソッド対チェーン）の各道の10,000,000回の反復を行い、その結果は、彼らはほとんど区別がつかなかったので、近くにあった。

Initializer Method took: 84 ms for 10,000,000 iterations, 8.4E-06 ms/each. 
Chained Constructors took: 81 ms for 10,000,000 iterations, 8.1E-06 ms/each. 

だから、本当に、性能面に近いがあるなしいずれにせよ恩恵を受ける主な利点は、readonlyフィールドを設定できる連鎖コンストラクタであり、ほとんどの場合、より読みやすくなります。

Answer 2

キーが重複したコードの量を減らすことです。この場合、パラメータ化されたコンストラクタから基本コンストラクタを呼び出すと、後で両方を更新することを忘れた後にバグを追加する機会が減ります。コンストラクタの連鎖

Answer 3

は、SRPとプログラムの流れを強制するための良い方法です。あなたもそれを「初期化」することがありますオブジェクトのライフサイクルにおける他の状況がある場合は、スタンドアロンInitialize()関数内で初期化コードを非表示にする意味を作ることができます。おそらく、それを素早くインスタンス化して遅延初期化できるようにしたければ、しかし、その機能を実行するためのライフサイクルの唯一の有効な時間がインスタンス化中であり、初期化が明確に定義された離散ステップのセットであれば、チェーン化はそれを容易にします。

Answer 4

Initialize（）関数を使用する利点は、オブジェクトをリセットしたい場合です。&オブジェクトを再作成するのではなく、単にinit関数を再度呼び出すことができます。

Answer 5

私はこれを言ってやけどかもしれないが、私はこのケースでは、デフォルトパラメータを使用して好む：私は10だった場合持っていた

public Foo(string parameter = null) 
{ 

} 

から15のオプションのパラメータとなる15種類のコンストラクタをしません私の意見ではエレガントなソリューションです。私は、デフォルトのパラメータが4.0フレームワークでのみ再導入されたと思います。