asp.net-mvc-3 – MVC3 – 具有复杂类型列表的Viewmodel

如果以前曾经问过，就道歉有一百万种方式来说明它，所以寻找答案已被证明是困难的。

我有一个具有以下属性的viewmodel：

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>
}