c# - CommandHandler 装饰器依赖

Question

我有一个问题，我希望我的处理程序使用从处理程序生成的数据:

1. UpdateUserProfileImageCommandHandlerAuthorizeDecorator
2. UpdateUserProfileImageCommandHandlerUploadDecorator
3. UpdateUserProfileImageCommandHandler

我的问题是架构和性能。

UpdateUserCommandHandlerAuthorizeDecorator 调用存储库 (entityframework) 以授权用户。我还有其他类似的装饰器应该使用和修改实体并将其发送到链中。

UpdateUserCommandHandler 应该只是将用户保存到数据库中。我目前必须进行另一个存储库调用并更新实体，而我本可以处理来自前一个装饰器的实体。

我的问题是该命令只接受用户 ID 和一些要更新的属性。在我从 Authorize 装饰器获得用户实体的情况下，我如何仍然在链上处理该实体？可以将 User 属性添加到命令并处理它吗？

代码:

public class UpdateUserProfileImageCommand : Command
{
    public UpdateUserProfileImageCommand(Guid id, Stream image)
    {
        this.Id = id;
        this.Image = image;
    }

    public Stream Image { get; set; }

    public Uri ImageUri { get; set; }
}

public class UpdateUserProfileImageCommandHandlerAuthorizeDecorator : ICommandHandler<UpdateUserProfileImageCommand>
{
    public void Handle(UpdateUserProfileImageCommand command)
    {
         // I would like to use this entity in `UpdateUserProfileImageCommandHandlerUploadDecorator`
         var user = userRespository.Find(u => u.UserId == command.Id);

         if(userCanModify(user, currentPrincipal))
         {
             decoratedHandler(command);
         }

    }
}

public class UpdateUserProfileImageCommandHandlerUploadDecorator : ICommandHandler<UpdateUserProfileImageCommand>
{
    public void Handle(UpdateUserProfileImageCommand command)
    {
         // Instead of asking for this from the repository again, I'd like to reuse the entity from the previous decorator
         var user = userRespository.Find(u => u.UserId == command.Id);

         fileService.DeleteFile(user.ProfileImageUri);

         var command.ImageUri = fileService.Upload(generatedUri, command.Image);

         decoratedHandler(command);       

    }
}

public class UpdateUserProfileImageCommandHandler : ICommandHandler<UpdateUserProfileImageCommand>
{
    public void Handle(UpdateUserProfileImageCommand command)
    {
         // Again I'm asking for the user...
         var user = userRespository.Find(u => u.UserId == command.Id);

         user.ProfileImageUri = command.ImageUri;     

         // I actually have this in a PostCommit Decorator.
         unitOfWork.Save();
    }
}

Answer 1

您不应该仅仅为了性能而尝试传递任何额外的数据。此外，使用装饰器，您无法更改契约(Contract)。相反，您应该允许缓存该用户实体，这通常应该是存储库实现的责任。使用 Entity Framework ，这实际上相当简单。您可以调用DbSet.Find(id)而EF会先在缓存中查找实体。这可以防止不必要的数据库往返。我一直这样做。

所以你唯一要做的就是添加一个Find(key)或 GetById映射到 EF 的存储库的方法 Find(key)方法，你就完成了。

此外，我同意皮特的观点。装饰器应该主要用于横切关注点。有时在装饰器中添加其他东西可能没问题，但您似乎将核心业务逻辑拆分到处理程序及其装饰器上。将文件写入磁盘是核心逻辑。您可能会担心遵守单一职责，但在我看来，您将单一职责拆分到多个类中。这并不意味着您的命令处理程序应该很大。正如 Pete 所说，您可能希望将其提取到服务并将该服务注入(inject)处理程序。

验证授权是一个横切关注点，因此在装饰器中使用它似乎没问题，但您当前的实现存在一些问题。首先，这样做会导致你有很多非泛型的装饰器，这会导致大量的维护工作。此外，如果用户未经授权，您会默默地跳过执行，这通常不是您想要的。

与其默默跳过，不如考虑抛出异常，防止用户在正常情况下无法调用该功能。这意味着如果抛出异常，则说明您的代码中存在错误，或者用户正在入侵您的系统。默默地跳过而不抛出异常会使查找错误变得更加困难。

您可能要考虑的另一件事是尝试将此授权逻辑实现为通用装饰器。例如有一个通用授权装饰器或验证装饰器。这可能并不总是可行，但您可以使用属性标记命令。例如，在我目前正在处理的系统中，我们将命令标记为:

[PermittedRole(Role.LabManagement)]

我们有一个 AuthorizationVerifierCommandHandlerDecorator<TCommand>检查正在执行的命令的属性并验证当前用户是否被允许执行该命令。

更新

这是我认为您的 UpdateUserProfileImageCommandHandler 的示例可能看起来像:

public class UpdateUserProfileImageCommandHandler 
    : ICommandHandler<UpdateUserProfileImageCommand>
{
    private readonly IFileService fileService;

    public UpdateUserProfileImageCommandHandler(IFileService fileService)
    {
        this.fileService = fileService;
    }

    public void Handle(UpdateUserProfileImageCommand command)
    {
         var user = userRespository.GetById(command.Id);

         this.fileService.DeleteFile(user.ProfileImageUri);

         command.ImageUri = this.fileService.Upload(generatedUri, command.Image);

         user.ProfileImageUri = command.ImageUri;     
    }
}