Node/Express.js技术问题：从数据库取数存入数组后元素消失

嘿，这个问题我太熟了——核心原因是Mongoose的find()方法是异步操作，你的同步代码跑得比异步回调快多了！

为什么数组是空的？ #

你写的for循环是同步执行的，每一次循环都会发起Toy.find()请求，但这些请求不会阻塞代码继续运行。当console.log(all_prices)执行的时候，所有的find()回调函数都还没来得及触发，all_prices自然还是空数组。

另外还有个小坑：你在内部循环里又用了变量i，和外层循环的i重名了，这会导致变量污染（因为var没有块级作用域），哪怕异步问题解决了，也可能拿到错误的循环索引。

解决方案：用异步/等待（Async/Await）来同步异步操作 #

这是最直观的写法，把整个逻辑包在一个async函数里，用await等待每一次find()的结果：

// 把逻辑封装成async函数
async function fetchToyPrices() {
  const all_prices = [];
  
  for (let i = 0; i < query_id_list.length; i++) { // 改用let避免变量污染
    const query = query_id_list[i];
    if (Object.keys(query).length === 0) {
      return res.json({}); // 空查询直接返回
    }

    try {
      // 用await等待find完成，代替回调函数
      const toys = await Toy.find(query);
      // 内部循环改用j，避免和外层i冲突
      for (let j = 0; j < toys.length; j++) {
        all_prices.push(toys[j].price);
      }
    } catch (err) {
      console.error('查询玩具失败:', err);
      return res.json({});
    }
  }

  console.log(all_prices); // 现在能正确输出所有价格了
  // 这里可以把结果返回给前端
  res.json({ prices: all_prices });
}

// 调用函数执行逻辑
fetchToyPrices();

更高效的方案：用Promise.all并行处理所有查询 #

如果你的查询之间没有依赖关系，用Promise.all可以并行发起所有请求，比逐个等待更快：

// 先把所有查询转换成Promise
const priceQueries = query_id_list.map(query => {
  if (Object.keys(query).length === 0) {
    // 空查询直接返回失败的Promise
    return Promise.reject(new Error('空查询参数'));
  }
  // 每个查询返回对应的价格数组
  return Toy.find(query).then(toys => toys.map(toy => toy.price));
});

// 等待所有Promise完成
Promise.all(priceQueries)
  .then(priceArrays => {
    // 把二维数组扁平化，得到所有价格的一维数组
    const all_prices = priceArrays.flat();
    console.log(all_prices);
    res.json({ prices: all_prices });
  })
  .catch(err => {
    console.error('查询出错:', err);
    res.json({});
  });

额外注意点 #

• 尽量用let/const代替var，避免变量提升带来的意外问题。
• 不要在多个回调里重复调用res.json()，不然会导致"Cannot set headers after they are sent to the client"的错误——应该统一在所有异步操作完成后或者错误发生时只返回一次响应。

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