Async模块报错：Cannot read property 'length' of undefined问题排查

我来帮你拆解这个错误的根源，以及对应的解决办法：

错误原因 #

这个报错的直接触发点是你访问resl.length时，resl（也就是results[0]）是undefined，背后有两个核心问题：

1. Mongoose聚合操作的参数格式错误
   你调用Club.aggregate时，直接传了单个$group对象，但Mongoose要求聚合管道必须是数组格式（每个元素对应一个聚合阶段）。这个错误会导致聚合操作失败，async.parallel会直接触发最终的错误回调，此时results参数会是undefined。

2. 缺少错误处理逻辑
   你的async.parallel最终回调里没有先判断err是否存在，直接去读取results[0]。当前面的任务出错时，results是undefined，自然会触发Cannot read property 'length' of undefined的报错。

解决方法 #

我们分两步修复：

1. 修正聚合操作的参数格式 #

把Club.aggregate的第一个参数改成数组形式：

function(callback){
  Club.aggregate([{ $group: { _id: "$positioninclub" } }], (err, newResult) => {
    callback(err, newResult);
  });
},

2. 增加错误处理逻辑 #

在最终回调里先检查err，如果有错误就先处理（比如返回错误页面或者打印日志），再继续后续逻辑：

完整修正后的代码 #

router.get('/', ensureAuthentication, function(req, res) {
  async.parallel([
    function(callback) {
      Club.find({}, (err, result) => {
        callback(err, result);
      });
    },
    function(callback) {
      // 修正聚合管道为数组格式
      Club.aggregate([{ $group: { _id: "$positioninclub" } }], (err, newResult) => {
        callback(err, newResult);
      });
    },
  ], (err, results) => {
    // 先处理错误
    if (err) {
      console.error('Async任务出错:', err);
      return res.status(500).send('服务器内部错误');
    }

    const resl = results[0];
    const resl1 = results[1];
    console.log(resl);
    
    const dataChunk = [];
    const chunkSize = 3;
    // 额外加一层判断，确保resl是数组
    if (Array.isArray(resl)) {
      for(let i = 0; i < resl.length ; i = i + chunkSize){
        dataChunk.push(resl.slice(i, i+chunkSize));
      }
    }
    console.log(dataChunk);
    
    res.render('dashboard', {title: 'Portal', resl: resl});
  })
});

可选优化：改用Promise风格写法 #

因为Mongoose从v5开始默认支持Promise，async也支持Promise任务，你可以把代码改成更简洁的Promise风格，避免回调嵌套：

router.get('/', ensureAuthentication, async function(req, res) {
  try {
    const [resl, resl1] = await Promise.all([
      Club.find({}).exec(),
      Club.aggregate([{ $group: { _id: "$positioninclub" } }]).exec()
    ]);

    console.log(resl);
    const dataChunk = [];
    const chunkSize = 3;
    if (Array.isArray(resl)) {
      for(let i = 0; i < resl.length ; i = i + chunkSize){
        dataChunk.push(resl.slice(i, i+chunkSize));
      }
    }
    console.log(dataChunk);
    
    res.render('dashboard', {title: 'Portal', resl: resl});
  } catch (err) {
    console.error('请求处理出错:', err);
    res.status(500).send('服务器内部错误');
  }
});

这样代码可读性更高，错误处理也更直观。

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