ASP.NET MVC创建产品时ModelState.IsValid始终为false的问题求助

从你的调试日志可以明确：错误出在Category字段而非CategoryID——虽然表单正确提交了选中的CategoryID，但EF Core会因为Product类中的Category导航属性默认被标记为必填（因为CategoryID是非可空int类型），而表单并未提交Category对象的任何数据，导致ModelState验证失败。

以下是三种可行的修复方案：

方案1：将导航属性设为可空（推荐） #

修改Product类中的Category导航属性为可空类型，EF将不再强制要求该属性必须有值，外键CategoryID已经能保证关联关系的有效性：

public class Product
{
    [Key]
    public int ProductID { get; set; }
    
    [Display(Name = "Product Name")]
    public string ProductName { get; set; }

    [Display(Name = "Category")]
    [ForeignKey("Category")]
    public int CategoryID { get; set; }

    [Display(Name = "Unit of Measurement")]
    public string Unit { get; set; }

    public decimal Price { get; set; }

    // 改为可空类型
    public Category? Category { get; set; }
}

方案2：在POST方法中移除Category的ModelState验证 #

如果不想修改实体模型，可以在POST的Create方法中，手动移除ModelState中关于Category的验证条目：

[HttpPost]
[ValidateAntiForgeryToken]
public async Task<IActionResult> Create([Bind("ProductID,ProductName,CategoryID,Unit,Price")] Product product)
{
    // 移除Category导航属性的验证错误
    ModelState.Remove("Category");

    if (!ModelState.IsValid)
    {
        var errorList = ModelState
                             .Where(x => x.Value.Errors.Count > 0)
                             .ToDictionary(kvp => kvp.Key,
                                           kvp => kvp.Value.Errors.Select(e => e.ErrorMessage).ToArray());

        foreach (var error in errorList)
        {
            Debug.WriteLine($"Key: {error.Key}, Errors: {string.Join(", ", error.Value)}");
        }

        ViewBag.CategoryID = new SelectList(await _context.Category.ToListAsync(), "CategoryID", "CategoryName", product.CategoryID);

        return View(product);
    }

    _context.Add(product);
    await _context.SaveChangesAsync();

    return RedirectToAction(nameof(Index));
}

方案3：使用视图模型（最佳实践） #

对于复杂表单提交，推荐使用视图模型（ViewModel），完全分离表单需求与数据实体，避免不必要的耦合：

1. 创建视图模型：

public class ProductCreateViewModel
{
    [Display(Name = "Product Name")]
    [Required]
    public string ProductName { get; set; }

    [Display(Name = "Category")]
    [Required]
    public int CategoryID { get; set; }

    [Display(Name = "Unit of Measurement")]
    public string Unit { get; set; }

    [Required]
    public decimal Price { get; set; }
}

2. 修改控制器方法：

public IActionResult Create()
{
    ViewBag.CategoryID = new SelectList(_context.Category, "CategoryID", "CategoryName");
    return View();
}

[HttpPost]
[ValidateAntiForgeryToken]
public async Task<IActionResult> Create(ProductCreateViewModel vm)
{
    if (!ModelState.IsValid)
    {
        ViewBag.CategoryID = new SelectList(await _context.Category.ToListAsync(), "CategoryID", "CategoryName", vm.CategoryID);
        return View(vm);
    }

    // 将视图模型映射为实体模型
    var product = new Product
    {
        ProductName = vm.ProductName,
        CategoryID = vm.CategoryID,
        Unit = vm.Unit,
        Price = vm.Price
    };

    _context.Add(product);
    await _context.SaveChangesAsync();

    return RedirectToAction(nameof(Index));
}

3. 更新视图的模型引用：

@model YourNamespace.ProductCreateViewModel

<div class="form-group">
    <label asp-for="CategoryID" class="control-label"></label>
    <select asp-for="CategoryID" asp-items="ViewBag.CategoryID" class="form-control"></select>
</div>

以上方案均可解决问题，其中视图模型方案是长期维护的最优选择，能有效避免实体模型被不必要的验证规则绑定。

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