访问/post接口触发Mongoose的ObjectId转换错误该如何解决？

• 核心是Express路由匹配顺序问题：Express会按照路由注册的先后顺序从上到下匹配路径，只要匹配到符合规则的路由就会停止后续匹配。
• 你项目中存在一个优先级更高的一级动态参数路由（大概率是用户相关的GET /:id路由，且用户路由的注册顺序在帖子路由之前），请求GET /post的时候，先命中了这个动态路由，把路径中的post当成了参数id的值，代码里拿着这个字符串post去USER模型中查询_id字段，而_id是ObjectId类型，字符串post无法转换为合法的ObjectId，就抛出了你看到的报错。
• 你把路由改成/post/allpost之后就正常，是因为这个路径是两级结构，不会匹配到一级的动态路由/:id，所以能正确命中你写的查询全部帖子的逻辑。

你可以任选以下一种方案修复：

• 方案1：调整路由注册顺序
  把所有静态路由全部放到动态参数路由前面，比如先注册帖子路由app.use('/', postRouter)，再注册带一级动态参数的用户路由app.use('/', userRouter)，这样请求GET /post会先匹配到帖子的静态路由，不会走到用户的动态路由里。

• 方案2：给路由统一加前缀（更推荐，从根源避免跨模块路由冲突）
  给不同模块的路由分配独立的前缀，比如用户相关路由统一加/user前缀，帖子相关路由统一加/post前缀：

// app.js 路由注册处
app.use('/user', require('./routes/userRouter'))
app.use('/post', require('./routes/postRouter'))

对应修改帖子路由里的路径即可，不会再和其他模块路由冲突：

// 原查询全部帖子路由 /post 改成 /，访问路径为 /post
router.get('/',authenticate,async (req,res)=>{
  // 原有逻辑不变
})
// 原单帖查询路由 /post/:id 改成 /:id，访问路径为 /post/:id
router.get('/:id',async (req,res)=>{
  // 原有逻辑不变
})

• 方案3：给动态参数加格式校验
  如果不想改动现有路由结构，可以给动态参数路由加正则限制，只有参数是合法ObjectId格式时才命中路由：

// 原一级动态路由 /:id 调整为
router.get('/:id([0-9a-fA-F]{24})', ...)

额外提醒：你写的updatepost路由存在语法错误，Post.findByIdAndUpdate(req.body.postId),{这里多了一个右括号，会导致更新逻辑不生效，修正写法如下：

router.put('/updatepost',authenticate,(req,res)=>{
    Post.findByIdAndUpdate(req.body.postId,{
        $set:req.body,
    },{
        new:true
    }).exec((err,result)=>{
        if(err){
            return res.status(422).json({error:err})
        }else{
            res.json(result)
        }
    })
})

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