Do not throw exceptions if Projektthema is trimmed empty

2 files changed, 33 insertions, 14 deletions

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.");