编译器是否连接 LINQ where 查询？

Question

考虑以下两个类似的代码示例。

一个 where 子句。

bool validFactory
  = fields
    .Where(
      fields => field.FieldType == typeof( DependencyPropertyFactory<T> ) &&
                field.IsStatic )
    .Any();

两个 where 子句。

bool validFactory
  = fields
    .Where( field => field.FieldType == typeof( DependencyPropertyFactory<T> ) )
    .Where( field => field.IsStatic )
    .Any();

我更喜欢第二种，因为我发现它更具可读性，并且会导致更少的格式问题，尤其是在使用自动格式时。当在单独的条件旁边（甚至上面）放置注释以阐明意图时，它也会更清晰。

我的直觉表明第二个代码示例的效率会较低。我当然可以自己编写一个简单的测试（如果没有人知道答案，我也会这样做）。现在我认为这对SO来说是完美的食物。 ;p

1. 一个比另一个更有效率吗？
2. 编译器是否足够聪明来优化它？

Answer 1

编译器不会尝试优化连续的“where”调用。 运行时库会这样做。如果您有一大堆彼此相邻的“where”和“select”调用，运行时将尝试将它们重新组织成更有效的形式。

在一些不寻常的情况下，“优化”当然会让事情变得更糟。我似乎记得乔恩·斯基特最近写了一篇关于此事的文章，尽管我不确定它在哪里。

Answer 2

编译器不允许对此进行优化，因为它不知道Where() 的作用。例如，您可能使用记录其结果的版本重载了Where()。 （抖动可以进行优化，但实际上不太可能。）

效率差异不太可能很大。您可以分析您的应用程序，看看它是否重要。

更新：显然抖动确实在这里执行优化。请参阅埃里克·利珀特的回答。

Answer 3

我不认为这里会有显着差异。但是，如果您强制枚举集合，那么我会期望有更多差异

bool validFactory   = fields
    .Where( field => field.FieldType == typeof( DependencyPropertyFactory<T> ) )
    .ToList()
    .Where( field => field.IsStatic )     
    .ToList()
    .Any(); 

在您的两个原始代码示例中，我看到相同的执行 - 第一项检查 FieldType，然后检查 IsStatic，如果存在则返回true。否则检查第二项，依此类推。不需要解析整个集合。

在上面的示例中，将独立于 IsStatic 检查来解析整个集合的 FieldType。这可能会降低效率。请注意，这在您的两个片段中都不是必需的。