将来のAPIから切り離す - 依存関係注入の必要性を最初に発見する

Question

OOPデザインパターンについての質問はありません。これはオーバーアーキテクチャーですが、私は判断ではなく、推薦が必要です。 「ビジネス要件」によって指定されたクラス階層があるとします。したがって、Pizza : Freight、Flowers : Freight、およびTonOfBricks : Freightの深いバージョンがあり、IDispatchService<F> where F : FreightとIDeliveryService<F> where F : Freightが得られました。

異なるIDeliveryServiceの実装では、貨物に関するさまざまな情報が必要であるとします。彼らが望むものはあなたのクラス階層と平行しておらず、どの種類の小包について1か月にどの配送サービス（すなわちAPI）を使用するのか分からない。さらに、1本のピザを真空管で送るのに十分な情報があるかもしれませんが、次のものではないかもしれません。

これで、あなたはピザをローラーブレードで手渡す人を雇うこともできることに気づきましたが、この方法で配送されるすべての物の重心を指定する必要があります。そうしないと、保険があなたを殺します。 IDeliveryService<Pizza>として実装していますか？新しいインターフェイスIHasCenterOfMassを追加していて、おそらくIDeliveryService<IHasCenterOfMass>として実装していますか？もしそうなら、ディスパッチャーはどのようにそれを気に入っていますか？重心のデータベースに関係を追加していますか？

Answer 1

トリックは、同じ基本クラス/インタフェースを共有している限り、派生するすべてのものが別の派生クラスで変更可能でなければならないことを示す置換原則に違反するものではありません。したがって、私のお勧めは、あなたのサービスがその必要なサービスの特定の情報を含むXXXServiceInformationインスタンスを受け入れ、ソースから潜在的にそれを読む方法を知ることです。例：

interface IDeliveryService<F> where F : Freight 
{ 
    void Deliver(); 
} 

class RollerBladesDeliveryService<F> : IDeliveryService<F> where F: Freight 
{ 
    public RollerBladesDeliveryService(RollerBladesDeliveryInformation<F> information){ ... } 

    void Deliver() { ... } 
} 

class RollerBladesDeliveryInformation<F> where F: Freight 
{ 
    public RollerBladesDeliveryInformation(F freight, double centerOfMass) { ... } 
} 

Answer 2

あなたが知っていることを計画すると、ウサギの穴があなたが決して使用しないコードだけを作成するなら、何をすべきかを計画します。 IDeliveryServiceを実装できる3つのデリバリサービスが分かっていない場合は、汎用インターフェイスを作成できるかもしれませんが、将来的に必要なものに収まらない可能性があります。

Answer 3

は、あなたが「ビジネス要件」によって決定されるクラス階層があるとします。貨物、花：：貨物とTonOfBricks：だから、あなたはピザの深いバージョンを持っている貨物を、あなたはIDispatchServiceを持ってここで、F：貨物とIDeliveryService F：貨物

停止権利のことを。それは正しいOOPではありません。ピザや花は貨物ではありません。彼らは貨物の中身です。この区別は、単一の責任でよりクリーンなコードを生成するため、非常に重要です。

は今、各IPizzaDeliveryService実装は同じ種類の情報を必要とするさまざまなIDeliveryService実装が貨物

ない限り問題に関するさまざまな情報を必要とします。私は同意することはできません

foreach (var freight in queue) 
{ 
    var service = GetServiceFor(freight.ContentType); 
    service.Dispatch(freight); 
} 

更新

public interface IFreight 
{ 
    IFreightContent Content{get;} 
    Type ContentType { get { return Content.GetType() }} 
    PostalAddress DeliveryAddress {get;} 
} 


public interface IFreightContent 
{ 
} 

// we don't just deliver the pizza, do we? ;) 
public class PizzaBox : IFreightContent 
{ 
}