LINQ语句中的多个OR子句

Question

我有下面的LINQ查询：

returnVal = context.ReservationRequests
   .Where(s => ((s.RequestStatusId.HasValue) &&
                (s.RequestStatusId.Value == ResStatusId)) &&
               ((string.IsNullOrEmpty(loggedInUserRole) 
                  || s.SubmitterGroupName == loggedInUserRole) 
                  ||(s.CreatedBy == ApplicationSecurityDirector.CurrentUserGuid)))                                                         
   .Skip(skip)
   .Take(take)
  .ToList();

该LINQ查询应该执行的操作将调查ReservationRequests表并查找其中RequestStatusId =（suppliedRequestStatusId）和SubmitterGroupName应该等于登录用户角色的记录（但是在某些情况下，用户未分配给任何角色）它也应该返回用户创建的任何请求。 因此，基本上，如果分配给特定组，则返回所有记录；如果有的话，还返回由亲自登录创建的请求。

上面的查询在用户登录到组的情况下工作正常，但是当用户未分配给任何组时，它不会返回正确的结果。 如果未将人员分配到任何组，则应返回该用户创建的所有记录。

下面是我编写的SQL查询，该查询返回所有案例的正确记录量，我基本上需要LINQ才能像此sql一样，但是我不确定这里缺少什么。

     SELECT *
  FROM [MyDB].[dbo].[ReservationRequests]
  where 
  (RequestStatusId = 2) 
  and 
  (SubmitterGroupName != null or SubmitterGroupName = null 
  or createdby = 'C5188D45-TEST-45BE-8C04-123455733A31')

有人可以调查我的LINQ查询，看看为什么返回的记录比我的SQL错误吗？ 谢谢，我已经看了一段时间了！

毕竟这里的建议是我更新的LINQ：

returnVal = context.ReservationRequests
.Where(s => ((s.RequestStatusId.HasValue) &&
(s.RequestStatusId.Value == ResStatusId)) &&
(s.SubmitterGroupName == loggedInUserRole ||
s.CreatedBy == ApplicationSecurityDirector.CurrentUserGuid))
.Skip(skip)
.Take(take)
.ToList();

那么问题：在loggingInUserRole为空的情况下，此查询是否有效？ 如果LoggedInUserRole为null，那么我只想返回已登录用户CreatedBy的记录。

另一个更新：（2017年9月22日）上午9.54，所以我运行了该语句。 在将用户分配给组但未将用户分配给组而不是仅显示已登录用户打开的请求的情况下，它会返回很多记录。

Answer 1

incorrect records是相对的...我想它返回的记录是正确的。 我们只能分析您在这里做什么以及区别在哪里：

在linq查询中搜索

s.SubmitterGroupName == loggedInUserRole

在您搜索的SQL语句中

SubmitterGroupName != null or SubmitterGroupName = null 

SubmitterGroupName不能为null，并且不能同时为null。因此，我猜loggedInUserRole为null。 与or一起使用，因此总是如此。 您只需要搜索createdby = 'C5188D45-TEST-45BE-8C04-123455733A31'

另外，在linq查询中

.Skip(skip).Take(take)

这在您的sql语句中丢失

所以我想你想要的是：

returnVal = context.ReservationRequests
.Where(s => ((s.RequestStatusId.HasValue) &&
(s.RequestStatusId.Value == ResStatusId)) &&
(s.SubmitterGroupName == loggedInUserRole || (s.SubmitterGroupName == null && s.CreatedBy == ApplicationSecurityDirector.CurrentUserGuid)))
.Skip(skip)
.Take(take)
.ToList();

Answer 2

我不完全确定为什么要在其中添加各种检查，但这应该可行：

returnVal = context.ReservationRequests
.Where(s => RequestStatusId == ResStatusId &&
    (s.SubmitterGroupName == loggedInUserRole || s.CreatedBy == ApplicationSecurityDirector.CurrentUserGuid)
.Skip(skip)
.Take(take)
.ToList();