ServiceController与RequestValidator中ModelState引用不一致问题排查求助

这个问题我之前也碰到过，核心原因是ASP.NET Core框架在请求处理流程中会替换Controller的ModelState实例——你在构造函数里绑定的是初始的空ModelState，后续框架完成模型绑定后更新了Controller的ModelState，但Validator里引用的还是旧的那个，所以才会出现两者状态和哈希码都不一致的情况。

具体原因拆解 #

• 构造函数执行时机过早：Controller的构造函数在请求开始前就被调用了，此时this.ModelState是一个全新的、空的ModelStateDictionary实例。你把这个实例赋值给validator.ModelState，让它指向这个初始对象。
• 模型绑定会替换ModelState：当请求进入ProvisionUser方法时，ASP.NET Core会完成UserProvisioningRequest的模型绑定和验证，这个过程中框架会重新创建一个包含验证错误的ModelStateDictionary实例，并替换掉Controller原本的ModelState属性。这时候：
  • this.ModelState指向的是新的、带有错误的实例（所以IsValid是false）
  • validator.ModelState还是指向构造函数里的旧实例（空的，所以IsValid是true）
• 哈希码不同也印证了这一点：两个变量指向的是完全不同的对象实例，哈希码自然不一样。

可行的解决方案 #

你有几种不同的修复方式，可以根据自己的场景选择：

1. 每次请求时重新赋值ModelState（最简单直接） #

在请求方法的开头，重新把当前最新的ModelState赋值给Validator，确保它引用的是经过模型绑定后的实例：

[HttpPost]
[Route("v0.1/users")]
public async Task<ActionResult<User>> ProvisionUser(
    [FromBody] UserProvisioningRequest userRequest)
{
    // 每次请求时更新Validator的ModelState引用
    this.validator.ModelState = this.ModelState;

    bool thisModelStateValidation = this.ModelState.IsValid;
    bool validatorModelStateValidation = this.validator.ModelState.IsValid;
    // 现在两者的状态应该完全一致了

    this.validator.ValidateRequest(userRequest);
    // ... 后续业务逻辑
}

2. 让Validate方法接收ModelState参数（更健壮，避免引用持有问题） #

修改RequestValidator，去掉它的ModelState属性，改为在调用Validate方法时传入当前的ModelState。这样就彻底避免了引用过期的问题：

public class RequestValidator
{
    // 移除ModelState属性
    public void ValidateRequest(UserProvisioningRequest userRequest, ModelStateDictionary modelState)
    {
        if (!modelState.IsValid)
        {
            // 这里就能拿到正确的验证状态了
        }
        // 自定义验证逻辑也直接操作传入的modelState
    }
}

然后在Controller里调用时传入当前的ModelState：

this.validator.ValidateRequest(userRequest, this.ModelState);

3. 使用ActionFilter注入ModelState（适合全局复用场景） #

如果你的RequestValidator需要在多个Controller里复用，可以创建一个ActionFilter，在模型绑定完成后自动把最新的ModelState赋值给Validator：

public class InjectModelStateFilter : IActionFilter
{
    private readonly RequestValidator _validator;

    public InjectModelStateFilter(RequestValidator validator)
    {
        _validator = validator;
    }

    public void OnActionExecuting(ActionExecutingContext context)
    {
        // 模型绑定完成后，获取Controller的ModelState并赋值给Validator
        if (context.Controller is ControllerBase controller)
        {
            _validator.ModelState = controller.ModelState;
        }
    }

    public void OnActionExecuted(ActionExecutedContext context) { }
}

然后在Startup/Program.cs里注册这个Filter，或者直接在Controller上标记使用：

[TypeFilter(typeof(InjectModelStateFilter))]
[Authorize]
[ApiController]
public class ServiceController : ControllerBase
{
    private readonly RequestValidator validator;

    public ServiceController(RequestValidator validator)
    {
        this.validator = validator;
        // 无需再在这里赋值ModelState
    }

    // ... 原有请求方法逻辑
}

这样就能保证Validator始终拿到的是经过模型绑定后的最新ModelState实例，不会再出现状态不一致的问题。

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