HTTP DELETE 的 RESTful API 不检查 null

Question

我目前正在为 Web 服务编写 RESTful API，但遇到了麻烦。我试图删除一封邮件，但首先我想检查邮件是否存在。我的问题是它不检查邮件是否为空并且不以 404 响应。我正在使用 express 和 mongoose

router.delete('/:id', (req, res) => {
    const { id } = req.params;
    Mail.findById(id)
      .exec()
      .then((mail) => {
        if (!mail) {
          console.log(mail) // returns null
          return res.status(404);
        }
      })
      .then(
        Mail.deleteOne({ _id: id })
          .exec()
          .then(() => {
            res.status(200).json({
              message: 'Mail deleted',
            });
          })
          .catch((err) => {
            res.status(500).json({ error: err });
          })
      );
  });

Answer 1

我认为您必须在第一个 then 块中将代码的删除部分作为 else 语句。您没有返回下一个 then 块可以使用的任何内容。

你可以这样做：

Mail.findById(id)
      .exec()
      .then((mail) => {
        if (!mail) {
          console.log(mail) // returns null
          return res.status(404).send() //need to send response;
        }
        Mail.deleteOne({ _id: id })
          .exec()
          .then(() => {
            res.status(200).json({
              message: 'Mail deleted',
            });
          })
      }).catch((err) => {
            res.status(500).json({ error: err });
      })

专业提示：如果您不知道，请学习异步等待。代码看起来会更干净！

然后它看起来像这样：

router.delete('/:id', async (req, res) => {
    const { id } = req.params;

    try {
      const mail = await Mail.findById(id);
      if(!mail) {
         return res.status(404).send();
      }

      await Mail.deleteOne({_id: id});      
      res.status(200).json({
              message: 'Mail deleted',
            });
    } catch(e) {
      res.status(500).json({ error: err });
    }