如果语句或Where扩展与For Each循环？

Question

我想知道哪两个代码示例会更有效率（或两者之间的差异可以忽略不计）？

For Each apple in AppleController.GetRedApples().Where(Function(a) PriceController.OnSale(a)) 
    'do something 
Next 

或

For Each apple in AppleController.GetRedApples() 
    If PriceController.OnSale(apple) Then 
     'do something 
    End If 
Next 

谢谢！

Answer 1

的差异可以忽略不计，而第二代码更具可读性，所以你应该喜欢，如果你想要一个

Answer 2

我会亲自分离出来：

Dim applesOnSale = AppleController.GetRedApples() _ 
            .Where(Function(a) PriceController.OnSale(a)) 
For Each apple in applesOnSale 
    'do something 
Next 

在一个完整的IDE中的第一部分可以是单个行 - 或者你可以使用VB查询表达式是这样的：

Dim applesOnSale = From apple in AppleController.GetRedApples() _ 
        Where PriceController.OnSale(apple) 

For Each apple in applesOnSale 
    'do something 
Next 

这将“你感兴趣的项目”与“我想要对项目做什么”分开，我个人认为它对可读性有用。

注意，在某些情况下，Where可能无法正常工作。然而， - 如果GetRedApples从LINQ返回一个表到SQL DataContext，例如，它会尝试将Where条款到SQL翻译 - 这是行不通的。

假设你使用LINQ to Objects，那么这个赋值实际上不会做任何循环 - 它只会设置查询。只有在循环查询时才会对每个项目评估“where”子句。

Answer 3

考虑在foreach vs linq/lambda讨论（也乔恩斯基特回答;-)这一个delve deeper into understanding LINQ/Lambda efficiency和性能。

Answer 4

的表现应该是差不多的，所以我会去为更可读的办法：

var applesOnSale = from apple in AppleController.GetRedApples() 
        where PriceController.OnSale(apple) 
        select apple; 

foreach(var apple in applesOnSale) 
{ 
    //do stuff 
} 

这是一样的你的第二个选择，但触摸更具可读性。