在 Spring Boot 应用中使用 JpaRepository.save() 更新实体时，若传入对象未正确关联数据库已有主键（如 @Id 字段为空或为 0），JPA 会将其视为新实体而执行 INSERT 而非 UPDATE，导致数据重复而非更新。
在 spring boot 应用中使用 `jparepository.save()` 更新实体时，若传入对象未正确关联数据库已有主键（如 `@id` 字段为空或为 0），jpa 会将其视为新实体而执行 insert 而非 update，导致数据重复而非更新。
Spring Data JPA 的 save(S entity) 方法遵循“存在即更新、不存在即插入”的语义（即 upsert 行为），其判断依据是实体是否具有有效的、非空的主键值，且该主键在数据库中已存在。但在你的代码中，问题根源在于：前端表单提交的 Patient 对象虽携带了 patientId，但控制器未确保该 ID 被正确绑定并传递至服务层；更关键的是，save() 调用前未验证该 ID 是否真实存在于数据库中，也未显式加载已有实体进行字段覆盖。
✅ 正确的更新流程（推荐）
应采用“查—改—存”三步法，确保操作原子性与安全性：
- 根据 patientId 查询已有实体（使用 findById()）；
- 将表单提交的新数据逐字段复制到已加载的托管实体上（避免直接 save() 原始 DTO）；
- 调用 save() 持久化已修改的托管实体（此时 JPA 知道这是更新操作）。
以下是重构后的关键代码示例：
? 修改 Controller（区分创建与更新）
// 使用 @PutMapping 明确语义：更新操作
@PutMapping("/save")
public ModelAndView updatePatient(@ModelAttribute("patient") Patient patient) {
// 确保 patient.patientId 不为 null，否则抛出异常或重定向错误页
if (patient.getPatientId() == null) {
throw new IllegalArgumentException("Patient ID is required for update.");
}
patientService.updatePatient(patient);
return new ModelAndView("patient/doneSuccessfully");
}
// 创建仍用 POST（保持语义清晰）
@PostMapping("/create")
public ModelAndView createPatient(@ModelAttribute("patient") Patient patient) {
patientService.savePatient(patient); // 原 savePatient 保留用于新增
return new ModelAndView("patient/doneSuccessfully");
}
?️ 更新 Service 实现
public void updatePatient(Patient updatedPatient) {
Long id = updatedPatient.getPatientId();
Patient existingPatient = patientRepository.findById(id)
.orElseThrow(() -> new EntityNotFoundException("Patient not found with id: " + id));
// 安全字段赋值（避免空值覆盖）
existingPatient.setFileNo(updatedPatient.getFileNo());
existingPatient.setPatientName(Optional.ofNullable(updatedPatient.getPatientName()).orElse(existingPatient.getPatientName()));
existingPatient.setPatientSurname(Optional.ofNullable(updatedPatient.getPatientSurname()).orElse(existingPatient.getPatientSurname()));
existingPatient.setPatientDisease(Optional.ofNullable(updatedPatient.getPatientDisease()).orElse(existingPatient.getPatientDisease()));
existingPatient.setDiagnosisInfo(Optional.ofNullable(updatedPatient.getDiagnosisInfo()).orElse(existingPatient.getDiagnosisInfo()));
existingPatient.setPatientSecurityNumber(updatedPatient.getPatientSecurityNumber());
// dateAdded 通常不应被更新，可忽略或设为当前时间（视业务而定）
patientRepository.save(existingPatient); // 此时 save() 执行 UPDATE
}
⚠️ 注意事项与最佳实践
- 前端表单必须包含隐藏域 ，确保 @ModelAttribute 能正确绑定主键；
- 禁止在更新时重置 @Id 字段为 null 或 0（Lombok 的 @Setter 可能隐式触发，建议在 DTO 层做校验）；
- 优先使用 @PutMapping（全量更新）或 @PatchMapping（局部更新），符合 RESTful 规范，便于前端/测试工具识别意图；
- 若需高性能批量更新，可考虑 @Modifying + @Query 自定义 JPQL，但需注意事务与缓存一致性；
- 启用 spring.jpa.show-sql=true 和 spring.jpa.properties.hibernate.format_sql=true 辅助调试 SQL 日志，确认执行的是 UPDATE 还是 INSERT。
通过以上改造，即可彻底解决“更新变新增”的问题，确保数据一致性与业务逻辑可靠性。
