Linq lambda表达式中的布尔短路计算

Question

我有以下的Linq lambda表达式:

private IEnumerable<SubjectSelectorSubjectGroup> GetSubjectList()
{
    User user = db.Users.Find(WebSecurity.CurrentUserId);
    return db.RequiredSubjects.Where(r => !r.Subject.Name.Contains("Home"))
                              .GroupBy(r => r.Subject)
                              .OrderByDescending(r => r.Count())
                              .Select(r => new SubjectSelectorSubjectGroup()
                              {
                                  SubjectId = r.Key.SubjectId,
                                  SubjectName = r.Key.Name,
                                  IsInFavourites = HttpContext.Current.Request.IsAuthenticated &&
                                                  (user.Elective1 != null && user.Elective1.SubjectId == r.Key.SubjectId ||
                                                   user.Elective2 != null && user.Elective2.SubjectId == r.Key.SubjectId ||
                                                   user.Elective3 != null && user.Elective3.SubjectId == r.Key.SubjectId),
                                  Occurrences = r.Count()
                              });
}

当用户未登录时，此函数中的user变量为空。这应该不是问题，因为短路布尔计算应该处理这个问题。问题是，它没有!相反，会抛出一个System.NullReferenceException。

当用户为空时，HttpContext.Current.Request.IsAuthenticated返回false。我通过注释掉引用user变量的括号部分来检查这一点，然后表达式计算正确。

有谁知道为什么Linq to Sql试图解引用user变量在这种情况下，它实际上不应该是必要的?有人有解决这个问题的方法吗?

Answer 1

整个表达式被翻译为SQL并作为SQL求值，这意味着&&操作符不会像预期的那样短路。

您可以通过构建要搜索的ElectiveX.SubjectId的列表或数组，然后在查询中使用tmpList.Contains(r.Key.SubjectId)来解决这个问题。这将被转换成WHERE IN (...) SQL表达式。

Answer 2

我是如何使用Anders Abel给出的建议来解决这个问题的。

private IEnumerable<SubjectSelectorSubjectGroup> GetSubjectList()
{
    List<string> userSubjects = new List<string>();
    if (HttpContext.Current.Request.IsAuthenticated)
    {
        User user = db.Users.Find(WebSecurity.CurrentUserId);
        if (user.Elective1 != null) { userSubjects.Add(user.Elective1.SubjectId); }
        if (user.Elective2 != null) { userSubjects.Add(user.Elective2.SubjectId); }
        if (user.Elective3 != null) { userSubjects.Add(user.Elective3.SubjectId); }
    }
    return db.RequiredSubjects.Where(r => !r.Subject.Name.Contains("Home"))
                              .GroupBy(r => r.Subject)
                              .OrderByDescending(r => r.Count())
                              .Select(r => new SubjectSelectorSubjectGroup()
                              {
                                   SubjectId = r.Key.SubjectId,
                                   SubjectName = r.Key.Name,
                                   IsInFavourites = userSubjects.Contains(r.Key.SubjectId),
                                   Occurrences = r.Count()
                              });
}

Answer 3

我能想到这种行为的唯一原因是您的查询正在获得转换为SQL(因为它的LINQ到SQL)，并且对于user.Elective它试图生成一个CASE语句。这就是你得到错误的原因。