#29: SetAll instead of AddAll to avoid Duplication
All checks were successful
Quality Check / Qualty Check (push) Successful in 8s
Quality Check / Qualty Check (pull_request) Successful in 9s

Signed-off-by: Dominik Säume <Dominik.Saeume@hmmh.de>
This commit is contained in:
Dominik Säume 2024-05-02 15:07:09 +02:00
parent be45122138
commit fcebe01f1e
Signed by: SZUT-Dominik
GPG key ID: 67D15BB250B41E7C

View file

@ -77,11 +77,10 @@ public class AllTreatmentController {
}
public void readAllAndShowInTableView() {
this.treatments.clear();
comboBoxPatientSelection.getSelectionModel().select(0);
this.dao = DaoFactory.getDaoFactory().createTreatmentDao();
try {
this.treatments.addAll(dao.readAll());
this.treatments.setAll(dao.readAll());
} catch (SQLException exception) {
exception.printStackTrace();
}
@ -109,7 +108,7 @@ public class AllTreatmentController {
if (selectedPatient.equals("alle")) {
try {
this.treatments.addAll(this.dao.readAll());
this.treatments.setAll(this.dao.readAll());
} catch (SQLException exception) {
exception.printStackTrace();
}
@ -118,7 +117,7 @@ public class AllTreatmentController {
Patient patient = searchInList(selectedPatient);
if (patient != null) {
try {
this.treatments.addAll(this.dao.readTreatmentsByPid(patient.getId()));
this.treatments.setAll(this.dao.readTreatmentsByPid(patient.getId()));
} catch (SQLException exception) {
exception.printStackTrace();
}