javascript - How can Mongoose "delete" non-existing documents? - Stack Overflow

admin管理员组

文章数量:1310542

Code:

function deleteItem(req, res) {
    Goods.findByIdAndRemove(req.params.id, (err) => {
    if (err) {
      res.send({
        success: false,
        error: err
      });
    } else {
      res.send({
        success: true,
        item: req.params.id
      });
    }
  })
}

If I pass an _id of just deleted document - Mongoose successfully "deletes" it.
If I pass an _id of never existed document, like 591dad9a1583ea0d1065d633 - it also "deletes" it.

Error throws only if pass trash like a34pnv530eargdzbs.

Could somebody tell me, what's going on, please ? :)

Code:

function deleteItem(req, res) {
    Goods.findByIdAndRemove(req.params.id, (err) => {
    if (err) {
      res.send({
        success: false,
        error: err
      });
    } else {
      res.send({
        success: true,
        item: req.params.id
      });
    }
  })
}

If I pass an _id of just deleted document - Mongoose successfully "deletes" it.
If I pass an _id of never existed document, like 591dad9a1583ea0d1065d633 - it also "deletes" it.

Error throws only if pass trash like a34pnv530eargdzbs.

Could somebody tell me, what's going on, please ? :)

• javascript
• node.js
• mongodb
• mongoose

Share Improve this question Follow asked May 18, 2017 at 15:14 SevaSeva 68522 gold badges77 silver badges1818 bronze badges

Add a ment  | 

1 Answer 1

Sorted by: Reset to default 12

If you check the related Mongoose documentation you will find the reason behind it:

Finds a matching document, removes it, passing the found document (if any) to the callback. http://mongoosejs./docs/api.html#model_Model.findByIdAndRemove

If the document doesn't exist in your database Mongoose wont throw an error. You should check the 2nd parameter of the callback:

Goods.findByIdAndRemove(req.params.id, function(err, doc) {
  if(err || !doc) {
    // Show an error page
  }
});

本文标签： javascriptHow can Mongoose quotdeletequot nonexisting documentsStack Overflow