所以,我觉得这个很直接.
这是我的代码：
Dictionary<int,IEnumerable<SelectListItem>> fileTypeListDict = new Dictionary<int,IEnumerable<SelectListItem>>();
foreach (PresentationFile pf in speakerAssignment.FKPresentation.PresentationFiles)
{
IEnumerable<SelectListItem> fileTypes = Enum.GetValues(typeof(PresentationFileType))
.Cast<PresentationFileType>().Select(x => new SelectListItem
{
Text = x.ToString(),Value = Convert.ToString((int)x),Selected = pf.Type == (int)x
});
fileTypeListDict.Add(pf.ID,fileTypes);
}
发生的事情是,最后字典将具有所有正确的键,但所有值将被设置为在循环的最后一次迭代期间创建的fileTypes列表.我确信这与用作参考的对象有关,但在我使用C#之前没有看到过这个问题.任何人都在意解释为什么会这样,以及我应该如何解决这个问题？
谢谢！
解决方法
这是臭名昭着的“foreach捕获问题”,并在C#5中“修复”(“固定”是一个强有力的词,因为它表明它之前是一个“bug”：实际上 – 规范现在改为承认这是混淆的常见原因).在这两种情况下,lambda捕获变量pf,而不是“在此迭代期间pf的值” – 但在C#之前的5中,变量pf在技术上限定在循环外(因此只有其中一个,句点),其中 – 在C#5及更高版本中,变量在循环内部作用域(因此,为了捕获目的,每次迭代都有一个不同的变量).
在C#4中,只是作弊：
foreach (PresentationFile tmp in speakerAssignment.FKPresentation.PresentationFiles)
{
PresentationFile pf = tmp;
//... as before
现在pf的范围在foreach中,它可以正常工作.没有它,只有一个pf – 并且因为你将执行推迟到结束,所以单个pf的值将是最后一次迭代.
另一种解决方法是：不要推迟执行：
fileTypeListDict.Add(pf.ID,fileTypes.ToList()); // note the ToList
现在它被评估,而pf是“当前”,因此将得到您期望的结果.
