如何封装循环内的Promise数据库调用，预渲染前获取好友列表？

嘿，我太懂你这种困扰了——异步操作的执行顺序总是容易踩坑！你当前的代码里，User.findById是异步的Promise操作，而console.log(friendsList)是同步执行的，所以在所有好友查询请求完成前，它就已经跑了，自然拿不到填充好的数组。

下面给你两种清晰的解决方案，都是基于Promise的正确用法来处理批量异步操作：

方案一：用async/await（推荐，代码更易读） #

把路由处理函数改成async函数，用await等待所有异步操作完成：

router.get('/friends', auth.isLogged(), async (req, res) => {
  try {
    // 先获取当前用户的好友ID列表
    const user = await User.findById(req.user._id, { friends: 1 });
    
    // 把每个好友ID转换成查询Promise，存到数组里
    const friendQueryPromises = user.friends.map(friend => User.findById(friend._id));
    
    // 等待所有Promise执行完成，拿到完整的好友数据数组
    const friendsList = await Promise.all(friendQueryPromises);
    
    console.log(friendsList); // 现在这里就能拿到全量数据了
    // 后续可以把数据传给模板渲染或者返回JSON
    res.render('friends', { friendsList });
  } catch (err) {
    // 别忘了捕获错误，避免服务器崩溃
    console.error('获取好友列表出错:', err);
    res.status(500).send('获取好友列表失败，请稍后重试');
  }
});

方案二：用Promise链式调用（不使用async/await的写法） #

如果你更习惯传统的Promise链式写法，也可以这么写：

router.get('/friends', auth.isLogged(), (req, res) => {
  // 第一步：获取当前用户的好友ID列表
  User.findById(req.user._id, { friends: 1 })
    .then(user => {
      // 生成所有好友查询的Promise数组
      const friendQueryPromises = user.friends.map(friend => User.findById(friend._id));
      // 等待所有Promise完成，把结果传给下一个then
      return Promise.all(friendQueryPromises);
    })
    .then(friendsList => {
      console.log(friendsList);
      res.render('friends', { friendsList });
    })
    .catch(err => {
      console.error('获取好友列表出错:', err);
      res.status(500).send('获取好友列表失败，请稍后重试');
    });
});

为什么原来的代码不行？ #

你之前用map遍历好友ID时，每个User.findById().then()都是独立的异步操作，它们会在后台并行执行，但主线程不会等待这些操作完成，直接就跳到console.log(friendsList)了——这时候所有的push操作都还没执行，数组自然是空的。

而Promise.all的作用就是接收一个Promise数组，等待所有Promise都成功resolve后，才会返回一个包含所有结果的数组，这样就能保证你拿到完整的好友数据了。

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