diff options
Diffstat (limited to 'src/main/java/at/ac/tuwien/sepm/assignment/groupphase')
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(); } } |