求助：Mongoose的findByIdAndRemove为何触发CastError错误？

嘿，我一眼就发现问题出在哪了！

你看错误信息里的value字段："5a6a8f967c9a190507b03433 "——注意到ID字符串末尾的那个空格了吗？Mongoose要求传入的_id必须是合法的ObjectId格式，而带空格的字符串显然不符合要求，所以触发了CastError，提示无法把这个值转换成ObjectId。

快速修复方案 #

只需要在使用req.params.id之前，用trim()方法去掉字符串前后的空格就可以了：

//DELETE Route
app.delete("/blogs/:id", function(req, res){
  // 清理ID前后的空格
  const blogId = req.params.id.trim();
  //Destroy blog
  Blog.findByIdAndRemove(blogId, function(err){
    if(err){
      res.send(err);
    } else {
      res.redirect("/blogs");
    }
  });
});

额外优化建议 #

为了避免以后再遇到类似问题，可以做这两个优化：

• 加一个参数验证中间件，提前检查并清理ID参数，确保它是合法的ObjectId格式
• 执行删除前，先用Mongoose的Types.ObjectId.isValid()方法验证ID的合法性，比如：

  const { Types } = require('mongoose');
  // ...
  const blogId = req.params.id.trim();
  if(!Types.ObjectId.isValid(blogId)){
    return res.status(400).send("Invalid blog ID");
  }
  

这样就能从根源上避免这类类型转换错误啦！

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