UnityをIoCとして使用するコンストラクタ依存性注入

Question

私はコンボボックスに3つの項目を示すフォームを持っています。 大陸、国および都市

アイテムを選択すると、都市を選択し、「結果を取得」ボタンをクリックすると、ビジネスとデータレイヤーを介してデータベースに選択コマンドを送信し、タイプの都市のリストを取得します。

リストはUIフォームのグリッドにバインドされます。

クラス：大陸、国および都市は、プロパティ文字列「名前」を持つIEntitiesインターフェイスを実装しています。

click(object sender, EventArgs e) 
{ 
    string selectedItem = comboBox.SelectedItem; 
    IEntities entity = null; 
    List<IEntities> list = null; 

    if (selectedItem == "Cities") 
    { 
     entity = new Cities("City"); 
    } 

    if (selectedItem == "Continents") 
    { 
     entity = new Continents("Continents"); 
    } 

    if (selectedItem == "Countries") 
    { 
     entity = new Countries("Countries"); 
    } 

    //Then I call a method in Business Layer to return list 
    BL bl = new BL(entity); 
    list = bl.GetItems(); 
    myDataGrid.DataContext = list;//to bind grid to the list 
} 

ビジネス・レイヤーは、次のようになります：

public class BL 
{ 

    public IEntities _entity; 

    //constructor sets the variable 
    public BL(IEntity entity) 
    { 
     _entity = entity; 
    } 

    public IList<Entities> GetItems() 
    { 
     //call a method in data layer that communicates to the database 
     DL dl = new DL(); 
     return dl.CreateItemsFromDatabase(_entity.Name);//name decides which method to call 
    } 
} 

私が代わりに工場（の一種）を使用するので、IOCとしてユニティを使用したい

ボタンクリックイベントは、使用して、ビジネスレイヤを呼び出しますパターンがボタンのクリックイベントでelsesとハードコードされたクラス名を使用して、私は関連するクラスインスタンスを作成するコンテナの設定を使用したいと思います。 IEntitiesインスタンスがBLクラスのコンストラクタに渡されるとき、私はUnityを使ってオブジェクトを渡したいと思います。あなたはそれをする方法をアドバイスできますか？

Answer 1

このデザインは存在するため、IoCコンテナの組み込みにはあまり適していません。

限り、あなたのComboBoxはまだ文字列が含まれているとして、あなたはifブロックのどこかのセットswitch文またはにハードコードされた値に対してということを比較する必要があるとしています。

また、BLクラスタイプIEntityのコンストラクタパラメータを取り、それは実行時に多くの異なるタイプのうちのいずれかの対象とすることができます。起動時にUnityを設定してBLをインスタンス化する方法はありませんが、そのパラメータとして何を使用するか（実際には何も得られません）。

興味深いことに、stringという名前をCreateItemsFromDatabaseメソッドに渡す唯一の目的で、これらのEntityオブジェクトをインスタンス化しているようです。あなたは何のためにその型を使用していません。コンストラクタパラメータをすべてスキップして、選択したstringをComboBoxからGetItemsメソッドに直接渡して、同じ結果を得ることができるようです。これを行う他の理由がある場合は、少なくともコンストラクタにその名前を指定しないでください。それを各クラス宣言の中でconstにしてください。

より適切なのは、GetItemsを一般的な方法にすることです。 IEntityをBLコンストラクタに渡す代わりに、具体的な型をメソッドに渡します。

var bl = new BL(); 
var countries = bl.GetItems<Countries>(); 
var cities = bl.GetItems<Cities>(); 
var continents = bl.GetItems<Continents>();