异步方法中Promise.all异常：getItemDetails同步化问题求助

嘿，我一眼就看出问题所在了——你的getItemDetails方法没有返回Promise，这就是为什么await它的时候，代码会直接跳去执行下一行！

为什么会这样？ #

你在方法里调用了Promise.all()，但没有把这个Promise对象返回出去。async方法如果没有显式返回值，默认会返回一个已resolve的undefined。所以当server.js里执行await getItemDetails(itemList)时，它其实是在等待一个立即完成的Promise（值为undefined），自然不会等内部的网络请求完成就继续往下走了。

而且你内部.then()里的return itemDetails只是那个回调函数的返回值，不是整个getItemDetails方法的返回值，外部根本拿不到这个结果。

修复方案1：修正原有的.then写法 #

只需要把Promise.all()的调用结果return出去就行：

async getItemDetails(itemList) {
  let itemDetails = [];
  // 关键：return这个Promise链
  return Promise.all(itemList.map(id => this.axios.get(`/api/Item/${id.ItemID}/details`)))
    .then(responses => {
      responses.forEach(response => itemDetails.push(response.data))
      // 这里直接返回itemDetails给下一个then
      return itemDetails;
    })
    .catch(error => {
      console.error(error);
      // 最好在这里重新抛出错误，让外部能捕获到
      throw error;
    });
}

修复方案2：用async/await重构（更简洁易读） #

既然方法已经标记为async，完全可以抛弃.then()，用await来处理异步流程，代码会更清晰：

async getItemDetails(itemList) {
  try {
    // 用await等待Promise.all完成
    const responses = await Promise.all(
      itemList.map(id => this.axios.get(`/api/Item/${id.ItemID}/details`))
    );
    // 直接映射出data，不用额外的push操作
    return responses.map(response => response.data);
  } catch (error) {
    console.error(error);
    throw error; // 抛出错误让外部处理
  }
}

额外提醒 #

• 一定要在catch里重新抛出错误，否则外部的await会认为操作成功，但实际上已经出错了，这样你很难排查问题。
• 用map代替forEach+push来生成结果数组，代码更简洁高效。

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