如何正确处理猫鼬CRUD上的错误答案

【问题标题】：How to properly handle error on mongoose CRUD如何正确处理猫鼬CRUD上的错误
【发布时间】：2018-06-01 18:48:29
【问题描述】：

我正在构建一个非常简单的 RESTFUL 应用，以了解 Node 和 Mongo 如何协同工作。

A 做了一个函数来通过 Id 在 mongodb 上查找“Persona”文档，所以这里是代码：

function getPersonabyId(req,res)
{
    //en params del objeto request esta lo que llega por get
    let idPersona=req.params.id;

    //mongoose trae un método para pillar por id
    Persona.findById(idPersona).exec((err,objPersona)=>
    {

        if(err)
        {
           res.status(500).send(err);
        }
        else{


            res.status(200).send(objPersona);
        }

    })


}

我使用这条路线，执行函数：

miniApp.get("/getPersonaById/:id",controladorPersona.getPersonabyId);

因此，当我将有效 ID 作为参数传递时，一切正常，我在响应中得到了正确的 Persona 对象。

但是当我使用无效的 ID 时（在 mongodb 上不存在具有该 ID 的文档），在 exec 回调中指定的错误被抛出......但是，不应该只抛出这个错误以防万一有服务器问题？不存在的 ID 不应该是 500 错误，不是吗？

我查找了信息，发现了这个：

https://coursework.vschool.io/mongoose-crud/

Person.find((err, people) => {  
    // Note that this error doesn't mean nothing was found,
    // it means the database had an error while searching, hence the 500 status
    if (err) return res.status(500).send(err)
    // send the list of all people
    return res.status(200).send(people);
});

阅读上面代码中的注释行，让我更加困惑......正如那些 cmets 中所说的那样，错误应该是错误数据库或类似的东西，而不是“未找到”错误......但是当找不到具有该 ID 的对象时，实际上会引发错误！

这是我使用无效 ID 时遇到的错误：

{
    "message": "Cast to ObjectId failed for value \"5b105ba453401c41d0e3da2\" at path \"_id\" for model \"Persona\"",
    "name": "CastError",
    "stringValue": "\"5b105ba453401c41d0e3da2\"",
    "kind": "ObjectId",
    "value": "5b105ba453401c41d0e3da2",
    "path": "_id"
}

【问题讨论】：

标签： node.js mongodb mongoose crud

【解决方案1】：

当您使用 mongoose 时，您可以使用 mongoose.Types.ObjectId.isValid(req.params.id) 验证传入值是否为 objectId。这将返回一个布尔值，可能会为您节省一些精力或编写用于对象 ID 验证的正则表达式。

【讨论】：

谢谢！很有用！

【解决方案2】：

您已在架构中将_id 定义为ObjectId。并非每个字符串都是有效的ObjectId。您传递给findById 的ID 无效。对象 ID 应为 24 个字符长的字符串。查询前添加此项检查，

if (id.match(/^[0-9a-fA-F]{24}$/)) {
  // Yes, it's a valid ObjectId, proceed with `findById` call.
} else{
  //throw an error
}

或者如果您想继续使用任何字符串作为_id，请在架构中将_id 定义为String 而不是ObjectId。

【讨论】：