异步并行聚合API数据时避免计数覆盖的优化方案

首先明确：在JavaScript默认的单线程事件循环模型下，你当前的计数逻辑本身不会出现多线程场景下的计数覆盖问题——tokenKeyValue[token.symbol]++是同步执行的原子操作，执行过程中不会被其他异步回调插入打断。
但你当前的实现存在明确缺陷，且可以通过调整逻辑彻底规避共享状态带来的所有潜在风险，同时100%保留并行请求的执行效率：

• 存在笔误：遍历目标是segmentData.users，完成数判断却用了segment.users.length，会导致最终统计逻辑触发时机完全错误
• 共享可变全局变量的写法可维护性极差：如果后续接入WebWorker、多线程 runtime，或者给回调加额外的异步逻辑，很容易出现竞态
• 最终汇总逻辑散落在每个请求的finally回调中，逻辑耦合度高，调试和修改都很麻烦

核心思路是拆分职责：每个并行的请求任务只负责返回当前单个地址对应的代币统计结果，不在任务内修改任何外部共享变量；等所有请求完成后，在主线程同步完成全量结果的汇总。
这种写法下所有API请求还是并行发起，总耗时等于最慢的单个请求耗时，性能和原实现完全一致，同时从根源上消除了共享状态带来的计数错误风险。
优化后代码：

// 并行发起所有请求，每个任务独立返回单地址的统计结果
const singleUserStats = await Promise.all(
  segmentData.users.map(async (user: { address: string }) => {
    try {
      const tokenBalances = await getBalancesForUser(user.address);
      const countMap: Record<string, number> = {};
      if (tokenBalances?.length) {
        for (const token of tokenBalances) {
          countMap[token.symbol] = (countMap[token.symbol] ?? 0) + 1;
        }
      }
      return countMap;
    } catch (e) {
      console.error(`地址${user.address}余额查询失败:`, e);
      // 单个请求失败返回空对象，不中断整体统计流程
      return {};
    }
  })
);

// 所有请求结束后，同步汇总所有结果
const tokenKeyValue: Record<string, number> = {};
for (const stat of singleUserStats) {
  for (const [symbol, count] of Object.entries(stat)) {
    tokenKeyValue[symbol] = (tokenKeyValue[symbol] ?? 0) + count;
  }
}

// 转换为你需要的数组结构
const tokenCount = Object.entries(tokenKeyValue).map(([symbol, value]) => ({
  symbol,
  value
}));

• 性能无损耗：所有API请求保持并行执行，没有新增任何串行等待逻辑
• 无竞态风险：全程没有跨任务修改的共享可变变量，哪怕后续迁移到多线程环境也不会出现计数覆盖问题
• 逻辑解耦：请求逻辑、汇总逻辑完全分离，不需要手动维护完成计数器，也不会因为笔误导致流程异常
• 错误隔离：单个地址请求失败不会影响其他地址的统计结果，容错性更高

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