diff options
| -rw-r--r-- | src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/CreateCarController.java | 43 |
1 files changed, 14 insertions, 29 deletions
diff --git a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/CreateCarController.java b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/CreateCarController.java index 1c74d2d..a4df90b 100644 --- a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/CreateCarController.java +++ b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/CreateCarController.java @@ -1,5 +1,9 @@ package at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.controller; +import static at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.controller.Helper.showServiceExceptionAlertAndWait; +import static at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.controller.Helper.showSuccessAlertAndWait; +import static at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.controller.Helper.showValidationErrorAlertAndWait; + import at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.dto.Vehicle; import at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.dto.Vehicle.ConstructionType; import at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.dto.Vehicle.Status; @@ -15,10 +19,7 @@ import java.util.stream.Stream; import javafx.collections.FXCollections; import javafx.event.ActionEvent; import javafx.fxml.FXML; -import javafx.scene.control.Alert; -import javafx.scene.control.Alert.AlertType; import javafx.scene.control.Button; -import javafx.scene.control.ButtonType; import javafx.scene.control.CheckBox; import javafx.scene.control.ChoiceBox; import javafx.scene.input.MouseButton; @@ -101,19 +102,17 @@ public class CreateCarController { setToStart(); } catch (InvalidVehicleException e) { LOG.debug("Validation of Vehicle failed"); - createComplete(AlertType.ERROR, "Ungültige Eingabe", e.getMessage()); + showValidationErrorAlertAndWait(e.getMessage()); setToStart(); return; } catch (ServiceException e) { LOG.error("ServiceException in createCar(). ", e); - createComplete(AlertType.ERROR, "Fehler", e.getMessage()); + showServiceExceptionAlertAndWait( + "Ein Fehler beim Erstellen des Fahrzeuges ist aufgetreten."); setToStart(); return; } - createComplete( - AlertType.CONFIRMATION, - "Speichern Erfolgreich", - "Auto wurde erfolgreich angelegt"); + showSuccessAlertAndWait("Auto wurde erfolgreich angelegt."); } else { LOG.debug("Button \"Speichern\" clicked."); try { @@ -131,19 +130,16 @@ public class CreateCarController { chooseVehicle = null; } catch (InvalidVehicleException e) { LOG.debug("Validation of Vehicle failed"); - createComplete(AlertType.ERROR, "Ungültige Eingabe", e.getMessage()); + showValidationErrorAlertAndWait(e.getMessage()); setToStart(); return; } catch (ServiceException e) { LOG.error("ServiceException in createCar(). ", e); - createComplete(AlertType.ERROR, "Fehler", e.getMessage()); + showServiceExceptionAlertAndWait(e.getMessage()); setToStart(); return; } - createComplete( - AlertType.CONFIRMATION, - "Bearbeiten Erfolgreich", - "Auto wurde erfolgreich bearbeitet"); + showSuccessAlertAndWait("Auto wurde erfolgreich bearbeitet."); } updateVehiclePane(); @@ -153,20 +149,14 @@ public class CreateCarController { if (cmbCtype.getSelectionModel().getSelectedItem() == null) { return ConstructionType.NORMAL; } - return ConstructionType.valueOf(cmbCtype.getSelectionModel().getSelectedItem().toString()); + return ConstructionType.valueOf(cmbCtype.getSelectionModel().getSelectedItem()); } private VehicleType parseType() { if (cmbTyp.getSelectionModel().getSelectedItem() == null) { return VehicleType.BKTW; } - return VehicleType.valueOf(cmbTyp.getSelectionModel().getSelectedItem().toString()); - } - - private void createComplete(AlertType alertType, String headerText, String contentText) { - Alert alert = new Alert(alertType, contentText, ButtonType.OK); - alert.setHeaderText(headerText); - alert.showAndWait(); + return VehicleType.valueOf(cmbTyp.getSelectionModel().getSelectedItem()); } private void setToStart() { @@ -235,12 +225,7 @@ public class CreateCarController { } } catch (ServiceException | IOException e) { LOG.error("Exception in updateVehiclePane(). ", e); - - Alert alert = new Alert(AlertType.ERROR); - alert.setTitle("Fehler"); - alert.setHeaderText("Fehler!"); - alert.setContentText(e.getMessage()); - alert.showAndWait(); + showServiceExceptionAlertAndWait(e.getMessage()); } } } |