使用多个Where子句的Linq筛选结果

Question

我正在尝试使用EF 5将多个搜索条件应用于结果集（在本例中，用于库目录搜索）。以下是相关代码：

public IQueryable<LibraryResource> GetSearchResults(string SearchCriteria, int? limit = null)
    {
        List<string> criteria = SearchCriteria.Split(new char[] { ' ' }, StringSplitOptions.RemoveEmptyEntries).ToList();
        IQueryable<LibraryResource> allResults = context.LibraryResources.Include("Type").Where(r => r.AuditInfo.DeletedAt == null);
        foreach (string criterion in criteria)
        {
            allResults = allResults.Where(r => (r.Title.Contains(criterion) || r.Keywords.Contains(criterion) || r.Author.Contains(criterion) || r.Comments.Contains(criterion)));
        }
        allResults = allResults.OrderBy(r => r.Title);
        if (limit.HasValue) allResults = allResults.Take(limit.Value);
        return allResults;
    }

示例搜索标准="历史时代"

出于某种原因，只应用最后一个标准。例如，在上面的示例中，返回所有在标题、作者、关键字和评论中包含"时代"的书籍，而不必按"历史"进行过滤。我逐步完成了代码，循环执行了两次，每次都有适当的标准。你能看到我看不到的东西吗？谢谢

Answer 1

您已经成为修改封闭变量值的牺牲品。

将代码更改为：

foreach (string criterion in criteria)
{
    var crit = criterion;
    allResults = allResults.Where(/* use crit here, not criterion */);
}

这里的问题是，在构建查询时，过滤表达式会关闭变量criterion，实际上会在计算查询时将其拉入范围。但是，此时criterion将只有一个值（它碰巧循环的最后一个值），所以除了最后一个过滤器之外，所有过滤器实际上都将变成最后一个的副本。

创建criterion的局部副本并在表达式中引用它可以纠正问题，因为crit每次都是一个不同的局部变量，其生存期不会从循环的一次迭代扩展到下一次迭代。

有关更多详细信息，您可能想阅读C#'；有原因吗；s重用foreach中的变量？，其中还提到，C#5.0将进行适用于此场景的突破性更改：循环变量criterion的生存期将发生更改，使此代码在没有额外本地的情况下正常工作。