Mongoose一对多关联开发时无法推送子集合ID到父集合关联字段问题

问题根因 #

你的代码存在3个核心问题导致更新父集合字段失败：

• 响应返回时机错误：在执行Course.findByIdAndUpdate之前就调用了res.send，Node.js异步机制下，请求响应结束后后续的数据库操作大概率会被中断，即使执行成功你也无法捕获异常。
• 异步操作未处理：findByIdAndUpdate是异步方法，你既没有加await等待执行完成，也没有通过then/catch链式调用监听结果，属于“发出去就不管”的状态，执行成功/失败都无法感知。
• 未校验courseId有效性：代码中没有展示courseId的来源，如果该值为空、格式不符合MongoDB ObjectId规范，或者对应Course文档不存在，更新操作会直接失败。

修复后的代码 #

推荐用async/await写法，逻辑更清晰，也方便捕获异常：

// 路由处理函数需要加async关键字
router.post('/lessons', async (req, res) => {
  try {
    // 1. 取courseId，根据你的接口定义调整取值位置，比如是req.params.courseId还是req.body.courseId
    const courseId = req.body.courseId
    if (!courseId) {
      return res.status(400).send({
        status: 400,
        message: "缺少关联课程ID参数courseId"
      })
    }
    // 2. 先创建子集合课时数据
    const newLesson = await Lessons.create(req.body)
    // 3. 等待父集合课程关联字段更新完成
    await Course.findByIdAndUpdate(
      courseId,
      { $push: { lessons: newLesson._id } },
      { new: true }
    )
    // 4. 所有操作都成功后再返回响应
    return res.status(200).send({
      status: 200,
      message: "课时创建成功",
      data: newLesson
    })
  } catch (err) {
    // 打印错误日志方便排查问题
    console.error("操作失败：", err)
    return res.status(500).send({
      status: 500,
      message: "操作失败",
      error: err.message
    })
  }
})

额外注意事项 #

• 不要加upsert: true配置：该配置会在courseId不存在时自动新建一条空的Course文档，反而会产生脏数据，你的场景只需要更新已存在的课程即可。
• 可以在findByIdAndUpdate后加.exec()方法，返回标准Promise对象，写法更规范，比如：await Course.findByIdAndUpdate(...).exec()
• 排查问题时可以先打印courseId和newLesson._id的值，确认格式正确，且对应Course文档确实存在。

内容的提问来源于stack exchange，提问作者Sultan Achmad