Linq具有if条件的多重where子句

Question

我正在尝试使用多个where子句和where子句进一步包含内联if条件编写Linq。

List<MeetingVM> students = (
from s in db.Meetings 
where MeetingIsActive == null || s.IsActive == MeetingIsActive 
where MeetingStat == null || MeetingStat == 5 ? (DateTime.UtcNow >= s.MeetingStartTime && DateTime.UtcNow <= s.MeetingStopTime) : s.Status== MeetingStat
where StartDate == null || (s.MeetingStartTime >= StartDate && s.MeetingStartTime <= EndDate) 
where s.Status!=4 
orderby s.MeetingStartTime ascending 
select new MeetingVM
{
    MeetingStartTime = s.MeetingStartTime,
    MeetingStopTime = s.MeetingStopTime,
    Alias = s.Alias,
    MeetingSubject = s.MeetingSubject,
    UserId = s.UserId,
    Status=s.Status

}).ToList();

即使MeetingStat == null，也会执行此条件（DateTime.UtcNow> = s.MeetingStartTime && DateTime.UtcNow <= s.MeetingStopTime）。

where MeetingStat == null || MeetingStat==5 ? (DateTime.UtcNow >= s.MeetingStartTime && DateTime.UtcNow <= s.MeetingStopTime) : s.Status== MeetingStat

如果我删除此..

MeetingStat==5 ? (DateTime.UtcNow >= s.MeetingStartTime && DateTime.UtcNow <= s.MeetingStopTime) :

有用。 但是我想写这个来检查

if (MeetingStat==5)
  (DateTime.UtcNow >= s.MeetingStartTime && DateTime.UtcNow <= s.MeetingStopTime) 
else 
  s.Status== MeetingStat.

我在做什么错？

Answer 1

我认为问题在于DateTime.UtcNow最好将其值存储在变量nad中，然后在LINQ中使用如下所示的值：

DateTime UtcTime = DateTime.UtcNow;
// and change the condition like this;
.Where MeetingStat != null && MeetingStat==5 ? (UtcTime >= s.MeetingStartTime && UtcTime <= s.MeetingStopTime) : s.Status== MeetingStat

之所以如此，是因为您的Where子句中的谓词未转换为SQL，如果您这样使用，则意味着代码将按预期工作，因为UtcTime现在可转换

Answer 2

首先，让我们以一种并非很难理解的方式来格式化代码：

List<MeetingVM> students = (
    from s in db.Meetings 
    where MeetingIsActive == null || s.IsActive == MeetingIsActive 
    where MeetingStat == null || MeetingStat == 5 ? (DateTime.UtcNow >= s.MeetingStartTime && DateTime.UtcNow <= s.MeetingStopTime) : s.Status== MeetingStat
    where StartDate == null || (s.MeetingStartTime >= StartDate && s.MeetingStartTime <= EndDate) 
    where s.Status!=4 
    orderby s.MeetingStartTime ascending 
    select new MeetingVM
    {
        MeetingStartTime = s.MeetingStartTime,
        MeetingStopTime = s.MeetingStopTime,
        Alias = s.Alias,
        MeetingSubject = s.MeetingSubject,
        UserId = s.UserId,
        Status=s.Status

    }).ToList();

好一点。

现在，我不太确定这一点，因此更了解的人可以随时纠正我，但是根据C＃运算符优先级 ，OR（“ ||”）运算符的优先级高于条件运算符（“？”）。 ：“）运算符，因此此行：

where MeetingStat == null || MeetingStat == 5 ? ...

本质上是这样评估的：

where (MeetingStat == null || MeetingStat == 5) ? ...

如果MeetingStat等于true，这将使表达式评估三元运算的第一个分支。 从您对问题的描述来看，这不是您想要的，因此我建议在括号中包含三元运算，以使您的意图清楚：

where MeetingStat == null || (MeetingStat == 5 ? ... )