如何优化这个LINQ表达式w/where条件和调用方法？

Question

我正在寻求任何意见或提示，关于下面的方法，我已经使用LINQ找到一个集合中的某个属性为null，然后通过结果（子列表）并在另一个属性上执行方法相同的集合。

private void SetRaises() 
{ 
    if (employeeCollection != null) 
    { 
     var noRaiseList = employeeCollection .Where(emp => emp.Raise == null).ToList(); 
     foreach (var record in noRaiseList) 
     { 
     CalculateRaise(record); 
     } 
    } 
} 

public void CalculateRaise(Employee emp) 
{ 
    if (emp!= null) 
     emp.Raise = emp.YearsOfService * 100; 
} 

我并不在第一种方法喜欢的部分，SetRaises（），是下面的代码片段：

foreach (var record in noRaiseList) 
{ 
    CalculateRaise(record); 
} 

是否有一部分直接集成到我的LINQ表达式，即一种方式我不知道的一些扩展方法？

谢谢！

Answer 1

你可以做的第一件事是：不产生中间列表：

var pending = employeeCollection.Where(emp => emp.Raise == null); 
foreach (var record in pending) 
{ 
    CalculateRaise(record); 
} 

这是相同的：

foreach (var record in employeeCollection.Where(emp => emp.Raise == null)) 
{ 
    CalculateRaise(record); 
} 

这是现在非缓冲延迟执行。

但坦率地说，这里的LINQ并没有给你太多。你也可以只：

foreach(var emp in employeeCollection) 
{ 
    if(emp.Raise == null) CalculateRaise(emp); 
} 

Answer 2

如果您不需要员工的名单没有提高，你可以在一行做到这一点：

employeeCollection.Where(emp => emp.Raise == null).ToList().ForEach(x => x.Raise = x.YearsOfService * 100); 

Answer 3

你可以使用ForEach链方法。但那只是糖语法。

employeeCollection.Where(emp => emp.Raise == null) 
        .ToList() 
        .ForEach(record => CalculateRaise(record)) 

Answer 4

应该是这样的：

var noRaiseList = employeeCollection .Where(emp => emp.Raise == null).ToList().ForEach(e=>e.Raise = e.YearsOfService * 100);