c# - 如何正确地将模型验证从 Controller 分离到 Service 中？

Question

我正在重构我正在进行的项目。在我现有的 Controller 中，我确实使用了存储库模式，但我仍然执行了太多的脚手架，这让我感到不舒服。那和我的一些 Controller 可能有 10 多个存储库(通过 Ninject)传入。因此，我决定引入一个服务层，我的目的是为每个 Controller 提供一个服务，而每个服务将注入(inject)多个存储库并完成我需要的工作。到目前为止效果很好，但我遇到了一些困惑:如何将模型验证从 Controller 移到服务层？

例如，看看我的 OfficesController 上的这个 Edit 方法:

[HttpPost]
public async Task<RedirectToRouteResult> Edit(
    short id,
    FormCollection form,
    [Bind(Prefix = "Office.Coordinates", Include = "Latitude,Longitude")] Coordinate[] coordinates) {
    if (id > 0) {
        Office office = await this.OfficesService.GetOfficeAsync(id);

        if ((office != null)
            && base.TryUpdateModel(office, "Office", new string[2] {
                "Name",
                "RegionId"
            }, form)
            && base.ModelState.IsValid) {
            this.OfficesService.UpdateOfficeAsync(office, coordinates);
        }

        return base.RedirectToAction("Edit", new {
            id = id
        });
    }

    return base.RedirectToAction("Default");
}

与 Controller 的方法相比，它的问题在于我仍然从数据库中获取一个 Office 对象，进行更新、验证，然后再次保存。在这种情况下，复杂性增加而不是减少。之前，我在方法中调用存储库，现在我调用调用存储库的服务来执行相同的功能。到目前为止，这种复杂性的增加只在我的 Edit 方法中表现出来，其他地方的复杂性都大大降低了，这正是我想要的。

那么，什么是将验证和模型更新逻辑从 Controller 转移到服务中的正确方法？欢迎推荐!

作为引用，我的项目结构如下:

• 数据:包含我所有的模型类
• Data.Google.Maps:包含反序列化特定 Kml 所需的所有类
• Data.Models:包含我的 DbContext、配置、 View 模型和部分 View 模型
• Data.Repositories:包含与 DbContext 对话的所有存储库。由于 EF 本身就是一个伪存储库，我正在利用我的“存储库”作为一种更具体的数据查询方式。例如:FindTechnicians() 或 FindActive() 等
• Data.Services:包含我将使用的所有服务。这些服务将有一个或多个存储库注入(inject)其中，并在我将完整的 View 模型传回 Controller 之前执行我需要完成的所有逻辑。
• Identity:包含我对 ASP.NET Identity 的实现。
• Web.Private:包含实际的 MVC 项目。

Answer 1

如果您还没有阅读，请阅读以下 2 篇文章:

• https://cuttingedge.it/blogs/steven/pivot/entry.php?id=91
• https://cuttingedge.it/blogs/steven/pivot/entry.php?id=92

您的问题的答案是 FluentValidation.NET和依赖装饰。

有了它，你可以做这样的事情:

private readonly IExecuteCommands _commands;

[HttpPost]
public async Task<RedirectToRouteResult> Edit(short id, UpdateOffice command) {

    // with FV.NET plugged in, if your command validator fails,
    // ModelState will already be invalid
    if (!ModelState.IsValid) return View(command);

    await _commands.Execute(command);
    return RedirectToAction(orWhateverYouDoAfterSuccess);
}

该命令只是一个普通的 DTO，就像一个 View 模型。可能看起来像这样:

public class UpdateOffice
{
    public int OfficeId { get; set; }
    public int RegionId { get; set; }
    public string Name { get; set; }
}

...和魔法验证器:

public class ValidateUpdateOfficeCommand : AbstractValidator<UpdateOffice>
{
    public ValidateUpdateOfficeCommand(DbContext dbContext)
    {
        RuleFor(x => x.OfficeId)
            .MustFindOfficeById(dbContext);

        RuleFor(x => x.RegionId)
            .MustFindRegionById(dbContext);

        RuleFor(x => x.Name)
            .NotEmpty()
            .Length(1, 200)
            .MustBeUniqueOfficeName(dbContext, x => x.OfficeId);
    }
}

这些验证规则中的每一个都将在您的操作方法执行之前运行，前提是您为依赖注入(inject)设置了验证器并且您正在使用 FV MVC 验证提供程序。如果存在验证错误，ModelState.IsValid 将为 false。

您还刚刚解决了 Controller 和(可能)服务层中的过度注入(inject)问题。您可以运行任何查询、执行任何命令或验证仅具有 3 个接口(interface)依赖项的任何对象。