Node.js中Promise如何无需超时等待多个外部API请求完成后再resolve

• 业务需求：基于输入的数组，循环发起指定API请求，结合返回值计算对应数值，汇总为数组后通过Promise向外传递结果
• 当前实现痛点：必须依赖固定时长的setTimeout（例如5000毫秒），在等待所有接口完成数据推入数组后再执行resolve，避免提前返回空数组，希望移除这种不可靠的固定超时逻辑
• 运行环境为Node.js，初始实现代码如下：

const calculateTotal = (arr) => {
  let clientHotspotsTotal = [];
  return new Promise((resolve, reject) => {
    arr.forEach((data) => {      
      axios.get(`enpoint/data?.address`)
        .then((response) => {
          if (response?.data) {
            console.log('API SUCCESS');
            const val = (response?.data?.data?.total * data?.percentage) / 100;            
            clientHotspotsTotal.push({
              total: val,
            });
          } else {
            reject('API failed!');
          }
        })
        .catch((error) => {
          throw new Error(error);
        });
    });
    resolve(clientHotspotsTotal);     ///  此处会返回空数组，因为未等待接口请求完成
    setTimeout(() => {
       resolve(clientHotspotsTotal)      //  此处不会返回空数组，因为超时等待期间API已完成请求并将数据推入数组
    }, 5000)
  });
};

• forEach循环内部发起的axios请求是异步非阻塞的，不会等待单个请求响应就会继续执行下一轮循环，循环结束后立刻执行的resolve(clientHotspotsTotal) 会拿到尚未填充数据的空数组
• 固定setTimeout等待的方案可靠性不足：网络波动时5秒可能不足以覆盖所有接口的响应耗时，接口响应较快时又会造成无意义的等待延迟
• 原catch块中直接抛出Error的写法，会导致异常脱离当前Promise的管控范围，触发未捕获异常，无法正确向外传递错误信息

核心逻辑为收集循环中生成的所有API请求Promise实例，等待全部请求完成后再统一触发resolve，具体调整点：

• 遍历输入数组时，将每个axios请求返回的Promise实例存入独立集合，不直接操作外部数组
• 借助Promise.all()方法监听所有请求的完成状态，等所有请求成功返回后，汇总计算结果再执行resolve
• 单个请求失败时，将错误传递给外层Promise的reject流程，确保错误能被正常捕获

调整后的可运行代码：

const calculateTotal = (arr) => {
  return new Promise((resolve, reject) => {
    // 收集每一项数据对应的API请求Promise
    const requestList = arr.map(async (item) => {
      try {
        // 修正原代码的接口参数拼接问题，正确传入地址参数
        const response = await axios.get(`enpoint/data?address=${item.address}`)
        if (!response?.data) {
          throw new Error('API failed!')
        }
        // 计算单条数据的结果并返回
        const totalValue = (response?.data?.data?.total * item?.percentage) / 100
        return { total: totalValue }
      } catch (err) {
        // 单个请求的错误抛出后交由外层统一捕获
        throw err
      }
    })

    // 等待所有请求完成后，汇总结果并resolve
    Promise.all(requestList)
      .then((resultList) => resolve(resultList))
      .catch((err) => reject(err))
  })
}

• 如果业务允许单个接口失败不影响整体流程，可以将Promise.all替换为Promise.allSettled，过滤掉失败的请求后再汇总有效结果
• 该方案无需任何固定超时等待，等待时长完全匹配实际接口响应总耗时，从机制上避免了提前返回空数组的问题
• 注意修正原代码中接口地址的模板字符串拼接问题，确保请求参数正确传递

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