Node.js回调函数异步问题：API无法返回完整评论数组

你的问题核心是异步操作的执行顺序问题——你在所有获取用户信息的异步请求完成前，就把还没填满的comments返回给客户端了。getUsers返回的是Promise，map里的.then是异步执行的，而res.json(comments)是同步执行的，所以此时数组还没被填充完整。

修复方案 #

1. 把路由处理函数改成async函数，用await控制异步操作顺序
2. 用Promise.all批量等待所有用户信息请求完成
3. 修正GET请求的参数获取方式（GET请求一般用req.query而非req.body）
4. 增加错误处理，避免数据库查询出错导致崩溃

修复后的代码：

router.get("/scomment", async (req, res) => {
  try {
    // GET请求从查询字符串取参数，POST才用req.body
    const { commentId, postId } = req.query;
    let comments = [];

    // 用await替代回调，等待评论文档查询完成
    const doc = await commentModel.findOne({ postId, commentId });
    if (!doc) {
      return res.status(404).json({ message: "未找到对应评论文档" });
    }

    // 把每个评论的用户查询包装成Promise，用Promise.all等待全部完成
    const commentPromises = doc.comments.map(async (comment) => {
      const users = await getUsers(comment.email);
      return {
        users,
        comment: comment.commentText
      };
    });

    // 等待所有Promise完成，拿到完整的评论数组
    comments = await Promise.all(commentPromises);

    // 此时数组已填满，返回给客户端
    res.json(comments);
  } catch (err) {
    // 捕获所有异常，返回明确的错误信息
    res.status(500).json({ message: "服务器出错", error: err.message });
  }
});

// 简化getUsers函数，去掉不必要的setTimeout和冗余变量
const getUsers = async (email) => {
  const info = await userModel.findOne({ email });
  if (!info) {
    throw new Error(`未找到邮箱为${email}的用户`);
  }
  return {
    name: info.name,
    email: info.email,
    profile: info.profile
  };
};

关键修改说明 #

• async/await：让异步代码逻辑更清晰，避免回调嵌套，确保每一步操作完成后再执行下一步
• Promise.all：批量处理异步请求，只有当所有用户信息都查询完成后，才会生成完整的comments数组
• 错误处理：用try/catch捕获数据库查询、用户不存在等所有异常，防止服务器崩溃，同时给客户端返回可识别的错误
• GET参数修正：符合HTTP规范，GET请求参数默认通过URL查询字符串传递，若确实需要用req.body，需确保Express已配置express.json()中间件

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