public class AssignSoftwareLicenseviewmodel { public int LicenseId { get; set; } public ICollection<SelectableDeviceviewmodel> Devices { get; set; } }
SelectableDeviceviewmodel的简化版本将是:
public class SelectableDeviceviewmodel { public int DeviceInstanceId { get; set; } public bool IsSelected { get; set; } public string Name { get; set; } }
在我的视图中,我尝试在输入表单中显示“设备”属性的可编辑复选框列表。
目前,我的视图看起来像这样:
@using (Html.BeginForm()) { @Html.HiddenFor(x => Model.LicenseId) <table> <tr> <th>Name</th> <th></th> </tr> @foreach (SelectableDeviceviewmodel device in Model.Devices) { @Html.HiddenFor(x => device.DeviceInstanceId) <tr> <td>@Html.CheckBoxFor(x => device.IsSelected)</td> <td>@device.Name</td> </tr> } </table> <input type="submit" value="Assign" /> }
问题是,当模型被发回控制器时,设备为空。
我的假设是,这是发生的,因为即使我正在编辑它的内容,Devices属性从未被明确地包含在窗体中。我尝试用HiddenFor包含它,但是这只是导致模型有一个空列表,而不是null。
任何想法我在这里做错什么?
解决方法
My assumption is that this is happening because even though I’m
editing its contents,the Devices property is never explicitly
included in the form.
不,你的假设是错误的。这不能正确绑定的原因是因为您的输入字段没有正确的名称。例如,它们被称为name =“IsSelected”,而不是name =“Devices [0] .IsSelected”。看看需要用于绑定到集合的正确的线格式:http://haacked.com/archive/2008/10/23/model-binding-to-a-list.aspx
But why this happens?
这是因为您在视图中使用的foreach循环。你使用x => device.IsSelects作为复选框的lambda表达式,但这并没有考虑到Devices属性(通过查看生成的网页的源代码可以看出)。
So what should I do?
我个人建议您使用编辑器模板,因为它们尊重复杂属性的导航上下文并生成正确的输入名称。所以在你的视图中摆脱整个foreach循环,并用一行代码替换它:
@Html.EditorFor(x => x.Devices)
并且现在定义一个自定义编辑器模板,该模板将由ASP.NET MVC为Devices集合的每个元素自动呈现。警告:此模板的位置和名称非常重要,因为这可以按照惯例工作:〜/ Views / Shared / EditorTemplates / SelectableDeviceviewmodel.cshtml:
@model SelectableDeviceviewmodel @Html.HiddenFor(x => x.DeviceInstanceId) <tr> <td>@Html.CheckBoxFor(x => x.IsSelected)</td> <td>@Html.DisplayFor(x => x.Name)</td> </tr>
另一种方法(我不推荐)将视图模型中的当前ICollection更改为索引集合(如IList T或数组T []):
public class AssignSoftwareLicenseviewmodel { public int LicenseId { get; set; } public IList<SelectableDeviceviewmodel> Devices { get; set; } }
然后代替foreach使用for循环:
@for (var i = 0; i < Model.Devices.Count; i++) { @Html.HiddenFor(x => x.Devices[i].DeviceInstanceId) <tr> <td>@Html.CheckBoxFor(x => x.Devices[i].IsSelected)</td> <td>@Html.DisplayFor(x => x.Devices[i].Name</td> </tr> }