Small Changes (deleted Comments and TODOs) added an Alert if Operation was saved successfully

3 files changed, 18 insertions, 16 deletions

diff --git a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/dao/DBOperationDAO.java b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/dao/DBOperationDAO.java
index 707d346..485d6fa 100644
--- a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/dao/DBOperationDAO.java
+++ b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/dao/DBOperationDAO.java
@@ -5,7 +5,6 @@ import at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.dto.Operation.S
 import at.ac.tuwien.sepm.assignment.groupphase.exception.ElementNotFoundException;
 import at.ac.tuwien.sepm.assignment.groupphase.exception.PersistenceException;
 import at.ac.tuwien.sepm.assignment.groupphase.util.JDBCConnectionManager;
-import java.beans.Statement;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
@@ -62,7 +61,6 @@ public class DBOperationDAO implements OperationDAO {
                     throw new PersistenceException(
                             "Schwere des Einsatzes konnte nicht validiert werden!");
             }
-            // TODO: CHECK IF TIME ZONE CORRECT
             if (operation.created() != null) {
                 pstmt.setTimestamp(3, Timestamp.from(operation.created()));
             } else {
@@ -91,9 +89,7 @@ public class DBOperationDAO implements OperationDAO {
             }
             pstmt.executeUpdate();
             ResultSet rs = pstmt.getGeneratedKeys();
-            if (rs.next()) {
-                return rs.getInt(1);
-            }
+            if (rs.next()) return rs.getInt(1);
             else throw new PersistenceException("Einsatz konnte nicht gespeichert werden");
         } catch (SQLException e) {
             throw new PersistenceException(e);
@@ -142,7 +138,7 @@ public class DBOperationDAO implements OperationDAO {
                 throw new PersistenceException(
                         "Fahrzeug für die Operation konnte nicht abgespeichert werden!");
         } catch (SQLException e) {
-            throw new PersistenceException(e);
+            throw new PersistenceException("Die Werte konnten nicht gespeichert werden!");
         } finally {
             if (pstmt != null) {
                 try {
diff --git a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/service/OperationServiceImpl.java b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/service/OperationServiceImpl.java
index 9ba3a63..10eef78 100644
--- a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/service/OperationServiceImpl.java
+++ b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/service/OperationServiceImpl.java
@@ -25,18 +25,18 @@ public class OperationServiceImpl implements OperationService {
     public long add(Operation operation) throws InvalidOperationException, ServiceException {
         List<Vehicle> vehicles = operation.vehicles();
         boolean rtw = false;
-        if (operation.vehicles().size() == 0) {
-            throw new InvalidOperationException(
-                    "Es muss mindestens ein Fahrzeug ausgewählt werden!");
+        if (faultyInput(operation.opCode())){
+            throw new InvalidOperationException("Code ist ungültig!");
         }
         if (faultyInput(operation.destination())){
             throw new InvalidOperationException("Adresse ist ungültig!");
         }
-        if (faultyInput(operation.opCode())){
-            throw new InvalidOperationException("Code ist ungültig!");
+        if (operation.vehicles().size() == 0) {
+            throw new InvalidOperationException(
+                    "Es muss mindestens ein Fahrzeug ausgewählt werden!");
         }
         for (Vehicle vehicle : vehicles) {
-            if (vehicle.status() == Vehicle.Status.ABGEMELDET)
+            if (vehicle.status() == Vehicle.Status.ABGEMELDET || (vehicle.status()!=Vehicle.Status.FREI_FUNK && vehicle.status()!=Vehicle.Status.FREI_WACHE))
                 throw new InvalidOperationException(
                         "Abgemeldete Fahrzeuge dürfen nicht zu einem Einsatz geschickt werden!");
             if (vehicle.type() == VehicleType.NEF && !rtw) {
@@ -58,12 +58,11 @@ public class OperationServiceImpl implements OperationService {
                     }
                 }
                 if (!rtw)
-                    // TODO: NUR WARNUNG AUSGEBEN
                     throw new InvalidOperationException(
                             "Zu einem Fahrzeug des Typs NAH muss auch ein Fahrzeug des Typs RTW geschickt werden!");
             }
         }
-        String[] codeParts = operation.opCode().split("\\-");
+        String[] codeParts = operation.opCode().split("-");
         String severity = "";
         for (int i = 0; i < codeParts[1].length(); i++) {
             if (((int) (codeParts[1].charAt(i)) >= 65 && (int) (codeParts[1].charAt(i)) <= 79) || ((int) (codeParts[1].charAt(i)) >= 97 && (int) (codeParts[1].charAt(i))<=111)) {
diff --git a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/userInterface/CreateOperationController.java b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/userInterface/CreateOperationController.java
index 8df8acc..919e187 100644
--- a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/userInterface/CreateOperationController.java
+++ b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/userInterface/CreateOperationController.java
@@ -19,6 +19,7 @@ import java.util.List;
 import javafx.collections.FXCollections;
 import javafx.fxml.FXML;
 import javafx.scene.control.Alert;
+import javafx.scene.control.Alert.AlertType;
 import javafx.scene.control.Button;
 import javafx.scene.control.Label;
 import javafx.scene.control.ListCell;
@@ -40,7 +41,6 @@ public class CreateOperationController {
 
     // TODO: Anders?
     OperationService operationService = new OperationServiceImpl();
-    VehicleService vehicleService = new VehicleServiceImpl();
 
     public CreateOperationController() {}
 
@@ -107,7 +107,7 @@ public class CreateOperationController {
             this.lvVehicles.setItems(
             //TODO: ALLE FREI STATI Frei FUnk Frei Wache
                     FXCollections.observableArrayList(
-                            vehicleService.list(EnumSet.of(Vehicle.Status.FREI_FUNK))));
+                            vehicleService.list(EnumSet.of(Vehicle.Status.FREI_FUNK, Vehicle.Status.FREI_WACHE))));
         } catch (ServiceException e) {
             Alert alert = new Alert(Alert.AlertType.ERROR);
             alert.setTitle("Fehler");
@@ -168,6 +168,13 @@ public class CreateOperationController {
             alert.setHeaderText("Fehler!");
             alert.setContentText(e.getMessage());
             alert.showAndWait();
+            return;
         }
+        Alert alert = new Alert(AlertType.CONFIRMATION);
+        alert.setTitle("Erfolg");
+        alert.setHeaderText("Erfolgreich gespeichert");
+        alert.setContentText("Der Einsatz wurde erfolgreich gespeichert.");
+        alert.showAndWait();
+        fillList();
     }
 }