ModelState.IsValid与DbUpdateConcurrencyException应放Controller还是Repository？

作为刚接触MVC和Repository模式的新手，你问的这个问题正好戳中了关注点分离这个核心设计原则，咱们一步步拆解：

1. ModelState.IsValid：必须留在控制器里 #

ModelState.IsValid是ASP.NET Core MVC控制器的专属属性，它负责验证用户通过视图/API提交的模型数据是否符合校验规则（比如必填项、格式验证等）。这属于**表现层（Presentation Layer）**的职责——控制器要先确保输入合法，再把干净的数据交给Repository处理。

你尝试把它放到Repository里编译报错，本质原因是Repository属于数据访问层（Data Access Layer），它不应该知道ModelState这种和Web请求、视图绑定相关的对象。Repository的职责只应该是和数据库交互，处理数据的增删改查，完全不需要关心用户输入是否合法。

2. DbUpdateConcurrencyException：异常捕获留在控制器，数据操作封装在Repository #

DbUpdateConcurrencyException是EF Core在处理并发冲突时抛出的异常（比如两个用户同时修改同一条数据）。这里的处理要拆分：

• Repository负责执行数据更新和保存操作：你可以把_context.Update(ProductTransaction)和await _context.SaveChangesAsync()的逻辑封装到Repository的方法里，但不要在Repository里处理异常。
• 控制器负责捕获异常并处理HTTP响应：因为异常处理后的逻辑（比如返回NotFound或者重新抛出异常）属于Web层的职责——控制器需要根据异常情况返回对应的HTTP状态码或视图，而Repository不应该知道HTTP响应的概念。

给你的代码修改建议 #

调整后的Repository代码

// 封装实体映射逻辑
public ProductTransaction MapViewModelToEntity(ProductTransactionViewModel viewModel)
{
    return new ProductTransaction
    {
        Id = viewModel.Id,
        Date = viewModel.Date,
        CustomerName = viewModel.CustomerName
    };
}

// 封装更新与保存逻辑
public async Task UpdateProductTransactionAsync(ProductTransaction productTransaction)
{
    _context.Update(productTransaction);
    await _context.SaveChangesAsync(); // 不捕获异常，直接抛给上层控制器处理
}

// 封装存在性检查
public bool ProductTransactionExists(int id)
{
    return _context.ProductTransactions.Any(e => e.Id == id);
}

调整后的Controller代码

if (ModelState.IsValid)
{
    try
    {
        var productTransaction = _productTransactionRepo.MapViewModelToEntity(ProductTransactionViewModel);
        await _productTransactionRepo.UpdateProductTransactionAsync(productTransaction);
        return RedirectToAction(nameof(Index)); // 操作成功后跳转或返回响应
    }
    catch (DbUpdateConcurrencyException)
    {
        if (!_productTransactionRepo.ProductTransactionExists(ProductTransactionViewModel.Id))
        {
            return NotFound();
        }
        else
        {
            throw; // 可选择重新抛异常给全局处理器，或返回自定义错误视图
        }
    }
}
// ModelState无效时，返回原视图让用户修正输入
return View(ProductTransactionViewModel);

核心原则总结 #

• 表现层（控制器）：处理请求接收、输入验证、HTTP响应、异常的Web相关处理。
• 数据访问层（Repository）：专注于数据持久化逻辑，不依赖任何Web层的对象或概念。

这样拆分后，你的代码会更符合规范，也更容易维护和测试~

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