我目前有一个基于ASP.NET网站上文章Validating with a service layer的服务层。
根据this的答案,这是一种糟糕的方法,因为服务逻辑与违反单一责任原则的验证逻辑混合在一起。
我非常喜欢提供的替代方案,但在重新分解我的代码时遇到了一个我无法解决的问题。
考虑以下服务接口:
interface IPurchaseOrderService
{
void CreatePurchaseOrder(string partNumber, string supplierName);
}
基于链接的答案具有以下具体实现:
public class PurchaseOrderService : IPurchaseOrderService
{
public void CreatePurchaseOrder(string partNumber, string supplierName)
{
var po = new PurchaseOrder
{
Part = PartsRepository.FirstOrDefault(p => p.Number == partNumber),
Supplier = SupplierRepository.FirstOrDefault(p => p.Name == supplierName),
// Other properties omitted for brevity...
};
validationProvider.Validate(po);
purchaseOrderRepository.Add(po);
unitOfWork.Savechanges();
}
}
传递给验证程序的PurchaseOrder
对象还需要另外两个实体Part
和Supplier
(我们假设此示例中PO只有一个部分)。 / p>
如果用户提供的详细信息与数据库中需要验证程序抛出异常的实体不对应,则Part
和Supplier
对象都可以为null。
我遇到的问题是,在此阶段验证器丢失了上下文信息(部件号和供应商名称),因此无法向用户报告准确的错误。我能提供的最好的错误是“采购订单必须有一个相关的部分”,这对用户没有意义,因为他们确实提供了一个部件号(它只是不存在于数据库)。
使用ASP.NET文章中的服务类,我正在做这样的事情:
public void CreatePurchaseOrder(string partNumber, string supplierName)
{
var part = PartsRepository.FirstOrDefault(p => p.Number == partNumber);
if (part == null)
{
validationDictionary.AddError("",
string.Format("Part number {0} does not exist.", partNumber);
}
var supplier = SupplierRepository.FirstOrDefault(p => p.Name == supplierName);
if (supplier == null)
{
validationDictionary.AddError("",
string.Format("Supplier named {0} does not exist.", supplierName);
}
var po = new PurchaseOrder
{
Part = part,
Supplier = supplier,
};
purchaseOrderRepository.Add(po);
unitOfWork.Savechanges();
}
这允许我向用户提供更好的验证信息,但意味着验证逻辑直接包含在服务类中,违反了单一责任原则(代码也在服务类之间重复)。
有没有办法让两全其美?我是否可以将服务层与验证层分开,同时仍提供相同级别的错误信息?
答案 0 :(得分:52)
简答:
你正在验证错误的东西。
很长的回答:
您正在尝试验证PurchaseOrder
,但这是一个实现细节。相反,您应该验证的是操作本身,在这种情况下是partNumber
和supplierName
参数。
自己验证这两个参数会很尴尬,但这是由你的设计引起的 - 你错过了一个抽象。
长话短说,问题在于你的IPurchaseOrderService
界面。它不应该采用两个字符串参数,而是一个参数(Parameter Object)。我们将此参数对象称为 CreatePurchaseOrder
:
public class CreatePurchaseOrder
{
public string PartNumber;
public string SupplierName;
}
使用更改的IPurchaseOrderService
界面:
interface IPurchaseOrderService
{
void CreatePurchaseOrder(CreatePurchaseOrder command);
}
CreatePurchaseOrder
参数对象包装原始参数。此参数对象是描述创建采购订单的意图的消息。换句话说:这是一个命令。
使用此命令,您可以创建IValidator<CreatePurchaseOrder>
实施,该实施可以执行所有正确的验证,包括检查是否存在正确的零件供应商并报告用户友好的错误消息。
但为什么IPurchaseOrderService
负责验证? 验证是一个贯穿各领域的问题,您应该防止它与业务逻辑混合。相反,你可以为此定义一个装饰器:
public class ValidationPurchaseOrderServiceDecorator : IPurchaseOrderService
{
private readonly IValidator<CreatePurchaseOrder> validator;
private readonly IPurchaseOrderService decoratee;
ValidationPurchaseOrderServiceDecorator(
IValidator<CreatePurchaseOrder> validator,
IPurchaseOrderService decoratee)
{
this.validator = validator;
this.decoratee = decoratee;
}
public void CreatePurchaseOrder(CreatePurchaseOrder command)
{
this.validator.Validate(command);
this.decoratee.CreatePurchaseOrder(command);
}
}
通过这种方式,您可以通过简单地包装真实PurchaseOrderService
:
var service =
new ValidationPurchaseOrderServiceDecorator(
new CreatePurchaseOrderValidator(),
new PurchaseOrderService());
问题当然,使用这种方法是为系统中的每个服务定义这样的装饰器类真的很尴尬。这将导致严格的代码发布。
但问题是由一个缺陷引起的。定义每个特定服务(例如IPurchaseOrderService
)的接口通常是有问题的。您定义了CreatePurchaseOrder
,因此已经有了这样的定义。您现在可以为系统中的所有业务操作定义一个抽象:
public interface ICommandHandler<TCommand>
{
void Handle(TCommand command);
}
通过这种抽象,您现在可以将PurchaseOrderService
重构为以下内容:
public class CreatePurchaseOrderHandler : ICommandHandler<CreatePurchaseOrder>
{
public void Handle(CreatePurchaseOrder command)
{
var po = new PurchaseOrder
{
Part = ...,
Supplier = ...,
};
unitOfWork.Savechanges();
}
}
通过这种设计,您现在可以定义一个通用装饰器来处理系统中每个业务操作的所有验证:
public class ValidationCommandHandlerDecorator<T> : ICommandHandler<T>
{
private readonly IValidator<T> validator;
private readonly ICommandHandler<T> decoratee;
ValidationCommandHandlerDecorator(
IValidator<T> validator, ICommandHandler<T> decoratee)
{
this.validator = validator;
this.decoratee = decoratee;
}
void Handle(T command)
{
var errors = this.validator.Validate(command).ToArray();
if (errors.Any())
{
throw new ValidationException(errors);
}
this.decoratee.Handle(command);
}
}
注意这个装饰器与先前定义的ValidationPurchaseOrderServiceDecorator
几乎相同,但现在作为泛型类。这个装饰器可以包裹在你的新服务类中:
var service =
new ValidationCommandHandlerDecorator<PurchaseOrderCommand>(
new CreatePurchaseOrderValidator(),
new CreatePurchaseOrderHandler());
但是由于这个装饰器是通用的,你可以将它包装在系统中的每个命令处理程序中。哇!干嘛怎么样?
这种设计也使得以后添加横切关注点非常容易。例如,您的服务目前似乎负责在工作单元上调用SaveChanges
。这也可以被认为是一个跨领域的问题,并且可以很容易地提取给装饰者。这样,您的服务类变得更加简单,只需要更少的代码进行测试。
CreatePurchaseOrder
验证程序可能如下所示:
public sealed class CreatePurchaseOrderValidator : IValidator<CreatePurchaseOrder>
{
private readonly IRepository<Part> partsRepository;
private readonly IRepository<Supplier> supplierRepository;
public CreatePurchaseOrderValidator(
IRepository<Part> partsRepository,
IRepository<Supplier> supplierRepository)
{
this.partsRepository = partsRepository;
this.supplierRepository = supplierRepository;
}
protected override IEnumerable<ValidationResult> Validate(
CreatePurchaseOrder command)
{
var part = this.partsRepository.GetByNumber(command.PartNumber);
if (part == null)
{
yield return new ValidationResult("Part Number",
$"Part number {command.PartNumber} does not exist.");
}
var supplier = this.supplierRepository.GetByName(command.SupplierName);
if (supplier == null)
{
yield return new ValidationResult("Supplier Name",
$"Supplier named {command.SupplierName} does not exist.");
}
}
}
你的命令处理程序如下:
public class CreatePurchaseOrderHandler : ICommandHandler<CreatePurchaseOrder>
{
private readonly IUnitOfWork uow;
public CreatePurchaseOrderHandler(IUnitOfWork uow)
{
this.uow = uow;
}
public void Handle(CreatePurchaseOrder command)
{
var order = new PurchaseOrder
{
Part = this.uow.Parts.Get(p => p.Number == partNumber),
Supplier = this.uow.Suppliers.Get(p => p.Name == supplierName),
// Other properties omitted for brevity...
};
this.uow.PurchaseOrders.Add(order);
}
}
请注意,命令消息将成为域的一部分。用例和命令之间存在一对一的映射,而不是验证实体,这些实体将是一个实现细节。这些命令成为合同并将得到验证。
请注意,如果您的命令包含尽可能多的ID,它可能会让您的生活更轻松。因此,您的系统可以从定义命令中受益,如下所示:
public class CreatePurchaseOrder
{
public int PartId;
public int SupplierId;
}
执行此操作时,您无需检查给定名称的部件是否存在。表示层(或外部系统)为您传递了一个ID,因此您不必再验证该部分的存在。当没有该ID的部分时,命令处理程序当然应该失败,但在这种情况下,存在编程错误或并发冲突。在任何一种情况下,都不需要将富有表现力的用户友好验证错误传达回客户端。
然而,这确实将获得正确ID的问题转移到表示层。在表示层中,用户必须从列表中选择一个部分,以便我们获取该部分的ID。但我仍然体验到这一点,使系统更容易和可扩展。
它还解决了您所指的文章的评论部分中陈述的大多数问题,例如:
如果您想了解有关此类设计的更多信息,请务必查看this article。