为什么IF语句会影响我的LINQ语句的结果？

Question

我最近一直在和LINQ一起工作，感谢StackOverflowers的帮助，我得到了这个声明：

var traceJob =
    from jobDefinition in service.JobDefinitions
    where jobDefinition.Id == traceGuid
    select jobDefinition;

if (traceJob != null && traceJob.Count() == 1)
{
 traceJob.First().RunNow();
 Console.WriteLine(traceJob.First().DisplayName + "  Last Run Time: " + traceJob.First().LastRunTime);
}

但是，我感到困惑，因为使其工作的部分是if(traceJob.Count() ==1) 。 如果我删除该部分，那么我得到一个ObjectNullRef错误，说明traceJob的枚举没有产生任何结果。

现在，据我所知，检查计数的if语句实际上不应该改变Linq语句的结果吗？ 任何人都可以向我解释为什么我会看到这种行为？

Answer 1

不，它不应该。 我的猜测是你遇到了枚举真的是空的情况，并且通过检查count> 0， First()不会失败。

作为旁注， Any()在这里可能是更好的检查（取决于您的存储库的底层存储）它可能比Count()更快：

if (traceJob != null && traceJob.Any())
{
 traceJob.First().RunNow();
 Console.WriteLine(traceJob.First().DisplayName + "  Last Run Time: " + traceJob.First().LastRunTime);
}

Answer 2

var traceJob =
    (from jobDefinition in service.JobDefinitions
    where jobDefinition.Id == traceGuid
    select jobDefinition).SingleOrDefault();

您可以使用singleOrDefault检查单个结果。 它将返回与where条件匹配的结果，如果未找到匹配则返回null。 如果找到与查询匹配的多个匹配项，则会抛出异常。

这包括tracejob == null以及tracejob.count == 1条件。

MSDN文章

Answer 3

我不知道你的“服务”的实际实现，但通常linq查询实际上只在请求时填充他们的结果。 所以Count（）确实改变了traceJob的状态，很可能是填充内部集合。 看起来First（）不会填充内部集合，或者即使它应该正常也不能正确地执行。

Answer 4

关于https://stackoverflow.com/a/1745716/1289709
我认为在这里使用FirstOrDefault更合理。

每当您使用SingleOrDefault时，您都清楚地声明查询最多只能产生一个结果。 另一方面，当使用FirstOrDefault时，查询可以返回任意数量的结果，但是您声明只需要第一个结果。

所以将代码更改为以下内容：

var traceJob = (from jobDefinition in service.JobDefinitions
    where jobDefinition.Id == traceGuid
    select jobDefinition).FirstOrDefault();

    if (traceJob != null)
    {
       traceJob.RunNow();
       Console.WriteLine(traceJob.DisplayName + "  Last Run Time: " + traceJob.LastRunTime);
    }

Answer 5

我认为你的问题源于linq语句在被调用时的执行方式，而不是它们被声明的位置。

所以你的声明：

var traceJob =
    from jobDefinition in service.JobDefinitions
    where jobDefinition.Id == traceGuid
    select jobDefinition;

这大致在功能上等同于：

IEnumerable<JobDefinition> GetJobDefinitions(YourService service, Guid traceGuid) 
{
    foreach(var jobDefinition in service.JobDefinitions) 
        if(jobDefinition.Id == traceGuid) 
            yield return jobDefinition;
}

因此，当您调用traceJob.Count()您正在调用GetJobDefinitions(service, traceGuid).Count()并且每次调用traceJob.First()您都会再次调用循环。

如果可以反复调用service.JobDefinitions这可能不是问题。 但是，如果结果随时间而变化（例如，如果在执行期间添加了作业），或者如果后续运行具有不同的结果，则会出现问题。

在任何情况下，您最好只执行一次循环：

var traceJobs =
    from jobDefinition in service.JobDefinitions
    where jobDefinition.Id == traceGuid
    select jobDefinition;

// This is where the loop above actually executes - if it's empty it will return null
var firstJob = traceJobs.FirstorDefault();

if(firstJob != null)
{
    firstJob.RunNow();
    Console.WriteLine(firstJob.DisplayName + "  Last Run Time: " + firstJob.LastRunTime);
}

或者，您可以通过将循环转换为列表或数组来强制执行循环：

var traceJobsExecuted = traceJobs.ToList();