小心使用 List 的 ForEach 方法

opendotnet 08-03 3778

前言：

如今各位老铁们对“netforeach”大约比较关心，小伙伴们都需要分析一些“netforeach”的相关内容。那么小编在网上网罗了一些对于“netforeach””的相关知识，希望姐妹们能喜欢，兄弟们快快来了解一下吧！

小心使用 List 的 ForEach 方法Intro

最近在我们的项目中发现有使用 list 的 ForEach 方法，并且 ForEach 里的是一个异步方法，这导致原本我们想要等待 ForEach 中的任务完成之后再继续其他另外一个 task，但是出现了 ForEach 的 task 还未完成，另外一个 task 已经开始执行了

Sample

前面说的可能有些绕，我们来看个示例

public static void MainTest()
{
var list = Enumerable.Range(1, 5).ToList();
 list.ForEach(async i =>
 {
await Task.Delay(TimeSpan.FromSeconds(1));
 Console.WriteLine($"SubTask {i} completed.");
 });
 Console.WriteLine("Task completed");
 Console.ReadLine();
}

大家可以先猜一下输出结果是什么哈

我们预期的结果应该是上面 ForEach 里的 task 都完成之后再输出 Task completed

但实际上并非如此，输出结果如下(SubTask 输出的顺序可能会有不同，但应该都是先输出 Task completed)

ForEach output

为什么会出现这样的结果呢，为什么没有按照预期等 sub task 完成呢

我们可以看下这个方法的签名，可以参考微软文档：

public void ForEach (Action<T> action);

可以看到 ForEach 这个方法的参数是一个 Action<T> 是一个同步的委托方法，如果我们在里面写了一个异步方法是不会等待的，这等同于委托是一个 async void 的方法，这就会导致在 await 的时候 subTask 并未结束就已经返回开始执行其他的逻辑了

如何修改让他符合我们的预期呢，我们可以使用 foreach 遍历来代替

public static async Task MainTestAsync()
{
var list = Enumerable.Range(1, 5).ToList();
foreach (var i in list)
 {
await Task.Delay(TimeSpan.FromSeconds(1));
 Console.WriteLine($"SubTask {i} completed.");
 }
 Console.WriteLine("Task completed");
 Console.ReadLine();
}