From f1327bcab1dd85aebeda1d5368adb8a43e1c4d1f Mon Sep 17 00:00:00 2001 From: Stefan Suhren Date: Mon, 11 Jan 2016 17:23:46 +0100 Subject: Do not throw exceptions if Projektthema is trimmed empty --- .../view/ProjektHinzufuegenController.java | 45 +++++++++++++++------- .../view/StudentenverwaltungsController.java | 2 +- 2 files changed, 33 insertions(+), 14 deletions(-) (limited to 'src/de/fhswf/in/inf/se/projektthemenvergabe') diff --git a/src/de/fhswf/in/inf/se/projektthemenvergabe/view/ProjektHinzufuegenController.java b/src/de/fhswf/in/inf/se/projektthemenvergabe/view/ProjektHinzufuegenController.java index 1a5a193..08dea73 100644 --- a/src/de/fhswf/in/inf/se/projektthemenvergabe/view/ProjektHinzufuegenController.java +++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/view/ProjektHinzufuegenController.java @@ -5,6 +5,8 @@ import de.fhswf.in.inf.se.projektthemenvergabe.model.Ansprechpartner; import de.fhswf.in.inf.se.projektthemenvergabe.model.Projekt; import de.fhswf.in.inf.se.projektthemenvergabe.model.Student; import javafx.fxml.FXML; +import javafx.scene.control.Alert; +import javafx.scene.control.Alert.AlertType; import javafx.scene.control.Button; import javafx.scene.control.ComboBox; import javafx.scene.control.TextArea; @@ -51,6 +53,8 @@ public class ProjektHinzufuegenController private Projekt projekt; + private Main main; + private Stage projekthinzufuegenStage; private boolean wasAborted = true; @@ -65,6 +69,7 @@ public class ProjektHinzufuegenController public void setMain(Main main, Projekt projekt) { this.projekt = projekt; + this.main = main; projekthinzufuegenStage = main.getProjekthinzufuegenStage(); @@ -199,25 +204,39 @@ public class ProjektHinzufuegenController @FXML private void handleSave() { - if (projekt == null) + if (!projektthemaTextField.getText().trim().isEmpty()) { - projekt = new Projekt(projektthemaTextField.getText().trim(), - student1ComboBox.getValue(), student2ComboBox.getValue(), - student3ComboBox.getValue(), - ansprechpartnerComboBox.getValue()); + if (projekt == null) + { + projekt = new Projekt(projektthemaTextField.getText().trim(), + student1ComboBox.getValue(), student2ComboBox.getValue(), + student3ComboBox.getValue(), + ansprechpartnerComboBox.getValue()); + } + else + { + projekt.setProjektthema(projektthemaTextField.getText().trim()); + } + + projekt.setSkizze(projektskizzeTextField.getText().trim()); + projekt.setProjektbeschreibung( + projekthintergrundTextField.getText().trim()); + projekt.setProjekteinhalte(projektinhalteTextField.getText().trim()); + + wasAborted = false; + projekthinzufuegenStage.close(); } else { - projekt.setProjektthema(projektthemaTextField.getText().trim()); - } + Alert alert = new Alert(AlertType.ERROR); + alert.setTitle("Projektthema ist leer"); + alert.setHeaderText("Projektthema kann nicht leer sein."); + alert.setContentText("Bitte überprüfen Sie das Projektthema."); - projekt.setSkizze(projektskizzeTextField.getText().trim()); - projekt.setProjektbeschreibung( - projekthintergrundTextField.getText().trim()); - projekt.setProjekteinhalte(projektinhalteTextField.getText().trim()); + alert.initOwner(main.getProjekthinzufuegenStage()); - wasAborted = false; - projekthinzufuegenStage.close(); + alert.showAndWait(); + } } /** diff --git a/src/de/fhswf/in/inf/se/projektthemenvergabe/view/StudentenverwaltungsController.java b/src/de/fhswf/in/inf/se/projektthemenvergabe/view/StudentenverwaltungsController.java index 0037124..a86bbe9 100644 --- a/src/de/fhswf/in/inf/se/projektthemenvergabe/view/StudentenverwaltungsController.java +++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/view/StudentenverwaltungsController.java @@ -119,7 +119,7 @@ public class StudentenverwaltungsController } catch (NumberFormatException e) { - Alert alert = new Alert(AlertType.WARNING); + Alert alert = new Alert(AlertType.ERROR); alert.setTitle("Matrikelnummer ungültig"); alert.setHeaderText( "Die von Ihnen eingegebene Matrikelnummer ist nicht gültig, da sie keine ganze Zahl ist."); -- cgit v1.2.3-70-g09d2