ASP.NET Core MVC中Update服务方法的Nullable设计及逻辑优化咨询

学习Web API时看到作者将Update服务方法设计为可空返回类型Task<Comment?>，通过判断返回值是否为null返回NotFound，逻辑清晰合理。但在自己的ASP.NET Core MVC项目中结合Fluent Validation和AutoMapper使用该方案时，发现逻辑不通。

现有代码 #

CategoryService代码 #

public async Task<Category?> UpdateCategoryAsync(UpdateCategoryDto updateCategoryDto)
{
    var category = await GetCategoryByIdAsync(updateCategoryDto.Id);

    if (category == null)
        return null;

    mapper.Map(updateCategoryDto, category);

    await unitOfWork.GetRepository<Category>().UpdateAsync(category);
    await unitOfWork.SaveChangesAsync();

    return category;
}

原控制器Update方法 #

[HttpPost]
public async Task<IActionResult> Update(UpdateCategoryDto updateCategoryDto)
{
    var category = mapper.Map<Category>(updateCategoryDto);
    var result = validator.Validate(category);
    var exists = await categoryService.Exists(category);

    if (exists)
        result.Errors.Add(new ValidationFailure("Name", "This category name already exists"));

    if (result.IsValid)
    {
        await categoryService.UpdateCategoryAsync(updateCategoryDto);
        return RedirectToAction("Index", "Category", new { Area = "Admin" });
    }

    result.AddToModelState(ModelState);

    return View(updateCategoryDto);
}

修改后的控制器Update方法 #

public async Task<IActionResult> Update(UpdateCategoryDto updateCategoryDto)
{
    var category = mapper.Map<Category>(updateCategoryDto);
    var result = validator.Validate(category);
    var exists = await categoryService.Exists(category);

    if (exists)
        result.Errors.Add(new ValidationFailure("Name", "This category name already exists"));

    if (result.IsValid)
    {
        var value = await categoryService.UpdateCategoryAsync(updateCategoryDto);

        if (value == null)
            return NotFound();

        return RedirectToAction("Index", "Category", new { Area = "Admin" });
    }

    result.AddToModelState(ModelState);

    return View(updateCategoryDto);
}

核心问题 #

修改后的控制器中，若updateCategoryDto为null，根本无法通过验证，导致服务返回null的判断完全没有实际作用。纠结是将服务方法改为返回Task<Category>，还是调整控制器逻辑，希望得到合理的优化建议。

优化建议 #

1. 将“分类存在性校验”整合到Fluent Validation流程 #

当前的验证只针对Category的字段规则，没有校验updateCategoryDto.Id对应的分类是否存在。应该把这个校验加入Fluent Validation的规则中，让无效请求在进入业务逻辑前就被拦截：

public class UpdateCategoryDtoValidator : AbstractValidator<UpdateCategoryDto>
{
    private readonly ICategoryService _categoryService;

    public UpdateCategoryDtoValidator(ICategoryService categoryService)
    {
        _categoryService = categoryService;

        // 基础字段校验
        RuleFor(x => x.Name).NotEmpty().MaximumLength(50);

        // 校验Id对应的分类是否存在
        RuleFor(x => x.Id)
            .MustAsync(async (id, cancellationToken) => 
                await _categoryService.GetCategoryByIdAsync(id) != null)
            .WithMessage("指定的分类不存在");

        // 校验名称是否重复（排除当前分类）
        RuleFor(x => x.Name)
            .MustAsync(async (dto, name, cancellationToken) => 
                !await _categoryService.Exists(new Category { Id = dto.Id, Name = name }))
            .WithMessage("该分类名称已存在");
    }
}

2. 调整服务层返回值与职责 #

如果控制器已经通过验证确保了updateCategoryDto.Id对应的分类存在，那么服务层的UpdateCategoryAsync就不需要再处理分类不存在的情况，此时可以将返回值改为Task<Category>，避免不必要的null判断：

public async Task<Category> UpdateCategoryAsync(UpdateCategoryDto updateCategoryDto)
{
    // 此时可以确保category不为null，因为控制器已经验证过
    var category = await GetCategoryByIdAsync(updateCategoryDto.Id);

    mapper.Map(updateCategoryDto, category);

    await unitOfWork.GetRepository<Category>().UpdateAsync(category);
    await unitOfWork.SaveChangesAsync();

    return category;
}

3. 简化控制器逻辑流程 #

优化后的控制器可以去掉冗余的null判断，流程更清晰：

[HttpPost]
public async Task<IActionResult> Update(UpdateCategoryDto updateCategoryDto)
{
    // 模型绑定失败时，updateCategoryDto可能为null，直接返回视图
    if (updateCategoryDto == null)
    {
        ModelState.AddModelError("", "请求数据无效");
        return View();
    }

    var result = await validator.ValidateAsync(updateCategoryDto);

    if (!result.IsValid)
    {
        result.AddToModelState(ModelState);
        return View(updateCategoryDto);
    }

    await categoryService.UpdateCategoryAsync(updateCategoryDto);
    return RedirectToAction("Index", "Category", new { Area = "Admin" });
}

补充说明 #

• Web API中用可空返回值+NotFound是合理的，因为API通常需要明确返回资源不存在的状态码；但MVC场景下，更适合通过前端验证+服务器端验证提前拦截无效请求，而不是在业务逻辑后返回NotFound（用户更期望看到带错误提示的表单页面，而非404页面）。
• 保持验证逻辑的集中性，把所有校验规则放到Fluent Validation中，能让控制器代码更简洁，也便于维护。

内容的提问来源于stack exchange，提问作者Fakhriyya