на первый взгляд (так как сервисный слой не предоставляется) Вы должны удалить вставляемый = false и обновляемый = false из @ JoinColumn
@JoinColumn(name = "doctor_id", nullable = false, insertable = false, updatable = false)
изменить это на:
@JoinColumn(name = "doctor_id", nullable = false)
Поскольку эти директивы не позволяют jpa вставлять / обновлять столбец DOCTOR_ID
Также я предпочитаю использовать werappers вместо примитивного типа, так как @Id меняет int на Integer как предлагается здесь Использование класса Integer-обёртки или примитива int в отображении гибернации
Также кажется, что вы уже сохранили doctor
(так как он уже назначил идентификатор), вы должны сначала выбрать доктора, чтобы и добавьте в него пациента с обоих концов:
public void assignToDoctor(Doctor doctor) {
doctor.patients.add(this);
this.doctor = doctor;
}
вот полный пример:
public static void main(String[] args) {
SpringApplication.run(DemostackApplication.class, args);
}
@Component
public static class AppRunner implements ApplicationRunner {
@Autowired
MainService mainService;
@Override
public void run(ApplicationArguments args) throws Exception {
Doctor doctor = new Doctor();
doctor.department = "a";
doctor.doctorName = "Covid19 Ninja";
doctor = mainService.saveDoctor(doctor);
Patient patient = new Patient();
patient.patientName = "test";
patient.problem = "test";
patient.assignToDoctor(doctor);
Patient newPatient = mainService.savePatient(patient);
}
}
@Service
public static class MainService {
@Autowired
DoctorRepo doctorRepo;
@Autowired
PatientRepo patientRepo;
@Transactional
public Doctor saveDoctor(Doctor doctor) {
return doctorRepo.save(doctor);
}
@Transactional
public Patient savePatient(Patient patient) {
return patientRepo.save(patient);
}
}
@Entity
@Table(name = "patient")
public static class Patient implements java.io.Serializable {
@Id
@GeneratedValue(strategy = GenerationType.AUTO)
@Column(name = "patient_id")
private Integer patientId;
@Column(name = "patient_name", length = 200)
private String patientName;
@Column(name = "problem", length = 200)
private String problem;
@ManyToOne(fetch = FetchType.LAZY)
@JoinColumn(name = "doctor_id", nullable = false)
private Doctor doctor;
public void assignToDoctor(Doctor doctor) {
doctor.patients.add(this);
this.doctor = doctor;
}
}
@Entity
@Table(name = "doctor")
public static class Doctor implements java.io.Serializable {
@Id
@GeneratedValue(strategy = GenerationType.AUTO)
@Column(name = "doctor_id")
private Integer doctorId;
@Column(name = "doctor_name", length = 200)
private String doctorName;
@Column(name = "department", length = 200)
private String department;
@OneToMany(fetch = FetchType.LAZY, mappedBy = "doctor")
private Set<Patient> patients = new HashSet<Patient>(0);
}
Я не использовал геттер / сеттер, но вы должны:)
РЕДАКТИРОВАТЬ
ваш registerPatient()
лог c должен выглядеть примерно так:
@Transactional
public String registerPatient(Patient patient) {
Integer doctorId= patinet.getDoctor().getId();
//fetch the doctor from database
Doctor doctor = doctorRepository.findById(doctorId).orElseThrow(() -> new RuntimeException("doctor not found"));
//create bidirectional reference between patient and doctor
patient.setDoctor(doctor);
doctor.getPatients().add(patient);
//save patient
patient = patientRepo.save(patient);
return "OK";
}