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;
    }

示例SearchCriteria =“历史时代”

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

Answer 1

您已成为修改已结算变量值的牺牲品。

将代码更改为：

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

这里的问题是，当您构建查询时，您的过滤表达式会关闭变量criterion，实际上是在评估查询时将其拉入范围。但是，那时criterion只会有一个值（最后一个值才会循环），所以除了最后一个过滤器之外的所有过滤器实际上都会变成最后一个重复的值之一。

创建criterion的本地副本并在表达式中引用它会更正问题，因为crit每次都是不同的局部变量，其生命周期不会延伸到循环的一次迭代到下一次。

有关更多详细信息，您可能需要阅读Is there a reason for C#'s reuse of the variable in a foreach?，其中还提到C＃5.0将采用适用于此场景的重大更改：循环变量criterion的生命周期将转到更改，使此代码正常工作，无需额外的本地。

Linq过滤结果包含多个Where子句

1 个答案: