NodeJS与MongoDB开发中map异步查询导致响应返回空数组的求助

你当前遇到的响应提前发送、返回空数组的问题，是两个异步执行逻辑的特性叠加导致的：

• Array.map方法本身是同步执行的，当你给它传入异步回调函数时，map不会等待回调内部的异步操作（也就是MongoDB查询）完成，会立刻返回一个由待决Promise组成的数组。外层的同步执行流会直接跳过所有数据库查询逻辑，顺着代码走到res.send的位置发送响应，此时quotations数组还没被任何数据填充。
• 代码本身存在字段名匹配错误：MongoDB中定义的字段是open_quotation，你代码中判断时写的是openquotation，即使时序修正了，这个判断逻辑也永远无法命中待报价的条目。

方案1：批量查询（性能最优，推荐使用） #

不需要循环单条查询，直接利用MongoDB的查询能力一次性完成过滤和匹配，减少数据库交互次数：

// 记得将当前路由处理函数声明为async
const { items } = req.body;
// 提取所有传入条目的ID
const itemIds = items.map(item => item._id);
// 一次性查询：匹配ID范围、且存在有效待报价的条目
const quotations = await Item.find({
  _id: { $in: itemIds },
  open_quotation: { $ne: null }
});
res.send({
  response: true,
  quotations: quotations
});

方案2：修正异步遍历的执行时序（适配需要逐条逻辑校验的场景） #

如果必须保留逐条查询、JS层判断的逻辑，需要将map生成的所有异步Promise通过Promise.all整体等待，确认所有查询完成后再发送响应：

const { items } = req.body;
const quotations = [];
// 生成所有异步查询任务的Promise列表
const queryTasks = items.map(async item => {
  const targetItem = await Item.findOne({ _id: item._id });
  // 修正字段名，匹配数据库定义的open_quotation
  if (targetItem && targetItem.open_quotation) {
    quotations.push(targetItem);
  }
});
// 等待所有查询任务全部执行完毕
await Promise.all(queryTasks);
res.send({
  response: true,
  quotations: quotations
});

• 路由处理函数必须添加async关键字，否则代码中的await语法会直接抛出运行时错误。
• 循环内单条调用findOne的写法会产生大量重复的数据库连接请求，数据量稍大时就会出现明显的性能瓶颈，优先使用批量查询方案。
• 数据库字段名要和代码判断逻辑完全一致，避免下划线、大小写不一致导致的判断失效。

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