如何正确创建Promise数组批量调用API获取加密货币价值？

让我来帮你一步步解决这两个问题，先理清楚问题出在哪，再给出正确的实现：

问题1：带await的for循环执行顺序不符合预期 #

你原来的for循环代码有几个关键错误，导致日志顺序和计算逻辑混乱：

• 循环里错误使用了i.coinName，应该是coins[i].coinName（i是数组索引，不是数组元素）
• 把await和.then()混在一起使用，await本身就会等待Promise完成并返回结果，不需要嵌套.then()
• 变量累加和日志输出的逻辑没有对齐异步执行顺序

修正后的串行for循环写法（更易读的for...of版本） #

async function calculateTotalValue(coins) {
  let totalValue = 0;
  for (const coin of coins) {
    try {
      // await直接获取API返回结果，无需额外then
      const response = await this.coinRepository.getActiveCoin(coin.coinName);
      // 注意API返回的是数组，取第一个元素的欧元价格
      const priceEur = response[0].price_eur;
      const coinValue = priceEur * coin.amount;
      totalValue += coinValue;
      console.log(`币种 ${coin.coinName} 价值：${coinValue}`);
    } catch (error) {
      console.error(`获取 ${coin.coinName} 价格失败：`, error);
      // 可选择抛出错误终止计算，或跳过该币种继续
      throw error;
    }
  }
  console.log(`总价值：${totalValue}`);
  return totalValue;
}

这个写法会逐个等待每个API请求完成后再执行下一个，日志会按顺序输出每个币种的计算过程，最后返回总价值。

问题2：Promise.all未按预期执行/未计算总价值 #

你原来的Promise.all代码有两个核心问题：

1. 所有Promise完成后没有计算总价值，也没有resolve()最终结果，导致调用方无法获取计算后的数值
2. 代码结构不完整（缺少闭合的}）

修正后的并行Promise.all写法（效率更高） #

function calculateTotalValue(coins) {
  // 把每个币种转换成「获取价格并计算价值」的Promise
  const promises = coins.map(coin => {
    return this.coinRepository.getActiveCoin(coin.coinName)
      .then(response => {
        const priceEur = response[0].price_eur;
        return priceEur * coin.amount; // 直接返回该币种的价值
      })
      .catch(error => {
        console.error(`获取 ${coin.coinName} 价格失败：`, error);
        return 0; // 失败时按0计算，或抛出错误终止全部计算
      });
  });

  // 等待所有请求完成后累加结果
  return Promise.all(promises)
    .then(coinValues => {
      const totalValue = coinValues.reduce((sum, value) => sum + value, 0);
      console.log(`各币种价值列表：`, coinValues);
      console.log(`总价值：${totalValue}`);
      return totalValue;
    })
    .catch(error => {
      console.error('批量获取价格失败：', error);
      throw error;
    });
}

这个写法会同时发起所有API请求，等所有请求完成后统一计算总价值，比串行循环效率高很多，适合币种数量较多的场景。

额外注意事项 #

• 确保coins数组的每个元素都包含coinName和amount属性，否则会出现计算错误
• API请求失败的处理逻辑可根据业务需求调整：是跳过失败币种继续计算，还是直接终止整个流程
• 你使用的request库已被弃用，建议换成axios或node-fetch这类原生支持Promise的HTTP库，代码会更简洁

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