Use helper methods for showing alerts in CreateCarController [#25963]

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());
         }
     }
 }