定义正确的构造函数注入和ISP(SOLID)的抽象

时间:2013-03-14 12:26:49

标签: c# design-patterns dependency-injection abstraction

假设我想出于不同的原因抽象集合上的操作:

现在为了简单起见,让我们了解一下

的集合
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
  }
}

在此之前我已经清楚了解所有内容,并且我理解的是Interface Segregation Principle也得到了遵守。

继续开发以满足其他需求,因此我定义并实现其他接口,例如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个实例注入TitleAndSoldSearcherConsumer ctor;我已经通过了每个接口的实现。

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

我可以同时与ISP / SOLID和DRY保持密切联系吗?

2 个答案:

答案 0 :(得分:8)

  

是的,我可以用另一种方法定义一个IBookCollection   接口并在所有消费者中使用它,但这样做不会违反   ISP?

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

让我担心的另一件事是ITitleSearcher接口的多个实现。我不确定这里是否存在违反某些设计原则的情况,但您的设计中似乎存在一些您应该注意的模糊性。此外,对于每个搜索操作,您都在创建一个新的抽象。您已经拥有ITitleSearcherISubTitleSearcherISoldDateRangeSearcher,并且可能会增加数十个。我认为你在这里缺少的是对系统中查询的一般抽象。所以你可以做到这一点:

定义查询参数的抽象:

public interface IQuery<TResult> { }

这是一个没有成员的接口,只有一个通用类型TResultTResult描述了该查询的返回类型。例如,您可以按如下方式定义查询:

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

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

您还需要的是对知道如何处理特定查询的类的抽象:

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

了解该方法在TQuery中的用法并返回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);
    }
}

这是一个装饰器,可以在运行时包装所有命令处理程序实现,增加了验证它的能力。

有关更多背景信息,请查看以下文章:Meanwhile... on the query side of my architecture

答案 1 :(得分:1)

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

interface ISearcher {
  bool IsAMatch(Book book);
}

并从中获取您的搜索者。 另外,不要将搜索功能放入集合中 - 集合用于存储和迭代。也许我刚才描述了访客模式。