Mongoose中findOne结果无法在函数外部访问的问题咨询

你这是踩了Node.js异步编程的经典坑——异步代码的执行顺序问题。collection.findOne是异步方法，它的回调函数要等数据库查询完成后才会触发，但你的外部代码不会等着这个异步操作结束，所以当你在queryCollection函数外面访问rest时，查询还没跑完，rest自然是空的。

问题根源拆解 #

你当前的代码逻辑是这样的：

1. 定义了var rest = []
2. 调用queryCollection，它内部启动了一个异步数据库查询
3. 主线程会直接跳过异步操作，继续执行queryCollection后面的代码（也就是你写的c...部分），但这时候findOne的回调还没执行，rest里根本没数据

修复方案1：正确利用回调函数 #

既然已经用到了回调，那就要把依赖rest数据的后续操作（比如插入另一个集合的代码）全放到回调函数里，确保只有当查询完成、rest有数据后才执行这些操作：

var rest = [];

function queryCollection(collection, callback, city){
  collection.findOne({cidade: city}, function(err, result) {
    if (err) {
      console.log(err);
      // 出错时也要调用回调，避免后续代码卡住
      callback(err);
      return;
    }
    console.log(result); // 此处有结果
    rest.push(result);
    console.log(rest); // 此处有结果
    // 把查询结果传给回调，方便后续直接使用
    callback(null, rest);
  });
}

// 调用时，把插入另一个集合的逻辑放到回调里
queryCollection(Location, function(err, data){
  if (err) {
    // 先处理错误
    return;
  }
  // 这里可以安全地用data（也就是rest）执行插入操作
  // 示例：AnotherCollection.insertMany(data, function(insertErr){ ... })
}, '你的目标城市参数');

修复方案2：改用Promise + async/await（更推荐） #

回调嵌套多了容易陷入“回调地狱”，用Promise和async/await能让异步代码看起来像同步代码，可读性强很多：

先把queryCollection改成返回Promise的形式：

function queryCollection(collection, city){
  return new Promise((resolve, reject) => {
    collection.findOne({cidade: city}, function(err, result) {
      if (err) {
        reject(err);
        return;
      }
      resolve(result);
    });
  });
}

// 用async函数处理异步逻辑
async function main(){
  try {
    const result = await queryCollection(Location, '你的目标城市参数');
    const rest = [result]; // 把结果存入数组
    console.log(rest); // 这里能正常拿到数据
    // 在这里执行插入另一个集合的操作
    // 示例：await AnotherCollection.insertOne(rest[0]);
  } catch (err) {
    console.log(err);
  }
}

// 调用主函数
main();

额外小提醒 #

• 尽量别用全局变量rest，最好通过函数返回或回调传递数据，代码会更模块化，不容易出问题
• 异步操作一定要处理错误，不然出错了很难排查

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