JPA中MedicalNote复合主键实现及H2测试删除失败咨询

一、先明确你的实现核心问题 #

复合主键包含关联实体（Doctor/Patient）+日期的设计本身是可行的，但你的映射大概率存在以下问题：

1. 复合主键未正确使用可嵌入类（@Embeddable）+@MapsId关联，导致Hibernate无法解析关联关系与查询条件；
2. 双向一对多的缓存/关联维护不一致，删除后缓存未刷新或一方集合未同步；
3. 删除操作未在事务中执行，或未处理Hibernate一级缓存的影响。

二、正确的实体映射实现 #

1. 定义可嵌入的复合主键类 #

复合主键必须实现Serializable，且正确重写equals和hashCode：

@Embeddable
public class MedicalNoteId implements Serializable {
    private Long doctorId;
    private Long patientId;
    private LocalDate date;

    // 必须提供无参构造器
    public MedicalNoteId() {}

    public MedicalNoteId(Long doctorId, Long patientId, LocalDate date) {
        this.doctorId = doctorId;
        this.patientId = patientId;
        this.date = date;
    }

    // 基于三个字段实现equals和hashCode
    @Override
    public boolean equals(Object o) {
        if (this == o) return true;
        if (o == null || getClass() != o.getClass()) return false;
        MedicalNoteId that = (MedicalNoteId) o;
        return Objects.equals(doctorId, that.doctorId) &&
               Objects.equals(patientId, that.patientId) &&
               Objects.equals(date, that.date);
    }

    @Override
    public int hashCode() {
        return Objects.hash(doctorId, patientId, date);
    }

    // 省略getter/setter
}

2. MedicalNote实体映射 #

用@EmbeddedId定义复合主键，通过@MapsId关联Doctor/Patient实体到主键中的对应字段：

@Entity
public class MedicalNote {
    @EmbeddedId
    private MedicalNoteId id;

    @ManyToOne(fetch = FetchType.LAZY)
    @MapsId("doctorId") // 将doctor关联到主键的doctorId字段
    private Doctor doctor;

    @ManyToOne(fetch = FetchType.LAZY)
    @MapsId("patientId") // 将patient关联到主键的patientId字段
    private Patient patient;

    // 其他业务字段
    private String content;

    // 构造器（维护双向关联）
    public MedicalNote(Doctor doctor, Patient patient, LocalDate date, String content) {
        this.id = new MedicalNoteId(doctor.getId(), patient.getId(), date);
        this.doctor = doctor;
        this.patient = patient;
        this.content = content;
        // 同步更新一方的集合
        doctor.getMedicalNotes().add(this);
        patient.getMedicalNotes().add(this);
    }

    // 省略无参构造器、getter/setter
}

3. Doctor实体的双向关联配置 #

添加orphanRemoval = true，确保从集合移除时自动删除数据库记录：

@Entity
public class Doctor {
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;

    private String name;

    @OneToMany(mappedBy = "doctor", cascade = CascadeType.ALL, orphanRemoval = true)
    private List<MedicalNote> medicalNotes = new ArrayList<>();

    // 推荐添加关联维护方法，避免手动操作集合出错
    public void addMedicalNote(MedicalNote note) {
        medicalNotes.add(note);
        note.setDoctor(this);
        note.getId().setDoctorId(this.getId());
    }

    public void removeMedicalNote(MedicalNote note) {
        medicalNotes.remove(note);
        note.setDoctor(null);
    }

    // 省略getter/setter
}

Patient实体的关联配置与Doctor完全一致。

三、修正删除操作与测试逻辑 #

1. Spring Data JPA Repository方法 #

确保Repository方法定义正确：

public interface MedicalNoteRepository extends JpaRepository<MedicalNote, MedicalNoteId> {
    void deleteByDoctorId(Long doctorId);
}

2. 测试时处理缓存与事务 #

Hibernate一级缓存会保留已加载的实体，删除后必须刷新会话并清空缓存，才能从数据库获取最新状态：

@Test
@Transactional
void testDeleteByDoctorId() {
    // 1. 准备测试数据
    Doctor doctor = doctorRepository.save(new Doctor("Dr. Smith"));
    Patient patient = patientRepository.save(new Patient("John Doe"));
    MedicalNote note = new MedicalNote(doctor, patient, LocalDate.now(), "Test content");
    medicalNoteRepository.save(note);

    // 2. 执行删除
    medicalNoteRepository.deleteByDoctorId(doctor.getId());

    // 3. 刷新会话+清空一级缓存，避免读取旧数据
    entityManager.flush();
    entityManager.clear();

    // 4. 验证结果
    List<MedicalNote> remainingNotes = medicalNoteRepository.findByDoctorId(doctor.getId());
    assertEquals(0, remainingNotes.size());
}

四、常见问题排查 #

• 缓存问题：删除后未调用entityManager.clear()，导致查询返回缓存中的旧实体；
• 事务缺失：删除操作未在@Transactional注解的方法中执行，导致修改未提交到数据库；
• 关联维护错误：删除时未同步更新Doctor的medicalNotes集合，导致双向关联不一致（但数据库记录会被删除，仅缓存有残留）；
• 复合主键实现错误：未正确重写equals/hashCode，导致Hibernate无法识别实体唯一性，查询/删除逻辑失效。

内容的提问来源于stack exchange，提问作者Mostafa M. Galal