假设我想出于不同的原因抽象集合上的操作:
现在为了简单起见,让我们了解一下
的集合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保持密切联系吗?
答案 0 :(得分:8)
是的,我可以用另一种方法定义一个IBookCollection 接口并在所有消费者中使用它,但这样做不会违反 ISP?
您不会违反ISP,但您的图书集将开始承担太多责任,您将违反单一责任原则。
让我担心的另一件事是ITitleSearcher
接口的多个实现。我不确定这里是否存在违反某些设计原则的情况,但您的设计中似乎存在一些您应该注意的模糊性。此外,对于每个搜索操作,您都在创建一个新的抽象。您已经拥有ITitleSearcher
,ISubTitleSearcher
和ISoldDateRangeSearcher
,并且可能会增加数十个。我认为你在这里缺少的是对系统中查询的一般抽象。所以你可以做到这一点:
定义查询参数的抽象:
public interface IQuery<TResult> { }
这是一个没有成员的接口,只有一个通用类型TResult
。 TResult
描述了该查询的返回类型。例如,您可以按如下方式定义查询:
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);
}
并从中获取您的搜索者。 另外,不要将搜索功能放入集合中 - 集合用于存储和迭代。也许我刚才描述了访客模式。