在asp.net mvc中一次更新多条记录

Question

我正在尝试使用asp.net mvc 4 和EF6 创建一个网站，我想一次更新多行。但由于某种原因，它不起作用，我收到这样的错误，

System.NullReferenceException：对象引用未设置为对象的实例

这是我的代码，

控制器

[HttpPost]
    public ActionResult MakeDue(List<BillCheck> BillLists)
    {
        if (Session["username"] != null)
        {
            if (ModelState.IsValid)
            {
                foreach (var BillId in BillLists)
                {
                    var getDue = db.BillChecks.Where(p => p.id == BillId.id).FirstOrDefault();
                    getDue.due = BillId.due;
                }
                db.SaveChanges();
                return RedirectToAction("Success");
            }
            else
            {
                return RedirectToAction("Failed");
            }
        }
        else
        {
            return RedirectToAction("Login");
        }
    }

查看

@using (Html.BeginForm("MakeDue", "Home"))
{
    @Html.ValidationSummary(true)
    @foreach(var item in Model.DueList)
    {
        @Html.HiddenFor(modelItem => item.id)
        <tr>
            <td>@Html.DisplayFor(modelItem => item.flat)</td>
            <td>@Html.DisplayFor(modelItem => item.name)</td>
            <td>@Html.TextBoxFor(modelItem => item.due)</td>
        </tr>
    }
    <input type="submit" class="btn btn-success" value="Update" />
}

我的代码有问题吗？如何一次更新due 的所有输入？

Answer 1

您的第一个问题是您使用foreach 循环会生成重复的name 属性，这些属性不会绑定到集合，因此BillLists 参数将始终是一个空集合（它也是生成重复的id 属性，这是无效的 html）。您需要为 typeof BillCheck 使用 for 循环或自定义 EditorTemplate。使用for 循环，您的视图需要是

using (Html.BeginForm("MakeDue", "Home"))
{
  @Html.ValidationSummary(true)
  @for(int i = 0; i < Model.DueList.Count; i++)
  {
    
    <tr>
      <td>
        @Html.HiddenFor(m => m.DueList[i].id)
        @Html.DisplayFor(m => m.DueList[i].flat)</td>
      <td>@Html.DisplayFor(m => m.DueList[i].name)</td>
      <td>@Html.TextBoxFor(m => m.DueList[i].due)</td>
    </tr>
  }
  <input type="submit" class="btn btn-success" value="Update" />
}

另请注意，@Html.HiddenFor() 帮助器必须位于 <td> 元素内才能成为有效的 html。

下一个问题是视图中的模型不是typeofList<BillCheck>，但它确实包含一个名为DueList的属性，即typeofList<BillCheck>所以你的POST方法需要是

public ActionResult MakeDue(YourModel model)

其中YourModel 是您用于生成视图的类名（即在@model ??? 语句中）。然后你在控制器中循环需要的方法是

foreach (var BillId in model.DueList)
{
  var getDue = db.BillChecks.Where(p => p.id == BillId.id).FirstOrDefault();
  if (getDue != null) // add this
  {
    getDue.due = BillId.due;
  }
}
db.SaveChanges();

还要注意添加了if (getDue != null) 检查。

旁注：您正在检查if (ModelState.IsValid)。如果ModelState 无效，建议您返回视图，以便用户更正任何错误。