依存関係の注入は、ctorで実行するか、メソッドごとに実行する必要がありますか？

考慮してください：

public class CtorInjectionExample
{
    public CtorInjectionExample(ISomeRepository SomeRepositoryIn, IOtherRepository OtherRepositoryIn)
    {
        this._someRepository = SomeRepositoryIn;
        this._otherRepository = OtherRepositoryIn;
    }

    public void SomeMethod()
    {
        //use this._someRepository
    }

    public void OtherMethod()
    {
        //use this._otherRepository
    }
}

に対して：

public class MethodInjectionExample
{
    public MethodInjectionExample()
    {
    }

    public void SomeMethod(ISomeRepository SomeRepositoryIn)
    {
        //use SomeRepositoryIn
    }

    public void OtherMethod(IOtherRepository OtherRepositoryIn)
    {
        //use OtherRepositoryIn
    }
}

Ctorインジェクションは拡張を困難にしますが（新しい依存関係が追加されたときにctorを呼び出すコードは更新する必要があります）、メソッドレベルのインジェクションはクラスレベルの依存関係からカプセル化されたままになり、これらのアプローチに対する/他の引数を見つけることができません。

注射のための決定的なアプローチはありますか？

（NB私はこれに関する情報を検索し、この質問を客観的にしようとしました。）

依存するのは、注入されたオブジェクトがクラス内の複数のメソッドで使用されており、各メソッドに挿入しても意味がない場合、メソッド呼び出しごとに依存関係を解決する必要がありますが、依存関係が使用されるのは1つのメソッドをコンストラクターに挿入するのは適切ではありませんが、使用できないリソースを割り当てるためです。

まず最初に、「新しい依存関係が追加されたときに、ctorを呼び出すコードはすべて更新する必要があります」に対処しましょう。明確にするために、依存関係の注入を行っており、依存関係を持つオブジェクトでnew（）を呼び出すコードがある場合、それは間違っています。

DIコンテナは、関連するすべての依存関係を注入できる必要があります。そのため、コンストラクターの署名を変更する必要はありません。そのため、引数は実際には保持されません。

メソッドごとのインジェクションとクラスごとのインジェクションの考え方については、メソッドごとのインジェクションには2つの大きな問題があります。

1つの問題は、クラスのメソッドが依存関係を共有する必要があることです。多数の依存関係（おそらく4〜5以上）を持つクラスが表示された場合、そのクラスが最有力候補になります。 2つのクラスにリファクタリングします。

次の問題は、メソッドごとに依存関係を「注入」するために、それらをメソッド呼び出しに渡す必要があることです。これは、メソッド呼び出しの前に依存関係を解決する必要があることを意味します。したがって、おそらく次のようなコードの束になります。

var someDependency = ServiceLocator.Resolve<ISomeDependency>();
var something = classBeingInjected.DoStuff(someDependency);

ここで、アプリケーションの10か所でこのメソッドを呼び出すとしましょう。これらのスニペットは10個あります。次に、別の依存関係をDoStuff（）に追加する必要があるとしましょう：そのスニペットを10回変更する必要があります（またはメソッドでラップする必要があります。この場合、DI動作を手動で複製するだけで無駄です）時間の）。

したがって、基本的にあなたがやったことは、DI利用クラスに独自のDIコンテナを認識させることです。これは根本的に悪い考えです

それをコンストラクター注入と比較してください。コンストラクターインジェクションでは、特定のDIコンテナーに縛られることはなく、クラスの依存関係を満たすことは直接責任を負わないため、メンテナンスはかなり頭痛のないものです。

無関係なヘルパーメソッドを含むクラスにIoCを適用しようとしているように思えますが、使用法に基づいてヘルパークラスをいくつかのサービスクラスに分割し、ヘルパーを使用して呼び出します。これはまだ素晴らしいアプローチではありません（単に渡される引数を処理するよりも複雑なことをするメソッドでクラス化されたヘルパーは、一般的に間違って記述されたサービスクラスにすぎません）が、少なくともデザインを少しきれいに保つでしょう。

（注：前に提案したアプローチを実行しましたが、それ以降は繰り返さないことは非常に悪い考えでした。結局、本当に分離する必要のないクラスを分離しようとしていました。各メソッド呼び出しが他のインターフェイスのほぼ固定された選択を必要とするインターフェイスのセットを使用します。これは維持するのに悪夢でした。）

controlの反転にはさまざまな種類があり、質問では2つしか提案されていません（ファクトリを使用して依存関係を注入することもできます）。

答えは次のとおりです。ニーズに依存します。ある種類と別の種類を使用した方が良い場合があります。

コンストラクターはオブジェクトを完全に初期化するためにあるので、個人的にはコンストラクターインジェクターが好きです。後でセッターを呼び出す必要があるため、オブジェクトは完全には構築されません。

少なくとも私にとって最も柔軟なもの、つまりプロパティインジェクションを逃しました。Castle / Windsorを使用し、クラスに新しいautoプロパティを追加するだけで、問題なく、インターフェイスを壊すことなく、注入されたオブジェクトを取得できます。