Question

我正在构建一个非常简单的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);
});

阅读上面代码中的注释行，让我更加困惑......正如那些评论中所说的，错误应该是错误数据库或类似的东西，而不是“未找到”错误...但是当找不到具有该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"
}

Answer 1

您已在架构中将_id定义为ObjectId。并非每个字符串都是有效的ObjectId。您传递给findById的身份证无效。对象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。

Answer 2

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

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

2 个答案: