ASP.Net MVC的ViewBag一个坑，不用跳进去

ASP.Net MVC的ViewBag一个坑，不要跳进去

如鹏的学习管理系统是使用ASP.net MVC 5开发的，今天一个新版本发布后网站出现一个Bug，学生在下拉列表中选中的项再加载显示的时候发现仍然没被选中。详细一点说吧：
假如有这样一个Action：

public ActionResult Index()
{
List<SelectListItem> persons = new List<SelectListItem>(); 
persons.Add(new SelectListItem { Text = "腾讯", Value = "qq" });
persons.Add(new SelectListItem { Text = "如鹏", Value = "rupeng", Selected = true });
ViewBag.persons = persons;
return View();
}

Cshtml是这样的：

@Html.DropDownList("persons", (IEnumerable<SelectListItem>)ViewBag.persons)
生成的html是这样的：

<select id="persons" name="persons">
<option value="qq">腾讯</option>
<option value="rupeng">如鹏</option>
</select>

　竟然第二项没有处于选中状态，太诡异了吧！

只要把DropDownList第二个参数的"persons"改成和”ViewBag.persons”的persons名字不一样就可以，比如：
@Html.DropDownList("persons1", (IEnumerable<SelectListItem>)ViewBag.persons)
这样就正确生成了：

<select id="persons1" name="persons1">
<option value="qq">腾讯</option>
<option selected="selected" value="rupeng">如鹏</option>
</select>

好诡异！！！

咋办？看源码！
DropDownList是定义在SelectExtensions扩展类中，DropDownList方法最终是调用SelectInternal方法，核心代码是这一段：

if (!flag && obj == null && !string.IsNullOrEmpty(name))
{
obj = htmlHelper.ViewData.Eval(name);
}
if (obj != null)
{
selectList = SelectExtensions.GetSelectListWithDefaultValue(selectList, obj, allowMultiple);
}

这个name参数就是我们传递给DropDownList的第一个参数"persons"，上面代码主要逻辑就是：首先到ViewData中查找名字为"persons"的值，我们知道ViewData和ViewBag是一样的，所以htmlHelper.ViewData.Eval(name)取到的值就是我们在Index中定义的List<SelectListItem>()集合。GetSelectListWithDefaultValue方法的代码是：

private static IEnumerable<SelectListItem> GetSelectListWithDefaultValue(IEnumerable<SelectListItem> selectList, object defaultValue, bool allowMultiple)
{
IEnumerable enumerable= new object[]
{
defaultValue
};
IEnumerable<string> collection = 
from object value in enumerable
select Convert.ToString(value, CultureInfo.CurrentCulture);

HashSet<string> hashSet = new HashSet<string>(collection, StringComparer.OrdinalIgnoreCase);
List<SelectListItem> list = new List<SelectListItem>();
foreach (SelectListItem current in selectList)
{
current.Selected = ((current.Value != null) ? hashSet.Contains(current.Value) : hashSet.Contains(current.Text));
list.Add(current);
}
return list;
}

注意，我们的List<SelectListItem>()集合被当成defaultValue参数传递给GetSelectListWithDefaultValue方法了（why？），在方法内部又把defaultValue给 Convert.ToString()一下，变成了”System.Collections.Generic.List`1[System.Web.Mvc.SelectListItem]”这么一个玩意， GetSelectListWithDefaultValue的主要逻辑就是查找selectList中等于”System.Collections.Generic.List`1[System.Web.Mvc.SelectListItem]”的值，能找到才算见了鬼呢！！！
经过上面的分析我们还可以知道，不能让cshtml中DropDownList的第一个name参数和ViewBag中任何一个属性重名，否则还是会有问题，比如

public ActionResult Index()
{
List<SelectListItem> persons = new List<SelectListItem>(); 
persons.Add(new SelectListItem { Text = "腾讯", Value = "qq" });
persons.Add(new SelectListItem { Text = "如鹏", Value = "rupeng", Selected = true });
ViewBag.persons = persons;
ViewBag.persons1 = new string[] { };
return View();
}

Cshtml如下：
@Html.DropDownList("persons1", (IEnumerable<SelectListItem>)ViewBag.persons)

生成的html中第二条数据照样不会被selected

不知道微软为什么把DropDownList这么简单的一个东西搞的这么复杂，正验证了这句话“写的越多，错的越多”。当然也许微软会给出理由说我们用错了，说“It’s not a bug,It’s a feature,by design”好吧！谢特！

5楼高卿: 这代码，有点眼熟，，，不过尽量不用viewbag的话。。那也就viewdata的了。或者其他前端框架了。

4楼Choo: 正确的用法是：@Html.DropDownList(quot;personsquot;, null)，，这样的写法方便性体现在DropDownListFor时，model对应属性的值就会自动被选上，而不需要在构建SelectList时去确定被选值。

3楼幻天芒: 想想也是这个问题，不过暂时没想到有说买更好的方式实现...

2楼小眼睛老鼠: 这是mvc的特性啊，也就是说前端的控件都会通过key 去viewdata 或者 viewbag作为当前控件的value，，也就是说如果你要设置某个控件例如 control1的value的话，那么只要在action中写成这样就好了，viewbag. control1 = ？？？，前端不用再在控件里面赋值了，，这里你正好吧value当做了dropdownlist的内部结构数据传进去了，，所以就。。。。。

1楼a.thinker: 约定大于配置，所以，你要确定asp.mvc约定了什么，，你可以试下另一种写法：，代码，var persons = new Listlt;objectgt; { new { Text = quot;腾讯quot;, Value = quot;qqquot;

ASP.Net MVC的ViewBag一个坑，不用跳进去

相关推荐