ASP.NET MVC 中的奇怪行为：从嵌套结构的列表中删除项目总是删除最后一个项目

Question

场景

我有一个父/子模型（确切地说是一个小调查表和一个或多个联系人）。由于历史原因，所有这些都将在同一个表单上完成，因此用户将拥有一个针对父级和一个子级的表单，并且他们将点击一个按钮来添加更多子级。子级有一些标准字段，与父级相同，没什么花哨的。主要要求是，在所有数据都有效并设置之前，数据不得接触数据库，而我必须返回服务器来添加删除子项。

实现

很快就可以在 ASP.NET MVC 中实现此功能（使用 MVC 2 和 VS 2010）。我有两种模型，一种用于父母，一种用于孩子，并且只有一个控制器。控制器有一个 Create 方法，它是一个 get 方法，并获取一个默认视图，其中包含一个包含一个子级的全新父级视图。我对子模型使用编辑器模板，效果很好。

我有一个 HTML 表单，其中有“保存”和“添加子项”，每个表单都有“删除”按钮。由于这无法存储在数据库中，因此我将临时模型存储在表单本身中，并在浏览器和服务器之间来回传递。性能在这里并不是什么大问题，而是开发成本，因为这些形式有相当多——所以请不要因为建议替代方法而分心，尽管无论如何我很欣赏评论。

为了找出要删除的子项，我创建了临时 GUID Id 并将它们与该子项关联。这将进入删除按钮的 HTML 输入值（当您有多个操作和相同表单时通常使用的技巧）。

我已经禁用了缓存。

问题

请查看下面的代码片段。我已经调试了代码，并且看到始终传递正确的 GUID、从控制器的列表中删除正确的项目以及在模板中呈现正确的项目。但总是最后一个被删除！我通常点击第一个删除，然后可以看到最后一个被删除。我继续，第一个项目是最后一个被删除的项目。

控制器

    public ActionResult Create()
    {
        EntryForm1 entryForm1 = new EntryForm1();
        entryForm1.Children.Add(new Child("FILL ME", "FILL ME"){ TempId = Guid.NewGuid()});
        return View("EntryForm1View", entryForm1);
    }

    [HttpPost]
    public ActionResult Create(EntryForm1 form1, FormCollection collection, string add)
    {
        if (add == "add")
            form1.Children.Add(new Child("FILL ME", "FILL ME") {TempId = Guid.NewGuid()});

        var deletes = collection.AllKeys.Where(s => s.StartsWith("delete_"));
        collection.Clear();
        if (deletes.Count() > 0)
        {
            string delete = deletes.FirstOrDefault();
            delete = delete.Replace("delete_", "");
            Guid g = Guid.Parse(delete);
            var Children = form1.Children.Where(x => x.TempId == g).ToArray();
            foreach (Child child in Children)
            {
                form1.Children.Remove(child);
            }               
            // HERE CORRECT ITEM IS DELETED, BELIEVE ME!!
        }
        if (ModelState.IsValid)
        {
            return Redirect("/");
        }
        return View("EntryForm1View", form1);
    }

查看代码段

    <% for (int i = 0; i < Model.Children.Count;i++ )
  {%>
        <h4> <%: Html.EditorFor(m=>m.Children[i])%></h4>

        <%
  }%>
        <p>
            <input type="submit" value="Create" name="add" />
            <input type="submit" value="add" name="add" />
        </p>

子编辑器模板代码段

        <%: Html.HiddenFor(x=>x.TempId) %>
        </span>
        <input type="submit" name='delete_<%: Html.DisplayTextFor(m => m.TempId) %>' value="Delete" />

非常感谢您的时间和关注

---

更新

我被要求提供模型课程，我将按原样分享它们。 Entryform1 是父级，Somesing 是子级。

索辛公开课 {

    public Somesing()
    {

    }

    public Somesing(string o, string a) : this()
    {
        OneSing = o;
        AnozerSing = a;
    }

    [StringLength(2)]
    public string OneSing { get; set; }

    [StringLength(2)]
    public string AnozerSing { get; set; }

    public Guid TempId { get; set; }

}

public class EntryForm1
{

    public EntryForm1()
    {
        Sings = new List<Somesing>();
    }

    public string FirstName { get; set; }

    public string LastName { get; set; }

    public int Age { get; set; }

    public List<Somesing> Sings { get; set; }

}

Answer 1

我认为问题出在 ModelState 上。当视图被渲染时（我认为这就是问题所在），在 POST 之后，最后一个值不会显示，即从视图中删除。

问题是 Model.Children.Count 将返回要显示的正确元素数。

让我们分解一下...

因此，如果您最初有 5 个，然后根据 Guid 删除了索引为 0 的第一个，那么您现在剩下的项目为 4 个，索引为 1 到 4。

但是，在渲染视图之后post 后，HtmlHelpers 不会查看发布的模型中的值，而是查看 ModelState 中包含的值。因此，在 ModelState 中，索引为 0 的项仍然存在，并且由于循环现在循环到 4，因此不会显示最后一个元素。

解决方案，使用 ModelState.Clear()

Answer 2

好的，正如 Ahmad 指出的那样，ModelState 是问题的关键。它包含这样的集合：

• FirstName
• LastName
• ...
• Sings[0].OneSing
• Sings[0].AnozerSing
• Sings [1].OneSing
• Sings[1].AnozerSing
• Sings[2].OneSing
• Sings[2].AnozerSing

现在如果我从列表中删除了项目 0，现在这些项目将在列表中向上移动，并且 ModelState 中的数据将与模型不同步。我原以为 ASP.NET MVC 能够足够聪明地找出并重新排序，但这要求太多了。

我实际上实现了 PRG（重定向后获取），并通过将模型保持在会话中，我能够显示正确的信息，但同样，这将删除集合中的所有验证，如果模型本身有效，它将愉快地保存并重定向回主页“/”。显然这是不可接受的。

因此，一个解决方案是删除 ModelState 中的所有项目，然后为模型本身添加一个新条目（键为 EmptyString）。如果您用错误“项目已删除”填充它，这实际上可以正常工作，因为这将显示在验证摘要中。

另一种解决方案是手动更改模型状态中的项目并根据新索引重新排列它们。这并不容易，但却是可能的。

Answer 3

ModelState.Clear() 将解决这个问题。

ModelState.Clear() 用于清除错误，但它也用于强制 MVC 引擎重建要传递到视图的模型。