带有 ViewBag 的 MVC DropDownListFor 未将值传递给控制器

Question

我很难在我的创建视图中为相关模型/表设置下拉列表。 My Guest 模型通过以下方式引用 PersonPrefix 模型：

客座模特：

public virtual PersonPrefix Prefix { get; set; }

PersonPrefix 模型：

public class PersonPrefix
{

    [Key]
    public int PersonPrefixID { get; set; }
    [StringLength(6)]
    public string Abbreviation { get; set; }
    [StringLength(255)]
    public string Name { get; set; }
    public virtual ICollection<Guest> Guests { get; set; }

}

为了能够从数据库中获取数据并将其显示在下拉列表中，我做了以下操作：

控制器：

public ActionResult Create()
{
    ViewBag.PersonPrefixes = new SelectList(db.PersonPrefixes, "PersonPrefixID", "Abbreviation");
    return View();
} 

我已经在帖子中添加了前缀对象

[HttpPost]
[ValidateAntiForgeryToken]
public ActionResult Create([Bind(Include = "GuestID,FirstName,MiddleName,Surname,BirthDate,SelectedPrefix")] Guest guest)
{
    if (ModelState.IsValid)
    {
        db.Guests.Add(guest);
        db.SaveChanges();
        return RedirectToAction("Index");
    }

    return View(guest);
}

这是目前视图中的相关代码，但它没有将值传递给控制器​​：

  <div class="form-group">
        @Html.LabelFor(model => model.Prefix, htmlAttributes: new { @class = "control-label col-md-2" })
        <div class="col-md-10" >
            @*@Html.DropDownListFor(m => m.Prefix,  new SelectList(ViewBag.PersonPrefixes, "Value", "Text", 1))*@
            @Html.DropDownListFor(m => m.Prefix,
            new SelectList(ViewBag.PersonPrefixes, "Value", "Text", 1))

        </div>
    </div>

感谢您的帮助！！

Answer 1

您不能将<select> 元素绑定到复杂对象。所有 html 表单控件都回发一个值（或者在 <select multiple> 的情况下是一个值类型数组）。如果您选择了一个值为（例如）5 的选项，那么DefaultModelBinder 将尝试设置guest.Prefix = 5;，这当然会失败。

您需要绑定到值类型（例如int、string 等）。在您的情况下，您甚至无法绑定到PersonPrefix 的PersonPrefixID，因为验证将在PersonPrefix 的其他属性上失败。与往常一样，您应该使用仅包含您需要编辑的属性的视图模型。

public class GuestVM
{
  [Display(Name = "Prefix")]
  [Required(ErrorMessage = "Please select a prefix")]
  public int SelectedPrefix { get; set; }
  .... // other properties of Guest
  public SelectList PrefixList { get; set; }
}

控制器

public ActionResult Create()
{
  GuestVM model = new GuestVM();
  model.PrefixList = new SelectList(db.PersonPrefixes, "PersonPrefixID", "Abbreviation");
  .... // set other properties as required
  return View(model); // always return an instance of the model
}

查看

@Html.LabelFor(m => m.SelectedPrefix)
@Html.DropDownListFor(m => m.SelectedPrefix, Model.PrefixList, "--please select--")
@Html.ValidationMessageFor(m => m.SelectedPrefix)

然后在 POST 方法中，初始化Guest 数据模型的新实例，并从发布的视图模型映射其属性，最后保存数据模型。

public ActionResult Create(GuestVM model)
{
  if (!ModelSTate.IsValid)
  {
    model.PrefixList = new SelectList(db.PersonPrefixes, "PersonPrefixID", "Abbreviation");
    return View(model);
  }
  // Initialize a new instance of the data model and set its properties
  Guest guest = new Guest()
  {
    FirstName = model.FirstName,
    MiddleName = model.MiddleName,
    .... // other properties
    Prefix = db.PersonPrefixes.Find(model.SelectedPrefix)
  };
  db.Guests.Add(guest);
  db.SaveChanges();
  return RedirectToAction("Index");
}

旁注：您不需要在视图中创建另一个SelectList（它已经是SelectList）并且您尝试将所选值设置为1 的最后一个参数将被忽略（其值为你的绑定属性决定了选择哪个选项）所以如果你想用value="1"预先选择一个选项，那么在你将模型传递给视图之前，在控制器中设置SelectedPrefix = 1;的值。

Answer 2

虽然最好使用 ViewModels，但根据斯蒂芬的回答，你在这里做错的具体是这样的：

首先，您的 DropDownListFor 是错误的。您的 ViewBag 条目已包含一个 SelectList，因此您无需创建另一个...您只需转换动态类型即可。

@Html.DropDownListFor(m => m.Prefix.PersonPrefixID, 
                           (SelectList)ViewBag.PersonPrefixes, "Select...")

第一个参数是从 PersonPrefixes 中的对象列表中选择的 id...所以当模型发布到控制器时，它将包含 ID（但不包含其他数据，因此您必须确保查询数据库以填充它）。

其次，您需要将 Post Action 更改为：

Create([Bind(Include = "GuestID,FirstName,MiddleName,Surname,BirthDate")] Guest guest)

然后在您的 PersonPrefix 类上使用 Bind 语法：

[Bind(Include = "PersonPrefixID")]
public class PersonPrefix
{
    [Key]
    public int PersonPrefixID { get; set; }
    .....
}

现在，当您发布时，PersonPrefix 的其余部分将为空，因为该数据不存在于帖子中，因此您必须使用发布的 PersonPrefixID 从数据库中检索它。

但是，正如我所说，最好使用 ViewModel。即便如此，我在这里所说的一些相同的事情仍然适用。您的 DropDownListFor 需要正确，并使用选定的 id 属性。而且，如果您使用的是 ViewModel，则没有理由使用 Bind 属性，因为您只是绑定了您需要的确切项目。