执行含多异步调用的任务列表时遇重复结果问题求助

核心问题：闭包捕获共享变量引发的竞态条件 #

你遇到的两个现象（执行时看似跳回lambda开头、结果出现重复ContactDto）本质上是同一个问题：所有异步lambda捕获了外部的共享变量contactDto。

在你的代码里，contactDto是定义在Select lambda外面的变量，当你启动多个并发任务时，所有任务都会引用同一个contactDto实例。当多个任务同时执行contactDto = Mapper.Map<ContactDto>(node);时，它们会互相覆盖这个变量的值——比如任务A刚把它赋值为节点X的Dto，任务B立刻把它改成节点Y的Dto，最终所有任务返回的都是最后被赋值的那个Dto，这就是结果重复的原因。

而调试时看到“跳回lambda开头”的现象，其实是调试器的误导：当另一个任务修改了共享的contactDto变量时，调试器显示的变量值被更新，让你误以为当前任务回到了开头重新执行，实际上是多个并发任务在同时操作同一个变量。

另外还有一个小问题：async lambda的返回值处理错误。你的lambda标记为async，但最后返回了Task.FromResult(contactDto)，这会导致lambda的实际返回类型是Task<Task<ContactDto>>，而不是预期的Task<ContactDto>。虽然Task.WhenAll也能处理这种嵌套的Task，但这是不必要的冗余，而且可能引发潜在的执行逻辑问题。

修复方案 #

要解决这个问题，只需要把contactDto的定义移到lambda内部，让每个任务拥有独立的变量副本，同时修正async lambda的返回值：

var syncActionTasks = updatedNodes.Select(async node => {
    // 将contactDto移到lambda内部，每个任务拥有独立的副本
    ContactDto contactDto = Mapper.Map<ContactDto>(node);
    try {
        var firstAsyncCallResult = await this.Contacts
            .GetByEmailAsync<ContactsRoot>(contactDto.Email);
        //...(代码省略)...
        var secondAsyncCallResult = await this.Contacts
            .AddOrUpdateAsync(firstAsyncCallResult);
        //...(代码省略)...
        var thirdAsyncCallResult = await this.Contacts
            .GetAccountContactAssociation(secondAsyncCallResult);
    } catch (ActiveCampaignException ex) {
        // 错误处理
    }
    // async lambda直接返回值即可，无需手动包装Task.FromResult
    return contactDto;
})
.ToList();
var result = await Task.WhenAll(syncActionTasks);
//...(代码省略)...

关键说明 #

1. 变量作用域隔离：把contactDto放到lambda内部后，每个异步任务都会创建自己的contactDto实例，不会再出现多个任务共享同一个变量的情况，从根本上避免了竞态条件。
2. 正确使用async/await：async方法会自动将返回值包装为Task，所以直接return contactDto;就可以，不需要调用Task.FromResult，这样代码更简洁，也避免了嵌套Task的问题。

这样修改后，每个任务都会独立处理自己的contactDto，并发执行时不会互相干扰，返回的result也会是每个节点对应的唯一ContactDto对象，同时调试时也不会再出现看似跳回开头的奇怪现象。

内容的提问来源于stack exchange，提问作者byte_slave