ASP.NET MVC表单提交后复杂对象属性始终为空的问题解决咨询

这个问题的核心是视图中表单字段的命名不符合ASP.NET MVC模型绑定的要求，导致集合属性Items和Name无法正确绑定到后端的ToDoList对象。

为什么会出现这个问题？ #

你在视图中使用foreach循环遍历Model.Items，生成的表单字段名称会是类似toDoItem.Task、toDoItem.IsDone这样的格式，但ASP.NET MVC的模型绑定需要集合类属性的字段名称遵循Items[0].Task、Items[1].IsDone这种带索引的命名规则，才能识别并组装成ICollection<ToDoItem>对象。

另外，你还遗漏了ToDoItem.Id的隐藏字段——即使绑定成功，EF也无法识别要更新的具体条目；同时视图里没有保留Name的输入字段，导致这个属性也无法被绑定。

解决方案1：改用for循环（适合IList类型集合） #

首先把ToDoList中的Items属性改为IList<ToDoItem>（因为ICollection没有索引器，for循环需要索引访问）：

public class ToDoList { 
    public Guid Id {get;set;} 
    public string Name { get; set; } 
    public virtual IList<ToDoItem> Items {get;set;} // 改成IList
}

然后修改视图中的循环部分：

@model DataLayer.TomTest.Entities.ToDoList 
<h2>@Model.Name</h2> 
@using (@Html.BeginForm()) { 
    @Html.AntiForgeryToken() 
    <!-- 保留Name的隐藏字段，确保该属性能被绑定 -->
    @Html.HiddenFor(model => model.Name)
    <table> 
        <tr> 
            <th> @Html.DisplayNameFor(model => model.Items.First().Id) </th> 
            <th> @Html.DisplayNameFor(model => model.Items.First().Task) </th> 
            <th> @Html.DisplayNameFor(model => model.Items.First().IsDone) </th> 
        </tr> 
        @for (int i = 0; i < Model.Items.Count; i++) { 
            <tr> 
                <td> @Model.Items[i].Id </td> 
                <td> @Html.EditorFor(model => model.Items[i].Task) </td> 
                <td> @Html.EditorFor(model => model.Items[i].IsDone, new {htmlAttributes = new {@Style = "margin-left: 10px;"}}) </td> 
                <!-- 必须添加Hidden字段保存Id，否则EF无法定位要更新的条目 -->
                @Html.HiddenFor(model => model.Items[i].Id)
            </tr> 
        } 
    </table> 
    <input type="submit" value="Save" class="btn btn-default"/> 
}

解决方案2：使用EditorTemplate（更优雅，支持ICollection） #

这种方法不需要修改集合类型，更适合ICollection的场景：

1. 在项目中创建Views/Shared/EditorTemplates文件夹（如果不存在），然后新建ToDoItem.cshtml文件，内容如下：

@model DataLayer.TomTest.Entities.ToDoItem
<tr>
    <td>@Model.Id</td>
    <td>@Html.EditorFor(m => m.Task)</td>
    <td>@Html.EditorFor(m => m.IsDone, new { htmlAttributes = new { @Style = "margin-left: 10px;" } })</td>
    @Html.HiddenFor(m => m.Id)
</tr>

2. 修改主视图的循环部分，替换foreach为Html.EditorFor：

@model DataLayer.TomTest.Entities.ToDoList 
<h2>@Model.Name</h2> 
@using (@Html.BeginForm()) { 
    @Html.AntiForgeryToken() 
    @Html.HiddenFor(model => model.Name)
    <table> 
        <tr> 
            <th> @Html.DisplayNameFor(model => model.Items.First().Id) </th> 
            <th> @Html.DisplayNameFor(model => model.Items.First().Task) </th> 
            <th> @Html.DisplayNameFor(model => model.Items.First().IsDone) </th> 
        </tr> 
        <!-- 直接调用EditorFor渲染整个Items集合，自动生成符合要求的字段名称 -->
        @Html.EditorFor(model => model.Items)
    </table> 
    <input type="submit" value="Save" class="btn btn-default"/> 
}

后端代码的优化建议 #

你当前直接设置_context.Entry(todoList).State = EntityState.Modified的方式存在风险，可能会覆盖数据库中未提交的字段。建议改为先从数据库获取原实体，再更新属性：

[HttpPost] 
[ValidateAntiForgeryToken] 
public async Task<ActionResult> Details(ToDoList todoList) { 
    if (ModelState.IsValid) { 
        // 从数据库获取包含Items的原实体
        var existingList = await _context.ToDoLists
            .Include(l => l.Items)
            .FirstOrDefaultAsync(l => l.Id == todoList.Id);
        
        if (existingList != null) {
            // 更新列表名称
            existingList.Name = todoList.Name;
            
            // 更新每个待办项
            foreach (var submittedItem in todoList.Items) {
                var existingItem = existingList.Items.FirstOrDefault(i => i.Id == submittedItem.Id);
                if (existingItem != null) {
                    existingItem.Task = submittedItem.Task;
                    existingItem.IsDone = submittedItem.IsDone;
                }
                // 如果需要处理新增/删除待办项，这里需要额外逻辑
            }
            
            await _context.SaveChangesAsync();
            return RedirectToAction("Details", new { id = todoList.Id });
        }
    } 
    return View(todoList); 
}

额外注意点 #

• 确保ToDoList和ToDoItem的EF关系配置正确（比如通过Fluent API或DataAnnotations），不过你的问题主要出在模型绑定，而非EF配置。
• 如果需要允许用户修改Name，可以把隐藏字段改成EditorFor或TextBoxFor，让用户直接编辑。

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