Question

如果之前有人问过，请道歉;有一百万种方式来表达它，因此难以找到答案。

我有一个具有以下属性的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; }
}

在我的视图中，我试图在输入表单中显示Devices属性的可编辑复选框列表。目前，我的View看起来像这样：

@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。

知道我在这里做错了吗？

Answer 1

我的假设是，这种情况正在发生，因为即使我是编辑其内容时，Devices属性永远不会显式包含在表格中。

不，你的假设是错误的。这不正确绑定的原因是因为您的输入字段没有正确的名称。例如，它们被称为name="IsSelected"而不是name="Devices[0].IsSelected"。看一下需要用来绑定到集合的正确的有线格式：http://haacked.com/archive/2008/10/23/model-binding-to-a-list.aspx

但为什么会这样呢？

这是因为您在视图中使用的foreach循环。您使用x => device.IsSelected作为复选框的lambda表达式，但这根本没有考虑Devices属性（通过查看生成的网页源代码可以看到）。

那我该怎么办？

我个人建议您使用编辑器模板，因为它们尊重复杂属性的导航上下文并生成正确的输入名称。因此，在视图中删除整个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; }
}

然后使用for循环代替foreach：

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

Answer 2

EditorFor模板工作并保持代码清洁。您不需要循环，模型会正确回发。

但是，有没有人在复杂的视图模型（嵌套的EditorFor模板）上进行验证有问题？我正在使用Kendo Validator并遇到各种各样的jquery错误。

MVC3 - 包含复杂类型列表的Viewmodel

2 个答案: