ASP.NET Core中TryValidateModel对合法模型始终返回false的问题

1. 最核心的触发原因：ModelState的残留错误 #

ASP.NET Core在完成控制器参数的模型绑定流程后，会自动执行一轮模型验证。这个阶段的执行时机远早于你在方法体里给post.Author赋值的逻辑：

• 你用[Bind("Id, Title, Content")]明确限制了模型绑定只读取表单里的Id、Title、Content三个字段，其余所有属性在绑定阶段都是默认值状态
• 此时post.Author为null，加上你给Author加了[Required]特性，这一轮自动验证会把"Author字段不能为空"的错误记录写入ModelState字典
• 而TryValidateModel方法本身不会清空ModelState中已经存在的错误，它的作用只是对传入的对象重新执行校验规则，把新发现的错误追加到ModelState里。你后续给Author、UpvotedBy手动赋值的操作，完全不会清除之前已经写好的错误记录，所以哪怕你现在对象的所有属性值都合法，ModelState里一直存着之前自动验证产生的错误，TryValidateModel自然永远返回false。这也是为什么你在调试器里看到post.Author非空、所有属性值都合法，但校验始终不通过的原因——ModelState里的错误记录是在赋值前产生的，不会因为你后面给属性赋值就自动消失。

2. 额外的规则冗余问题 #

你在Entry基类里给所有EF Core导航属性加的[Required]特性完全没必要：

• 那些ICollection<T>类型的集合属性，本来就在构造函数里初始化了非空实例，属于代码层面的内存初始化，根本不需要做"是否为null"的校验
• 这些属性本来就不属于前端表单的提交字段，额外加的校验规则只会在自动绑定阶段产生无意义的错误条目，除了增加排查难度没有任何作用

• 在完成所有对模型属性的手动修改、准备调用TryValidateModel之前，先清空自动绑定阶段产生的残留错误：

ModelState.Clear();

• 移除导航属性上不必要的[Required]特性：
  把Entry类中给Replies、SavedBy、HiddenBy、UpvotedBy、DownvotedBy这些集合属性加的[Required]全部删掉。这类EF Core导航属性不需要参与表单校验，构造函数的初始化已经保证了实例非空。
• 调整后的控制器核心逻辑参考：

[Authorize]
[HttpPost]
public async Task<IActionResult> Create([Bind("Id, Title, Content")] Post post)
{
    var user = await userManager.GetUserAsync(User);
    post.Author = user;
    post.UpvotedBy.Add(user);

    if (post.Title != null && post.Content != null)
    {
        post.Title = post.Title.Trim();
        post.Content = post.Content.Trim();
    }

    // 清空自动绑定阶段的残留错误
    ModelState.Clear();

    if (TryValidateModel(post))
    {
        context.Add(post);
        await context.SaveChangesAsync();
        return RedirectToAction("Details", new { id = post.Id });
    }
    return View(post);
}

直接在数据库实体类上加验证特性很容易引入这类问题。更合理的做法是单独定义和表单字段对应的视图模型（ViewModel），把需要用户输入的Title、Content字段抽出来加验证规则，手动把用户提交的值映射到实体对象上，再给实体的导航属性赋值，从根源上避免数据库结构属性和前端校验逻辑的耦合。

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