c# - DbQuery 在 foreach 循环中的行为不同。为什么？

Question

如果我使用下面的代码，我会得到学习类(class) 1 和类(class) 2 的学生列表。(这几乎是我想要的。)

IQueryable<Student> filteredStudents = context.Students;
filteredStudents = filteredStudents
    .Where(s => s.Courses.Select(c => c.CourseID).Contains(1));
filteredStudents = filteredStudents
    .Where(s => s.Courses.Select(c => c.CourseID).Contains(2));
List<Student> studentList = filteredStudents.ToList<Student>();  

但是，如果我尝试在 foreach 循环中执行此操作(如以下代码所示)，那么我会得到一个列表，其中包含已注册循环中最后一门类(class)的所有学生。

IQueryable<Student> filteredStudents = context.Students;
foreach (Course course in filter.Courses) {
    if (course != null) {             
        filteredStudents = filteredStudents
            .Where(s => s.Courses.Select(c => c.CourseID).Contains(course.CourseID));
    }
}
List<Student> studentList = filteredStudents.ToList<Student>();

这种行为让我很困惑。谁能解释为什么会这样？以及如何绕过它？谢谢。

Answer 1

问题是 foreach 循环只为所有循环迭代创建一个单个 course 变量，然后这个单个变量被捕获到一个闭包中。还请记住，过滤器直到循环之后才真正执行。将它们放在一起，当过滤器执行时，这个单个 course 变量已经前进到 Courses 过滤器中的最后一项；你只检查最后一门类(class)。

我看到了四种解决问题的方法。

首先

为循环的每次迭代创建一个新变量(这可能是您最好的快速修复)

IQueryable<Student> filteredStudents = context.Students;
foreach (Course course in filter.Courses) {
    if (course != null) {  
        int CourseID = course.CourseID;            
        filteredStudents = filteredStudents
            .Where(s => s.Courses.Select(c => c.CourseID).Contains(CourseID));
    }
}
List<Student> studentList = filteredStudents.ToList<Student>();

第二

解析循环内的 IEnumerable 表达式(效率可能低得多):

IEnumerable<Student> filteredStudents = context.Students;
foreach (Course course in filter.Courses) {
    if (course != null) {             
        filteredStudents = filteredStudents
            .Where(s => s.Courses.Select(c => c.CourseID).Contains(course.CourseID))
            .ToList(); 
    }
}
List<Student> studentList = filteredStudents.ToList<Student>();

第三

使用更合适的 linq 运算符/lambda 表达式来消除 foreach 循环:

var studentList = context.Students.Where(s => s.Courses.Select(c => c.CourseID).Intersect(filter.Courses.Select(c => c.CourseID)).Any()).ToList();

或者以更易读的方式:

IQueryable<Student> filteredStudents = context.Students;
var courses = filter.Courses.Select(c => c.CourseID);
var studentList = filteredStudents
       .Where(s => s.Courses.Select(c => c.CourseID)
                       .Intersect(courses)
                       .Any()
       ).ToList();

如果您稍微尝试一下，通过巧妙地在内部使用 HashSet，性能应该达到或远远超过 foreach 循环，或者 — 如果您非常幸运 —通过向数据库发送 JOIN 查询)。请小心，因为很容易在此处编写一些内容，在 Intersect() 或 Any() 方法内对数据库进行大量“额外”调用。即便如此，这是我倾向于选择的选项，除了我可能不会费心在最后调用 .ToList() 之外。

这也说明了为什么我对 Entity Framework、linq-to-sql 甚至 NHibernate 或 ActiveRecord 等 ORM 没有太多用处。如果我只是在编写 SQL，我可以知道我得到的是正确的连接查询。我也可以使用 ORM 来做到这一点，但现在我仍然需要了解我正在创建的特定 SQL，而且我还必须知道如何让 ORM 执行我想要的操作。

第四名

使用 C# 5.0。 This is fixed in the most recent version of C# , 因此 for/foreach 循环的每次迭代都是它自己的变量。