MEAN栈中findByIdAndUpdate未更新文档反而新增空ID文档问题

问题现象 #

执行患者信息更新操作后，数据库中新增了一条包含更新内容但ID为空的文档，原本需要更新的目标文档无任何变化。

问题根源 #

1. 参数获取不匹配：前端通过URL查询参数（?userId=id）传递用户ID，但后端错误地从请求体（req.body.userId）中读取ID，导致userId值为undefined。
2. Upsert配置触发新增：后端使用findByIdAndUpdate时设置了upsert: true，当查询条件（undefined）无法匹配任何现有文档时，会自动插入一条新文档，这就是数据库中出现空ID新文档的原因。

修复步骤及代码示例 #

1. 修正后端参数获取逻辑 #

将后端获取用户ID的方式改为从URL查询参数中读取，同时添加参数校验避免无效请求：

router.put('/update', function(req, res, next) {
  // 从URL查询参数中获取用户ID
  const userId = req.query.userId;

  // 校验用户ID是否存在
  if (!userId) {
    return res.send({status:400, message: '用户ID不能为空'});
  }

  const { firstName, lastName, username, email, birthDate, gender } = req.body;

  const patientObj = {
    firstName,
    lastName,
    username,
    email,
    birthDate,
    gender
  };

  // 移除upsert: true，仅更新已有文档；若业务需要自动创建，可保留但需确认逻辑合理性
  patientsModel.findByIdAndUpdate(userId, patientObj, { new: true }, function(err, patientResponse) {
    if (err) {
      return res.send({status:500, message: '无法更新患者信息'});
    }
    if (!patientResponse) {
      return res.send({status:404, message: '未找到目标患者'});
    }
    res.send({status:200, message: '患者信息更新成功', results: patientResponse});
  });
});

2. 前端代码确认（无需修改） #

前端的参数传递逻辑正确，确保用户ID通过URL查询参数发送即可：

editPatient(id:string,patient: Patient){
    const headers = { 'content-type': 'application/json'}  
    const body=patient;
    console.log(body)
     
    let url=environment.PATIENT_BASE_URL+environment.PATIENT.UPDATE_PATIENT + "?userId=" +id;
    return this.httpClient.put(url, body);
}

额外说明 #

• 若业务场景不需要自动创建新患者，务必移除upsert: true配置，避免意外生成无效文档。
• 建议在后端添加更多参数校验（如必填字段检查），提升接口健壮性。

内容的提问来源于stack exchange，提问作者Farida Yasser