MongoDB $pull无法从嵌套数组删除元素问题排查

这个问题的核心原因是MongoDB的类型匹配规则——你传入的_id是字符串类型，但数据库中pons数组里存储的是ObjectId对象，两者类型不匹配导致$pull找不到要删除的元素，所以操作没生效但也不会抛出错误。

为什么你的两种写法都无效？ #

• 第一种写法{$pull: {pons: {_id}}}：你试图匹配数组中包含_id字段的对象，但你的pons数组直接存储的是ObjectId值（不是嵌套对象），所以完全匹配不到目标元素。
• 第二种写法{$pull: {pons: _id}}：这里的_id是从req.params获取的字符串，而数据库里的元素是ObjectId类型，MongoDB不会自动转换类型进行匹配，因此也找不到对应元素。

修复步骤 #

1. 导入MongoDB的ObjectId工具（如果你使用Mongoose）：
   在文件顶部添加：

   const mongoose = require('mongoose');
   

2. 将字符串_id转换为ObjectId类型：
   在路由处理逻辑中，把从req.params获取的_id转换成ObjectId，再用于$pull操作：

   const { _id } = req.params;
   // 转换为ObjectId类型
   const targetPonId = mongoose.Types.ObjectId(_id);
   

3. 修改findOneAndUpdate的查询条件：
   直接用转换后的ObjectId匹配数组元素：

   User.findOneAndUpdate(
     { _id: req.user._id },
     { $pull: { pons: targetPonId } }
   ).catch((e) => {
     console.log(1, e);
     errors.push('Something went wrong. Pon reference was not deleted from user');
   })
   

完整修改后的路由代码 #

const mongoose = require('mongoose'); // 确保已引入

router.delete('/:_id', async (req, res) => {
  const { _id } = req.params;
  const errors = [];
  const targetPonId = mongoose.Types.ObjectId(_id); // 转换类型
  console.log(_id, targetPonId); // 可以打印对比类型

  await Promise.all([
    User.findOneAndUpdate(
      { _id: req.user._id },
      { $pull: { pons: targetPonId } }
    ).catch((e) => {
      console.log(1, e);
      errors.push('Something went wrong. Pon reference was not deleted from user');
    }),
  ]);

  if (errors.length > 0) {
    console.log(2, errors);
    res.json({ ok: false, errors });
  } else {
    res.json({ ok: true });
  }
});

额外验证建议 #

你可以在代码中打印req.user.pons里元素的类型，以及_id的类型，确认差异：

// 假设你能获取到当前用户的文档
const user = await User.findById(req.user._id);
console.log(typeof user.pons[0], user.pons[0]); // 输出应为object（ObjectId实例）
console.log(typeof _id, _id); // 输出应为string

内容的提问来源于stack exchange，提问作者Alex Ironside