Linq 中的 ToList().ForEach

Question

我是 Linq 新手。

我想在 foreach 语句中设置两个值，如下所示

我的实际代码是这样的

foreach (Employee emp in employees)
{
    foreach(Department dept in emp.Departments)
    {
        dept.SomeProperty = null;
    }
    collection.AddRange(emp.Departments);              
}

一点重构将上面的内容变成这样

foreach (Employee emp in employees)
{
    emp.Departments.ToList().ForEach(u => u.SomeProperty = null))
    collection.AddRange(emp.Departments);              
}

但我想要这样的东西

employees.ToList().Foreach(collection.AddRange(emp.Departments),
emp.Departments.ToList().ForEach(u => u.SomeProperty = null))
  

Answer 1

您不应该以这种方式使用 ForEach。阅读 Lippert 的“foreach”与“ForEach”

如果你想对自己（和世界）残酷一点，至少不要创建无用的 List

employees.All(p => {
    collection.AddRange(p.Departments);
    p.Departments.All(u => { u.SomeProperty = null; return true; } );
    return true;
});

请注意，All 表达式的结果是我们正在丢弃的 bool 值（我们使用它只是因为它“循环”所有元素）

我将重复一遍。您不应该使用 ForEach 来更改对象。 LINQ 应该以“函数式”方式使用（您可以创建新对象，但不能更改旧对象，也不能产生副作用）。而你所写的是创建这么多无用的 List 只是为了获得两行代码......

Answer 2

正如 xanatos 所说，这是对 ForEach 的误用。

如果您打算使用 linq 来处理此问题，我会这样做：

var departments = employees.SelectMany(x => x.Departments);
foreach (var item in departments)
{
    item.SomeProperty = null;
}
collection.AddRange(departments);

但是，循环方法更具可读性，因此更易于维护。

Answer 3

employees.ToList().ForEach(
     emp=>
     {
          collection.AddRange(emp.Departments);
          emp.Departments.ToList().ForEach(u=>u.SomeProperty = null);
     });

Answer 4

试试这个：

foreach (var dept in employees.SelectMany(e => e.Departments))
{
   dept.SomeProperty = null;
   collection.Add(dept);
}

Answer 5

employees.ToList().Foreach(u=> { u.SomeProperty = null; u.OtherProperty = null; });

请注意，我在每个 set 语句后使用了分号
那就是 -->

u.SomeProperty = null;
u.OtherProperty = null;

我希望这一定能解决您的问题。

Answer 6

您可以使用Array.ForEach()

Array.ForEach(employees, employee => {
   Array.ForEach(employee.Departments, department => department.SomeProperty = null);
   Collection.AddRange(employee.Departments);
});

Answer 7

你想要这个吗？

    employees.ForEach(emp =>
    {
        collection.AddRange(emp.Departments.Where(dept => { dept.SomeProperty = null; return true; }));
    });

Answer 8

尝试使用以下 Lambda 表达式组合：

employees.ToList().ForEach(emp => 
{
    collection.AddRange(emp.Departments);
    emp.Departments.ToList().ForEach(dept => dept.SomeProperty = null);                    
});