MongoDB通过ObjectId调用deleteOne删除Schema数据失败排查

问题描述 #

我尝试在MongoDB中使用deleteOne方法删除数据，但该功能似乎并未生效。我的代码如下：

const deleteGroup = async (req, res) => {
  const { groupId } = req.body
  try {
    console.log(req.body)
    Group.deleteOne({_id: new ObjectId(groupId) })

    res.send(`${groupId} deleted`);
    
  } catch (err) {
    console.log(err.message);
    res.status(400).json({ err: err.message });
  }
}

console.log输出的groupId为我想要删除的正确ID：6480898f2012404b9adbb71e。我是否错误使用了deleteOne？Group对象对应的Schema定义如下：

const mongoose = require('mongoose');
const { Schema } = mongoose;

const groupSchema = new Schema({
  group_name: {
    type: String,
    require: true,
    unique: true,
  },
  group_description: {
    type: String,
  },
  group_admin: {
    type: String,
    require: true
  },
  group_notes: {
    type: String,
  },
  group_public: {
    type: Boolean,
    require: true
  },
});

module.exports = mongoose.model('Group', groupSchema);

问题原因与修复方案 #

• 核心问题：未等待异步操作完成
  Mongoose的deleteOne是异步方法，返回Promise，但你没有用await等待它执行完成，代码直接跳过删除操作就返回了响应，数据库还没来得及处理删除请求。

• 修复步骤：

  1. 在Group.deleteOne(...)前添加await关键字，确保等待删除操作完成后再返回响应。
  2. 可以获取删除操作的结果，判断是否真的删除了文档，避免返回错误的成功提示。

修复后的代码：

const deleteGroup = async (req, res) => {
  const { groupId } = req.body
  try {
    console.log(req.body)
    // 添加await等待异步操作完成，同时获取结果
    const result = await Group.deleteOne({_id: new ObjectId(groupId) })
    
    // 判断是否成功删除文档
    if (result.deletedCount === 0) {
      return res.status(404).json({ err: '未找到对应文档' });
    }

    res.send(`${groupId} deleted`);
    
  } catch (err) {
    console.log(err.message);
    res.status(400).json({ err: err.message });
  }
}

• 额外提示：
  • 确保ObjectId已正确导入，比如从mongoose中引入：const { ObjectId } = require('mongoose').Types;，否则可能会报错。
  • Schema中的require拼写错误，应该是required，虽然这不是删除失败的直接原因，但会导致数据校验逻辑异常，建议修正。

内容的提问来源于stack exchange，提问作者tan shi yu