1

考虑以下代码（虽然有点做作，但它是对现实世界程序的主要简化）：

string[] strings = { "ab", "abcd", "abc", "ac", "b", "abcde", "c", "bc" };
string[] filters = { "a", "b", "c" };

// iteratively apply each filter
IEnumerable<string> filteredStrings = strings.ToArray();
foreach (string filter in filters)
{
    // in my real-world program lots of processing and stuff
    // happens here, hence why i need the enclosing foreach loop

    filteredStrings = filteredStrings.Where(s => s.Contains(filter));
}

如您所见，代码迭代地将字符串数组过滤为较小的字符串集。当 for-each 循环完成时，filteredStrings应该是通过所有过滤器的字符串子集。在这个例子中，这将是：

{ "abcd", "abc", "abcde" }

但是，我得到的输出是：

{ "abcd", "abc", "ac", "abcde", "c", "bc" }

它似乎只是过滤掉那些不包含的字符串"c"，我认为这与它是最后一个过滤器有关。我想我一定没有以IEnumerable.Where()正确的方式链接。这里发生了什么，我怎样才能得到正确的输出？

是的，根据我的代码中的注释，该 foreach 循环需要保持不变。

4

2 回答 2

9

where 委托没有以您期望的方式捕获局部变量。变量filter正在发生变化，它在所有 where 中都使用最后一个结果，因为 Linq 会进行惰性求值。

将迭代变量复制到本地变量，我相信它会捕获您期望的方式。

foreach (string filter in filters)
{
    string localFilter = filter;
    filteredStrings = filteredStrings.Where(s => s.Contains(localFilter));
}
于 2013-02-02T17:13:47.790 回答
3

您正在使用在循环中修改的变量：filter

创建它的副本：

foreach (string filter in filters)
{
    // in my real-world program lots of processing and stuff
    // happens here, hence why i need the enclosing foreach loop

    string f = filter;
    filteredStrings = filteredStrings.Where(s => s.Contains(f));
}
于 2013-02-02T17:14:59.423 回答