如何解决ViewBag传递至@Html.DropDownList失效问题？

我来帮你排查这个问题，这种情况我碰到过好几次，大概率是ViewBag赋值或者DropDownList绑定的细节没处理对，咱们一步步来解决：

1. 确认控制器GET方法的ViewBag赋值是否正确 #

你用@Html.DropDownList("TransporteId", null, ...)的时候，框架会自动去ViewBag里找同名的SelectList对象（也就是ViewBag.TransporteId）来填充下拉选项。如果你的GET方法里赋值的是其他名称（比如ViewBag.Transportes），那下拉列表会没有选项，提交时TransporteId自然是空值，导致控制器走else分支。

正确的赋值方式应该是这样：

// 假设你有一个Transportes实体集合，Id是值字段，Nome是显示文本
public ActionResult Create()
{
    // 把选项集合放到和DropDownList第一个参数同名的ViewBag属性里
    ViewBag.TransporteId = new SelectList(db.Transportes, "Id", "Nome");
    return View();
}

2. 确保DropDownList和控制器参数的匹配 #

DropDownList的第一个参数是name属性，这个值必须和控制器POST方法的参数名完全一致（虽然C#大小写不敏感，但统一命名能避免模型绑定的意外问题）。

比如你的POST方法应该接收同名参数：

[HttpPost]
public ActionResult Create(int TransporteId, /* 其他表单参数 */)
{
    if (TransporteId > 0) // 假设你的Id是正整数类型
    {
        // 你的正常逻辑
        return RedirectToAction("Index");
    }
    else
    {
        // 你现在走到的分支
        // 这里要注意下面的第3点！
        return View();
    }
}

3. POST验证失败时，务必重新赋值ViewBag #

HTTP是无状态的，当你在else分支返回视图时，之前GET方法里的ViewBag数据已经丢失了。如果不重新给ViewBag.TransporteId赋值，回到视图时下拉列表会变成空的，用户再次提交还是会触发else分支。

所以else分支里要补上重新赋值的代码：

[HttpPost]
public ActionResult Create(int TransporteId)
{
    if (TransporteId > 0)
    {
        // 处理逻辑
        return RedirectToAction("Index");
    }
    else
    {
        // 重新填充下拉选项！
        ViewBag.TransporteId = new SelectList(db.Transportes, "Id", "Nome");
        // 可以加个验证提示告诉用户问题
        ModelState.AddModelError("TransporteId", "请选择有效的运输方式");
        return View();
    }
}

4. 可选：改用DropDownListFor更稳妥 #

如果你用的是强类型视图（推荐做法），可以换成DropDownListFor，这样绑定逻辑更明确，不容易因为ViewBag命名出错：

首先在GET方法里把选项放到另一个ViewBag属性（比如ViewBag.TransportOptions）：

ViewBag.TransportOptions = new SelectList(db.Transportes, "Id", "Nome");

然后视图里用强类型绑定：

@Html.DropDownListFor(m => m.TransporteId, (SelectList)ViewBag.TransportOptions, "请选择运输方式", new { @class = "form-control" })

这样模型绑定会更可靠，也更符合ASP.NET MVC的最佳实践。

内容的提问来源于stack exchange，提问作者caio oliveira