4

我遇到了 Linq 的一个奇怪行为：使用两个看起来相同的 linq 表达式，我得到了不同的结果！如果我循环一次得到相同的结果，但在上面什么也找不到。

这是代码：

        Dictionary<String, String> mainDico = new Dictionary<String, String>();
        mainDico.Add("key1", "value1");
        mainDico.Add("key2", "value2");

        List<Dictionary<String, String>> ls = new List<Dictionary<String, String>>();

        Dictionary<String, String> fistDico = new Dictionary<String, String>();
        fistDico.Add("key1", "value1");
        fistDico.Add("key2", "value2");

        Dictionary<String, String> secondDico = new Dictionary<String, String>();
        secondDico.Add("key1", "other");
        secondDico.Add("key2", "other");

        ls.Add(fistDico);
        ls.Add(secondDico);


        IEnumerable<Dictionary<String, String>> failQuery = from p in ls
                                                            select p;

        IEnumerable<Dictionary<String, String>> successfulQuery = from p in ls
                                                            select p;

        String[] items = new String[] { "key1","key2" }; // with one element it works

        foreach (var item in items)
        {
            String temp = mainDico[item];
            failQuery = failQuery.Where(p => p[item] == temp);
            successfulQuery = successfulQuery.Where(p => p[item] == mainDico[item]);
        }

        Console.WriteLine(successfulQuery.SingleOrDefault() != null);//True
        Console.WriteLine(failQuery.SingleOrDefault() != null);//False
4

1 回答 1

5

问题是您正在关闭循环变量。

有问题的代码部分就在这里：

foreach (var item in items)
{
    String temp = mainDico[item];
    failQuery = failQuery.Where(p => p[item] == temp);
    successfulQuery = successfulQuery.Where(p => p[item] == mainDico[item]);
}

您正在创建一个item在第二种情况下关闭的 lambda（也是第一种情况；您应该真正解决该问题），并且直到 foreach 循环结束后才评估查询；这意味着item它将始终是 foreach 循环中的最后一项，而不是当前项。这可以通过创建一个新的局部变量来轻松解决，这就是您在第一种情况下所做的，这就是它起作用的原因。

这是一个相关链接，更详细地讨论了这个问题。（您可以通过搜索“关闭循环变量”找到更多信息。

请注意，这在 C# 5.0 中已更改，因为它经常导致混淆和错误。（这可能是某些人无法重现该问题的原因。）

还值得注意的是，这与字典无关。在您的查询item中，实际上始终是 中的最后一项foreach，而不是当前项，这就是它失败的原因。您所做的任何item依赖于当前值的操作都不会达到您想要的效果。

于 2012-10-31T18:33:29.177 回答