Mongoose中findById()遇无效ID崩溃问题求助

嘿，这个问题我太熟了！你的接口在传入无效ID时崩溃，核心原因是Mongoose的findById方法在接收到格式不合法的ObjectId时，会抛出CastError错误，而你的代码没有任何错误捕获逻辑，导致未处理的Promise拒绝直接让应用挂掉。下面给你几个实用的解决方案：

方案1：用try/catch捕获异步错误 #

这是最直接的方式，把异步查询逻辑包裹在try块里，在catch里精准处理错误：

router.get('/:id', async (req, res) => {
  try {
    const genre = await Genre.findById(req.params.id);
    if(!genre) return genreNotFoundError(res);
    res.send(genre);
  } catch (err) {
    // 专门处理无效ID格式的CastError
    if (err.name === 'CastError') {
      return res.status(400).send('无效的分类ID格式');
    }
    // 其他未知错误统一返回500
    res.status(500).send('服务器内部错误');
  }
});

// 辅助函数
function genreNotFoundError(res) {
  res.status(404).send('未找到该分类');
}

方案2：提前验证ID合法性 #

在发起数据库查询前，先检查ID是否是有效的ObjectId，这样能提前拦截无效请求，减少不必要的数据库操作：

const mongoose = require('mongoose');

router.get('/:id', async (req, res) => {
  // 先验证ID格式是否合法
  if (!mongoose.Types.ObjectId.isValid(req.params.id)) {
    return res.status(400).send('无效的分类ID格式');
  }

  try {
    const genre = await Genre.findById(req.params.id);
    if(!genre) return genreNotFoundError(res);
    res.send(genre);
  } catch (err) {
    res.status(500).send('服务器内部错误');
  }
});

// 辅助函数
function genreNotFoundError(res) {
  res.status(404).send('未找到该分类');
}

额外优化：全局错误处理中间件 #

为了避免其他接口也出现类似崩溃问题，建议在你的Express应用中添加全局错误处理中间件，兜底处理所有未捕获的错误：

// 注意要放在所有路由定义之后
app.use((err, req, res, next) => {
  console.error(err.stack); // 打印错误日志方便后续排查
  res.status(500).send('服务器内部错误');
});

这样即使某个接口遗漏了错误捕获，也不会导致整个应用崩溃，而是返回友好的错误响应。

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