薄いインターフェースと太いインターフェースの間の「細い線」とは何ですか？

Question

私はあなたが予約を予約し、既存の予約を変更し、既存の予約を取り消すことができる予約システムを持っています。私はInterface Segregation Principleを検討していましたが、私がインターフェースをどのように細くしなければならないのか、私が違反しているのかSingle Responsibility Principleを考えていました。

interface IReservation 
{ 
    void Book(); 
    void Modify(); 
    void Cancel(); 
} 

が、その後私は1つの予約システムならば、予約のためのこれらの方法のいずれかを実装する必要があり、例えば予約とちょうど懸念しているので、私は次のようでしたしないものを、考えた：私のintialデザインされました：

interface IReservation : IBooking 
{ 


} 

または

：

interface IBook 
{ 
    void Book(); 
} 


interface IModify 
{ 
    void Modify(); 
} 

interface ICancel 
{ 
    void Cancel(); 
} 

今、私はこのような何かを行うことができます

だから私はそれをこれほど薄くして遠くに持っていくことになります。また、IModifyやICancelが好きではないなど、インターフェースの名前を考えるのが難しくなります（IReservationインターフェース上にあるはずのメソッドのように思えます）。インターフェイスに入るべきものと、別のインターフェイス、クラスなどにセグメンテーションされるべきものを決定する方法...

Answer 1

あなたは、インターフェイスの範囲を見たときに考慮しなければならない二つのことがあります。

1. IReservationにこれらのメンバーを実装する必要がありますか？
2. メンバーX（メンバーなし）を参照することは意味がありますかY？

最初はあなたがカバーしたもので、「いいえ」の結論に達しました。第二は、それと同じくらい重要です。何かを "変更できる"と考えることは他に何もできなくても意味がありますか？そうでない場合は、IReservationとIModifiableReservation、またはその他の機能のグループ化を検討してください。

たとえば、CancelとModifyが手に入るので、両方を一緒にIModifiableReservationに置き、クラスにそのインターフェイスを実装させてください。

ご存じのように、少し細かすぎます。

Answer 2

アプリケーションが異なる種類の予約をサポートする必要があり、後でいくつかの一般的なロジックがそれらのすべてを扱う - 私は、サービスタイプごとに別々のインターフェイスを導入し、予約ごとに単一のインターフェイスを導入することを提案する。アイデア - 予約はサービスのセットを提供するので、共通インターフェイスIReservationServiceによって抽象化されたサービスのリストを公開し、各予約システムごとにただ、サービスごとに単一のクラスを作成し、予約のctorのを経由してサービスを登録します。

var reservationWithBooking = 
    new Reservation(new List<IReservationService { new BookingService() }); 

var reservationWithCancellation = 
    new Reservation(new List<IReservationService { new CancellationService(); }); 

var mixedReservation = 
    new Reservation(new List<IReservationService 
          { 
           new BookingService(), 
           new CancellationService() 
          }); 

インタフェース：

interface IReservationService 
{  
} 

interface IBookingService : IReservationService 
{ 
    void Book(...); 
} 

interface ICancellationService : IReservationService 
{ 
    void Cancel(...); 
} 

interface IReservation 
{ 
    IEnumerable<IReservationService> Services { get; } 
} 

class Reservation : IReservation 
{ 
    private IList<IReservationService> services; 

    public Reservation(IEnumerable<IReservationService> services) 
    { 
     this.services = new List<IReservationService>(services); 
    } 

    public IEnumerable Services<IReservationService> 
    { 
     get 
     { 
      return this.services; 
     } 
    } 
} 

Answer 3

私はあなたがIReservationインターフェイスを作成する必要がありますが、IBookableReservationとIModifiableReservationの両方から直接あなたのクラスを継承しない2つのインターフェイスAdam Robinson

interface IModifiableReservation 
{ 
    void Modify(); 
    void Cancel(); 
} 

によって

interface IBookableReservation 
{ 
    void Book(); 
} 

とas suggestedを持ってお勧めします。クライアントは、1つまたは両方のインターフェイスを使用できます。

インターフェイスを作成する必要はなく、ただ1つのクラスのパブリックメソッドを複製するだけです。 interfaceが "I"接頭辞を持つクラスと同じ名前を持つ場合、それは1:1 relationship between the interface and the concrete classes that implementであることを示すため、コードの匂いです。

複数 の実装を持っているつもりされていないときのインターフェイスを使用してReused Abstractions Principle (RAP)

とhttp://martinfowler.com/bliki/InterfaceImplementationPair.html

からを参照してくださいがsync.Furthermore ですべてを保つために余分な労力であるそれは実際にあなたのケースを隠します複数の の実装を提供してください。