2 files changed, 17 insertions, 75 deletions

diff --git a/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Ansprechpartner.java b/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Ansprechpartner.java
index 72bf249..fb59831 100644
--- a/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Ansprechpartner.java
+++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Ansprechpartner.java
@@ -4,8 +4,6 @@ import java.util.ArrayList;
 import java.util.List;
 
 import javafx.beans.Observable;
-import javafx.beans.property.ObjectProperty;
-import javafx.beans.property.SimpleObjectProperty;
 import javafx.beans.property.SimpleStringProperty;
 import javafx.beans.property.StringProperty;
 import javafx.util.Callback;
@@ -16,7 +14,7 @@ public class Ansprechpartner
 
    private StringProperty nachname = new SimpleStringProperty();
 
-   private ObjectProperty<Organisation> organisation = new SimpleObjectProperty<>();
+   private Organisation organisation;
 
    /**
     * Generates an extractor that fires when a property of a list value
@@ -26,21 +24,10 @@ public class Ansprechpartner
     */
    public static Callback<Ansprechpartner, Observable[]> extractor()
    {
-      return ansprechpartner -> {
-         List<Observable> extractorList = new ArrayList<>();
-
-         extractorList.add(ansprechpartner.vornameProperty());
-         extractorList.add(ansprechpartner.nachnameProperty());
-         extractorList.add(ansprechpartner.organisationProperty());
-
-         if (ansprechpartner.getOrganisation() != null)
-         {
-            extractorList
-                  .add(ansprechpartner.getOrganisation().nameProperty());
-         }
-
-         return extractorList.toArray(new Observable[extractorList.size()]);
-      };
+      return ansprechpartner -> new Observable[] {
+            ansprechpartner.vornameProperty(),
+            ansprechpartner.nachnameProperty(),
+            ansprechpartner.getOrganisation().nameProperty() };
    }
 
    /**
@@ -50,23 +37,15 @@ public class Ansprechpartner
     */
    public Ansprechpartner(Organisation organisation)
    {
-      this.organisation.set(organisation);
-
-      if (organisation != null)
+      if (organisation == null)
       {
-         this.organisation.get().addAnsprechpartner(this);
+         throw new IllegalArgumentException(
+               "Ansprechpartner sollten nicht arbeitslos sein.");
       }
+      this.organisation = organisation;
+      this.organisation.addAnsprechpartner(this);
    }
 
-   /**
-    * Getter for property organisation.
-    *
-    * @return Returns the organisation.
-    */
-   public ObjectProperty<Organisation> organisationProperty()
-   {
-      return organisation;
-   }
 
    /**
     * Getter for organisation.
@@ -75,7 +54,7 @@ public class Ansprechpartner
     */
    public Organisation getOrganisation()
    {
-      return organisation.get();
+      return organisation;
    }
 
    /**
@@ -86,17 +65,14 @@ public class Ansprechpartner
     */
    public void setOrganisation(Organisation organisation)
    {
-      if (this.organisation.get() != null)
+      if (organisation == null)
       {
-         this.organisation.get().removeAnsprechpartner(this);
-      }
-
-      this.organisation.set(organisation);
-
-      if (organisation != null)
-      {
-         organisation.addAnsprechpartner(this);
+         throw new IllegalArgumentException(
+               "Ansprechpartner sollten nicht arbeitslos sein.");
       }
+      this.organisation.removeAnsprechpartner(this);
+      this.organisation = organisation;
+      this.organisation.addAnsprechpartner(this);
    }
 
    /**
@@ -160,22 +136,4 @@ public class Ansprechpartner
    {
       this.vorname.set(vorname);
    }
-
-   /*
-    * (non-Javadoc)
-    * 
-    * @see java.lang.Object#equals(java.lang.Object)
-    */
-   @Override
-   public boolean equals(Object obj)
-   {
-      if (!(obj instanceof Ansprechpartner))
-      {
-         return false;
-      }
-
-      return ((Ansprechpartner) obj).vorname.get().equals(vorname.get())
-            && ((Ansprechpartner) obj).nachname.get().equals(nachname.get());
-   }
-
 }
\ No newline at end of file
diff --git a/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Organisation.java b/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Organisation.java
index 4525045..dca4511 100644
--- a/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Organisation.java
+++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Organisation.java
@@ -107,20 +107,4 @@ public class Organisation
    {
       return name.get();
    }
-
-   /*
-    * (non-Javadoc)
-    * 
-    * @see java.lang.Object#equals(java.lang.Object)
-    */
-   @Override
-   public boolean equals(Object obj)
-   {
-      if (!(obj instanceof Organisation))
-      {
-         return false;
-      }
-
-      return ((Organisation) obj).name.get().equals(name.get());
-   }
 }
\ No newline at end of file