依存性注入問題

Question

私は最近、依存性逆転原理と制御の反転に入った、と私は現在、（私が間違っているなら、私を修正）知っているから、そのDIPできましたが述べている

ハイレベルのモジュール/クラス（この1つは？何を意味するかわからない）の詳細に依存するべきではないハイ及びローレベルのモジュールの両方低レベルモジュール/クラス

に依存すべきではないが、抽象化（インターフェイス）

しかし、私が単純なプログラムを書いていたときこの原則を実装するために、私はこの問題に遭遇した（後で説明します）

私のプログラムに問題がある「ICharacterable」

 class Player : ICharacterable { 

    private int _health = 100; 

    public int Health{ 

     get{ 
      return _health; 
     } 

    } 

    public int HealthIncrease(int health){ 
     //increase health 
     return _health + health; 

    } 


} 

class Inventory{ 

    ICharacterable player = null; 

    //constructor injection 
    public Inventory(ICharacterable player) { 

     this.player = player; 

    } 

    //increase player health 
    public void UseHealthPotion(int health) { 

     player.HealthIncrease(health); 

    } 

} 

class Program{ 
static void Main(string[] args) { 

    Player john = new Player(); 
    Inventory inv = new Inventory(john); 

    inv.UseHealthPotion(30); 
    Console.WriteLine(john.Health); 

} 

} 

//interface 
public interface ICharacterable { 

    int HealthIncrease(int health); 

} 

を実装インベントリとプレイヤーのクラスで構成され、コンソールではなく、100を返します。私は、問題はPlayerの代わりにICharacterable型を注入していると宣言したという事実が原因であると考えています（しかし、それはDIPに違反します）とにかく私はこれを行うことができますか？あなたの方法でのみ加算の結果を返し、状態を変更しないので、

public int HealthIncrease(int health){ 
     //increase health 
     return _health += health; 

    } 

Answer 1

あなたは悪い結果を得る：ありがとう

Answer 2

これは、依存性注入の問題は、あなたはちょうどここにコードのバグを持っていませんオブジェクト。それ以外はすべて正しいです。それを変更します。

public int HealthIncrease(int health){ 
    //increase health 
    _health += health; 
    return _health; 
} 

あなたが唯一の私の意見では、プレーヤーを小さな交換を検討すべきであるインベントリではなく、その逆が含まれていなければなりません。そうすれば、OOPのデザインパターンに従うことになり、コードがはるかに改善されます。

ハイレベルモジュール/クラスが低レベル モジュール/クラスに依存すべきではない

この場合プレーヤー =ハイレベルクラスとインベントリ =低レベルクラス

高レベルモジュールと低レベルモジュールの両方が詳細に依存してはいけません（この意味はわかりません）。抽象（インターフェイス）

クイック説明：

抽象 =インターフェースや抽象クラス。かなり理解しやすいですが、あなたのICharactableインターフェイスです。

詳細 =実装。あなたの在庫クラスです。あなたのコードはこの原則と互換性があります。