Node.js+MongoDB使用async/await批量查询结果为空异常问题

1. 该现象的根本原因 #

你代码中productIds.map(async (productId) => {})的回调是异步函数，map遍历会返回一组Promise对象，但你没有等待这组Promise全部执行完成就直接读取orderedItems数组。
保留带await的无用变量行时，这段额外的数据库查询等待时间刚好够前面的所有商品ID查询请求返回结果，数据被push到数组里，所以能得到预期输出。如果删除该行或移除await，代码会直接执行到console.log，此时异步查询还没返回结果，数组自然是空的。这完全是时序巧合，不是稳定的逻辑效果。

2. 不能靠保留无用变量解决 #

这种方式完全不可靠，只是刚好利用了额外查询的等待时间。如果后续商品数量变多、数据库响应变慢，哪怕保留无用变量，也可能出现结果不全、甚至仍然返回空数组的问题，完全不能用于生产环境。

3. 现有代码的核心问题 #

• 没有正确处理异步任务的等待逻辑，未对异步查询返回的Promise数组做统一等待处理
• 依赖不可控的时间差获取结果，逻辑稳定性为0
• 循环单次查询数据库会产生多次不必要的IO请求，性能低下

正确实现方案 #

方案1：等待所有异步查询完成（适配原有遍历逻辑） #

用Promise.all统一等待所有查询Promise执行完成，直接拿到结果数组：

productRouter.post("/", async (req, res) => {
  const productIds = req.body.orderedItems.map((item) => item.id); 
  // 等待所有查询请求完成，直接得到结果数组
  const orderedItems = await Promise.all(
    productIds.map(productId => Product.findById(productId))
  );
  console.log(orderedItems);
  // 后续业务逻辑
});

方案2：批量查询（推荐，性能更高） #

用MongoDB的$in操作符一次性查询所有匹配ID的商品，仅需一次数据库请求：

productRouter.post("/", async (req, res) => {
  const productIds = req.body.orderedItems.map((item) => item.id); 
  // 批量查询所有匹配ID的商品
  const orderedItems = await Product.find({ _id: { $in: productIds } });
  // 如果需要结果顺序和productIds顺序一致，可自行排序处理
  console.log(orderedItems);
  // 后续业务逻辑
});

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