Express下如何将两类MongoDB查询结果共同传入myaccount.ejs模板

你重复为同一个路径/myaccount注册了两个独立的GET路由，Express会按照路由注册顺序匹配，后定义的路由会完全覆盖前面的路由，导致你编写的File集合查询逻辑根本不会被执行，自然没法同时拿到两个集合的查询结果用于模板渲染。

核心思路是移除重复的路由定义，在唯一的路由入口中并行发起File、User两个集合的查询，待两次查询都返回结果后，将两个结果合并为一个上下文对象，一次性传递给myaccount.ejs模板。

基础兼容实现（匹配你现有回调代码风格） #

直接在原有的单路由逻辑中嵌套两次查询，等两个查询都拿到结果后再触发渲染，同时补上缺失的错误响应逻辑，避免请求挂起：

// 仅保留一个 /myaccount 路由
app.get("/myaccount", (req, res) => {
  if (!req.isAuthenticated()) {
    // 补全未登录的跳转逻辑
    return res.redirect("/login");
  }

  // 先执行File集合查询
  File.find().exec((err, fileData) => {
    if (err) {
      console.log(err);
      return res.status(500).render("error", { message: "文件数据加载失败" });
    }
    // File查询成功后执行User集合查询
    User.find().exec((err, userData) => {
      if (err) {
        console.log(err);
        return res.status(500).render("error", { message: "用户数据加载失败" });
      }
      // 合并两个查询结果，一次性传递给模板
      res.render("myaccount", {
        file: fileData,
        user: userData
      });
    })
  })
})

更简洁的优化实现 #

由于Mongoose的查询实例本身支持Promise特性，可以用async/await配合Promise.all并行发起两个查询，避免回调嵌套，代码可读性更高：

app.get("/myaccount", async (req, res) => {
  if (!req.isAuthenticated()) {
    return res.redirect("/login");
  }

  try {
    // 并行执行两个查询，等待全部结果返回
    const [fileData, userData] = await Promise.all([
      File.find().exec(),
      User.find().exec()
    ])
    // 一次性传递所有页面所需数据
    res.render("myaccount", {
      file: fileData,
      user: userData
    })
  } catch (err) {
    console.log(err);
    res.status(500).render("error", { message: "页面数据加载失败" });
  }
})

• 你原有代码中的File.find()、User.find()是查询集合下的全量数据，实际业务场景中建议按照当前登录用户的ID给查询增加过滤条件（比如File.find({ ownerId: req.user._id })），减少无效数据加载，提升页面性能。
• 原有错误处理逻辑只打印日志没有给客户端返回响应，会导致请求一直挂起直到超时，需要在捕获错误时给客户端返回对应的状态码和错误页面。

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