瘦接口和胖接口之间的“细线”是什么？

Question

我有一个预订系统，可让您预订、修改现有预订和取消现有预订。我正在研究接口隔离原则，我想知道我应该使我的接口有多薄，以及我是否违反了单一责任原则。我最初的设计是：

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

但后来我想，如果一个预订系统不需要为预订实现这些方法之一并且只关注预订，例如，我做了以下操作：

interface IBook
{
     void Book();
}


interface IModify
{
    void Modify();
}

interface ICancel
{
    void Cancel();
}

现在我可以这样做了：

interface IReservation : IBooking
{


}

或

interface IReservation : IBooking, IModify
{


}

所以问题就变成了我是否将它像这样细化了。此外，为接口考虑名称变得更加困难，例如，我不喜欢 IModify 或 ICancel （它们对我来说就像应该在 IReservation 接口上的方法）。你如何确定什么应该进入一个接口，什么应该被隔离到另一个接口、类等等......

Answer 1

我建议有两个接口

interface IBookableReservation
{
     void Book();
}

和as suggestedAdam Robinson

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

您无需创建 IReservation 接口，而是直接从 IBookableReservation 和 IModifiableReservation 继承您的类。客户端可以使用一个或两个接口。

创建一个接口是没有意义的，它只是复制单个类的公共方法。 如果 interface 与仅带有“I”前缀的类具有相同的名称，则这是代码异味，因为它表明它是 1:1 relationship between the interface and the concrete classes that implementit。

见Reused Abstractions Principle (RAP)

来自http://martinfowler.com/bliki/InterfaceImplementationPair.html

当您不打算拥有多个接口时使用接口 实现是保持一切同步的额外努力。此外 它隐藏了您实际提供多个的情况 实现。

Answer 2

如果您的应用程序确实需要支持不同类型的预订，并且稍后一些通用逻辑应该能够处理所有这些 - 我建议为每种服务类型引入单独的接口，并为每个预订本身引入单个接口，这个想法 - 预订提供集合服务，因此您可以只公开由公共接口IReservationService 抽象的服务列表，并摆脱每个预订系统的多个接口实现。只需为每个服务创建一个类并通过 ctor of Reservation 注册服务：

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

在查看接口范围时，您必须考虑两件事：

1. 要求每个IReservation 实现这些成员是否有意义？
2. 在没有成员 Y 的情况下引用成员 X 是否有意义？

第一个是您所涵盖的内容，并且得出了“否”的结论。第二个同样重要。将某些东西视为“可以修改”而它不能做任何其他事情是否有意义？如果没有，请考虑制作IReservation 和IModifiableReservation，或其他一些功能分组。

例如，Cancel 和 Modify 似乎是相辅相成的，因此可能希望将它们放在一起在 IModifiableReservation 上，然后让您的类实现该接口。

如你所见，这似乎有点过于细化了。