JavaScript/Express/MongoDB入门：任务管理器优先级渲染报错求助

嘿Daniel，我一眼就看出问题所在啦！你现在的代码只从数据库里查询了high优先级的任务，但模板里尝试访问tasks_low（大概率还有tasks_mid），这些变量根本没在渲染页面的时候传进去，自然会报未定义的错误。而且MongoDB的查询是异步的，直接在第一个查询的回调里渲染页面，会导致其他优先级的任务还没查完就输出页面，这也是问题的核心。

下面给你两种清晰的解决方案，优先推荐第一种，代码更易读维护：

方案1：使用async/await（推荐） #

把路由处理函数改成异步函数，用Promise.all并行查询三个优先级的任务，等所有数据都准备好后再渲染页面：

app.get('/', async function(req, res) {
  try {
    // 同时发起三个优先级的查询，等待所有结果返回
    const [tasks_high, tasks_mid, tasks_low] = await Promise.all([
      db.tasks.find({ prio: 'high' }).toArray(),
      db.tasks.find({ prio: 'mid' }).toArray(),
      db.tasks.find({ prio: 'low' }).toArray()
    ]);

    // 把所有任务数据传给模板，键名要和模板里用的一致
    res.render('index', {
      title: '任务管理器',
      tasks_high,
      tasks_mid,
      tasks_low
    });
  } catch (err) {
    // 捕获并处理查询错误，避免服务器崩溃
    console.error('查询任务列表出错:', err);
    res.status(500).send('服务器内部错误，请稍后再试');
  }
});

为什么这么做？ #

• Promise.all让三个查询并行执行，比串行查询效率更高
• await会等待所有查询完成后再继续，确保所有任务数据都准备好了才渲染页面
• toArray()把MongoDB的游标转换成数组，这样模板里可以直接遍历渲染
• 用try/catch统一处理错误，代码更健壮

方案2：使用Promise链式调用（兼容旧环境） #

如果你的环境还不支持async/await，可以用Promise的链式写法，原理和上面一样：

app.get('/', function(req, res) {
  Promise.all([
    db.tasks.find({ prio: 'high' }).toArray(),
    db.tasks.find({ prio: 'mid' }).toArray(),
    db.tasks.find({ prio: 'low' }).toArray()
  ])
  .then(([tasks_high, tasks_mid, tasks_low]) => {
    res.render('index', {
      title: '任务管理器',
      tasks_high,
      tasks_mid,
      tasks_low
    });
  })
  .catch(err => {
    console.error('查询任务列表出错:', err);
    res.status(500).send('服务器内部错误，请稍后再试');
  });
});

最后要注意的小细节 #

• 确保模板里用的变量名和你渲染时传的完全一致，比如模板里写的是tasks_low，渲染时就不能写成lows或者其他名字
• 如果你的MongoDB驱动版本比较老，find()可能不返回Promise，这时候需要手动把它包装成Promise，或者用回调的方式，但还是推荐升级驱动到支持Promise的版本

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