在linq foreach中调用方法 - 有多少开销？

Question

我正在考虑用Linq替换很多内联的foreach，这样做会产生新的方法，例如Current：

foreach(Item in List)
{
    Statement1
    Statement2
    Statement3
}

理念：

List.Foreach(Item => Method(Item))

显然，Method（）包含Statement1..3

这是一种好的做法还是数千次调用方法会降低性能？ 我的名单有10,000-100,000个元素。

Answer 1

好吧，首先，您可以使用方法组转换使ForEach语句更有效

List.ForEach(Method);

这消除了一个间接层。

就个人而言，我认为这不是一个好主意。 第一种方法更具可读性，并且可能也可以执行。 在这里使用List<T>.ForEach什么好处？

Eric Lippert在一篇优秀的博文中谈到了这一点。 如果你已经有一个你想对每个元素执行的委托，我会使用List<T>.ForEach ，但我不会为了它而引入委托和额外的方法。

在效率方面，我不希望看到太大的差异。 第一种形式可能执行得更好，因为它没有委托调用的间接 - 但如果ForEach的迭代循环利用它可以访问内部数据结构的事实，则第二种形式可能更有效。 List<T> 。 我非常怀疑你会注意到它。 当然，如果你真的很烦，你可以尝试测量它。

Answer 2

如果你考虑改变的动机是身体中的三个陈述太复杂，那么我可能会使用普通的foreach ，但是将身体重构为一个方法：

foreach(var item in List) 
  Method(item);

如果正文中的代码并不复杂，那么我同意Jon的观点，即没有充分的理由使用ForEach （它不会使代码更具可读性/声明性）。

我通常不喜欢使用“LINQ-like”结构在LINQ查询结束时进行命令性处理。 我认为使用foreach更清楚地表明你已经完成了查询数据并且你现在正在进行一些处理。

Answer 3

我完全赞同Jon Skeet的回答。 但是既然我们正在讨论ForEach表现，那么我的问题会有所增加。 请注意，如果您的声明1~3彼此不相关，即：

foreach(Item in List) 
{ 
   DoSomething();
   DoAnotherThing(); 
   DoTheLastThing();
} 

上面的代码可能比以下代码更差：

foreach(Item in List) 
{ 
   DoSomething();
} 
foreach(Item in List) 
{ 
   DoAnotherThing(); 
} 
foreach(Item in List) 
{ 
   DoTheLastThing();
} 

后一个需要2个以上循环的代码具有更好性能的原因是因为当它持续调用DoSomething（）数千次时，一些必要的变量总是在CPU寄存器中warm 。 非常低的成本用于访问这些变量。 另一方面，如果在调用DoAnotherThing()之后立即调用DoSomthing() ，那些已经在CPU寄存器中的DoSomething()变量将会冷却下来。 在下一个循环中访问这些变量需要更多的成本。

Answer 4

我一直认为你应该首先编写代码以便于阅读，因为编译器和CLR在优化时做了出色的工作。 如果您通过基准测试发现，此代码可以更快地执行，那么请务必查看其他选项。

例如，for循环比foreach（）更快，因为它们使用在CLR内部优化的数组偏移。

但是，List.ForEach（）肯定没有foreach（），所以你只是将工作交给另一个方法，而不是自己做。

严格地说，引入更多的方法调用实际上会在第一次传递时减慢代码速度，因为CLR将在调用方法时对JIT进行编译，尽管后续调用方法不会。

所以我的建议是坚持编写可读代码，然后从那里开始，如果你能证明这是系统的瓶颈。