NodeJS删除MongoDB子项后组内列表未更新，filter失效求助

我有一个包含子项列表的组，尝试删除某个子项时，子项本身已成功删除，但该子项并未从组内的列表中移除。数据存储于MongoDB，代码基于NodeJS开发。最初我尝试使用getByIdAndDelete()方法删除子项，但发现删除后Node无法找到该子项，且数组filter功能似乎未生效。请问我哪里出错了？

相关代码 #

router.delete("/delete-child/:id", (req, res) => {
  const id = req.params.id;
  Child.findOne({ _id: id })
    .then((result) => {
      Group.findOne({ _id: result.group }).then((resp) => {
        resp.childrenList = resp.childrenList.filter(
          (childId) => childId !== id
        );
        resp.save().then((savedResp) => {
          console.log("deleteChild", savedResp);
        });
      });
      res.send(result);
    })
    .catch((err) => {
      console.log(err);
    });
  Child.findByIdAndDelete(id).then((deletedChild) => {
    console.log("deletedChild", deletedChild);
  });
});

1. 异步执行顺序混乱 #

你代码里Child.findByIdAndDelete(id)和Child.findOne()是并行执行的，很大概率会先把子项删除，后续再查询子项信息时已经拿不到数据，导致更新Group的逻辑直接失效。

2. ObjectId与字符串类型不匹配 #

MongoDB中存储的_id是ObjectId类型，而req.params.id是字符串格式，直接用childId !== id做比较时，因为类型不一致，过滤条件永远不成立，所以childrenList不会有任何变化。

修正后的代码 #

router.delete("/delete-child/:id", async (req, res) => {
  const id = req.params.id;
  try {
    // 先查询子项信息，获取所属的group ID
    const child = await Child.findOne({ _id: id });
    if (!child) {
      return res.status(404).send("子项不存在");
    }

    // 使用MongoDB的$pull操作符直接移除数组元素，自动处理类型匹配问题
    await Group.updateOne(
      { _id: child.group },
      { $pull: { childrenList: id } }
    );

    // 最后执行子项删除操作
    const deletedChild = await Child.findByIdAndDelete(id);
    console.log("deletedChild", deletedChild);

    res.send(deletedChild);
  } catch (err) {
    console.log(err);
    res.status(500).send("删除失败");
  }
});

额外优化说明 #

• 改用async/await替代嵌套then，代码更清晰易维护，也避免了异步顺序问题
• 使用MongoDB原生的$pull操作数组，比先查询再修改再保存的流程更高效，还能避开类型匹配的坑
• 增加子项不存在的判断，返回404状态码更符合HTTP规范
• 统一错误处理逻辑，返回对应的错误状态码和提示信息

内容的提问来源于stack exchange，提问作者Tamar Laufer