如何判断对象是否已创建并解决Safe模型跳转时的ObjectId转换报错

问题原因 #

• 你调用的是Safe.find({})方法，该方法返回值永远是数组，哪怕没有匹配数据也会返回空数组[]，永远不会等于undefined，所以判断逻辑if(safe===undefined)永远不会触发。
• 数组本身没有_id属性，所以safe._id取到的值为undefined，重定向路径就变成了mysafe/undefined，后续路由拿到这个undefined尝试转换为ObjectId时就触发了类型转换报错。
• 如果你的业务逻辑是每个用户对应一个Safe对象，用find()也不符合需求，哪怕有匹配数据你拿到的也是包含所有Safe对象的数组，不能直接取_id属性。

修复方案 #

根据业务场景调整查询和判断逻辑即可：
如果是要查询当前登录用户对应的单个Safe对象，用findOne()替代find()，示例代码如下：

app.get('/my',isLoggedIn,async (req,res)=> {
    // 按需补充用户过滤条件，比如查询当前登录用户的Safe：{ userId: req.user.id }
    // findOne返回单个对象，无匹配数据时返回null
    const safe = await Safe.findOne({ userId: req.user.id });
    // 不存在时跳转到创建页
    if(!safe){
        return res.redirect('/mysafe')
    }
    // 存在时跳转到详情页，路径开头加/避免相对路径拼接错误
    res.redirect(`/mysafe/${safe._id}`)
})

如果确实需要查询全量Safe，取数组首个元素的_id即可：

app.get('/my',isLoggedIn,async (req,res)=> {
    const safeList = await Safe.find({});
    if(safeList.length === 0){
        return res.redirect('/mysafe')
    }
    res.redirect(`/mysafe/${safeList[0]._id}`)
})

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