2

假设出于不同的原因我想对集合进行抽象操作：

现在为了简单起见，让我们对一个集合进行推理

class Book {
  public string Title { get; set; };
  public string SubTitle { get; set; }
  public bool IsSold { get; set; }
  public DateTime SoldDate { get; set; }
  public int Volums { get; set; }
}

我有一种只需要搜索Book::Title（区分大小写或不区分大小写）的类型，因此我可以定义我的抽象：

interface ITitleSearcher {
  bool ContainsTitle(string title);
}

然后实施

class CaseSensitiveTitleSearcher : ITitleSearcher { ... }
class NoCaseSensitiveTitleSearcher : ITitleSearcher { ... }

并将其作为

class TitleSearcherConsumer  {
  public TitleSearcherConsumer(ITitleSearcher searcher) { // <- ctor injection
  }
}

直到这里我都清楚了，据我所知，接口隔离原则也得到了遵守。

继续开发我必须满足其他要求，所以我定义然后实现其他接口，ITitleSearcher例如：

class CaseSensitiveSubTitleSearcher : ISubTitleSearcher { ... }
class SoldWithDateRangeSearcher : ISoldDateRangeSearcher { ... }

为了不违反 DRY（不要重复自己），我可以创建一个包装器IEnumerable<Book>：

class BookCollection : ITitleSearcher, ISubTitleSearcher, ISoldDateRangeSearcher
{
  private readonly IEnumerable<Book> books;

  public BookCollection(IEnumerable<Book> books)
  {
    this.books = books;
  }
  //...
}

现在，如果我有一个像我这样的消费者，TitleSearcherConsumer我可以毫无问题地传递一个BookCollection.

但如果我有这样的消费者：

class TitleAndSoldSearcherConsumer {
  public TitleAndSoldSearcherConsumer(ITitleSearcher src1, ISoldDateRangeSearcher src2) {

  }
}

我无法将BookCollection实例注入TitleAndSoldSearcherConsumerctor；我必须传递每个接口的实现。

是的，我可以IBookCollection用其他接口的所有方法定义一个并在所有消费者中使用它，但是这样做不会违反 ISP 吗？

我可以同时靠近 ISP/SOLID 和 DRY 吗？

4

2 回答 2

8

是的，我可以用其他接口的所有方法定义一个 IBookCollection 并在所有消费者中使用它，但这样做不会违反 ISP 吗？

您不会违反 ISP，但您的藏书将开始承担太多责任，您将违反单一责任原则。

让我担心的另一件事是ITitleSearcher接口的多个实现。我不确定这里是否违反了某些设计原则，但您的设计中似乎有些含糊不清，您可能应该看看。此外，对于每个搜索操作，您都在创建一个新的抽象。您已经拥有ITitleSearcher,ISubTitleSearcher和ISoldDateRangeSearcher并且可能会再添加几十个。我认为您在这里缺少的是对系统中查询的一般抽象。所以这是你可以做的：

定义查询参数的抽象：

public interface IQuery<TResult> { }

这是一个没有成员的接口，只有一个泛型类型TResult。描述了该TResult查询的返回类型。例如，您可以按如下方式定义查询：

public class SearchBooksByTitleCaseInsensitiveQuery : IQuery<Book[]>
{
    public string Title;
}

这是接受 aTitle并返回的查询的定义Book[]。

您还需要对知道如何处理特定查询的类进行抽象：

public interface IQueryHandler<TQuery, TResult>
    where TQuery : IQuery<TResult>
{
    TResult Handle(TQuery query);
}

看看该方法如何接受 aTQuery并返回 a TResult？一个实现可能如下所示：

public class SearchBooksByTitleCaseInsensitiveQueryHandler :
    IQueryHandler<SearchBooksByTitleCaseInsensitiveQuery, Book[]>
{
    private readonly IRepository<Book> bookRepository;

    public SearchBooksByTitleCaseInsensitiveQueryHandler(
        IRepository<Book> bookRepository) {
        this.bookRepository = bookRepository;
    }

    public Book[] Handle(SearchBooksByTitleCaseInsensitiveQuery query) {
        return (
            from book in this.bookRepository.GetAll()
            where book.Title.StartsWith(query.Title)
            select book)
            .ToArray();
     }
}

现在消费者可以依赖这样的特定IQueryHandler<TQuery, TResult>实现：

class TitleSearcherConsumer  {
    IQueryHandler<SearchBooksByTitleCaseInsensitiveQuery, Book[]> query;
    public TitleSearcherConsumer(
      IQueryHandler<SearchBooksByTitleCaseInsensitiveQuery, Book[]> query) {
    }

    public void SomeOperation() {
        this.query.Handle(new SearchBooksByTitleCaseInsensitiveQuery
        {
            Title = "Dependency Injection in .NET"
        });
    }
}

这到底给我带来了什么？

  • 通过定义IQueryHandler<TQuery, TResult>查询，我们定义了系统中非常常见的模式（查询）的一般抽象。
  • IQueryHandler<TQuery, TResult>定义单个成员并遵守 ISP 。
  • IQueryHandler<TQuery, TResult>实现实现单个查询并遵守 SRP。
  • 该IQuery<TResult>接口允许我们对查询及其结果进行编译时支持。消费者不能错误地依赖返回类型不正确的处理程序，因为它不会编译。
  • 通用IQueryHandler<TQuery, TResult>抽象允许我们将各种横切关注点应用于查询处理程序，而无需更改任何实现。

特别是最后一点很重要。诸如验证、授权、日志记录、审计跟踪、监控和缓存等横切关注点都可以使用装饰器非常容易地实现，而无需更改处理程序实现和消费者。看看这个：

public class ValidationQueryHandlerDecorator<TQuery, TResult>
    : IQueryHandler<TQuery, TResult>
    where TQuery : IQuery<TResult>
{
    private readonly IServiceProvider provider;
    private readonly IQueryHandler<TQuery, TResult> decorated;

    public ValidationQueryHandlerDecorator(
        Container container,
        IQueryHandler<TQuery, TResult> decorated)
    {
        this.provider = container;
        this.decorated = decorated;
    }

    public TResult Handle(TQuery query)
    {
        var validationContext =
            new ValidationContext(query, this.provider, null);

        Validator.ValidateObject(query, validationContext);

        return this.decorated.Handle(query);
    }
}

这是一个装饰器，可以在运行时围绕所有命令处理程序实现进行包装，从而增加对其进行验证的能力。

有关更多背景信息，请查看这篇文章：同时...在我的架构的查询方面。

于 2013-03-14T13:25:07.647 回答
1

你的界面太具体了imo。有一个谓词

interface ISearcher {
  bool IsAMatch(Book book);
}

并从中获得您的搜索者。另外，不要将搜索功能放入您的收藏中 - 收藏用于存储和迭代。也许我只是描述了访问者模式。

于 2013-03-14T12:40:34.077 回答