From db9dbb0dd840a1f2c7500b65e71ee3942265670f Mon Sep 17 00:00:00 2001 From: Stefan Suhren Date: Tue, 15 Dec 2015 20:05:13 +0100 Subject: Don't remove all Ansprechpartner on Organisation delete --- .../fhswf/in/inf/se/projektthemenvergabe/Main.java | 23 ++++---- .../model/Ansprechpartner.java | 62 ++++++++++++++++------ .../projektthemenvergabe/model/Organisation.java | 2 +- .../view/AnsprechpartnerListeController.java | 32 ++++++++--- 4 files changed, 85 insertions(+), 34 deletions(-) (limited to 'src/de') diff --git a/src/de/fhswf/in/inf/se/projektthemenvergabe/Main.java b/src/de/fhswf/in/inf/se/projektthemenvergabe/Main.java index fe144e7..e786f78 100644 --- a/src/de/fhswf/in/inf/se/projektthemenvergabe/Main.java +++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/Main.java @@ -175,15 +175,13 @@ public class Main extends Application { Organisation organisation = orga .get(rs.getString("organisation")); - if (organisation != null) - { - Ansprechpartner ansprechpartner = new Ansprechpartner( - organisation); - ansprechpartner.setVorname(rs.getString("vorname")); - ansprechpartner.setNachname(rs.getString("nachname")); - this.ansprechpartner.add(ansprechpartner); - } + Ansprechpartner ansprechpartner = new Ansprechpartner( + organisation); + ansprechpartner.setVorname(rs.getString("vorname")); + ansprechpartner.setNachname(rs.getString("nachname")); + + this.ansprechpartner.add(ansprechpartner); } organisation.addAll(orga.values()); @@ -223,7 +221,14 @@ public class Main extends Application { ps.setString(1, ans.getVorname()); ps.setString(2, ans.getNachname()); - ps.setString(3, ans.getOrganisation().getName()); + if (ans.getOrganisation() != null) + { + ps.setString(3, ans.getOrganisation().getName()); + } + else + { + ps.setString(3, null); + } ps.execute(); } 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 11873e0..66b6ab5 100644 --- a/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Ansprechpartner.java +++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Ansprechpartner.java @@ -1,6 +1,11 @@ package de.fhswf.in.inf.se.projektthemenvergabe.model; +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; @@ -11,7 +16,7 @@ public class Ansprechpartner private StringProperty nachname = new SimpleStringProperty(); - private Organisation organisation; + private ObjectProperty organisation = new SimpleObjectProperty<>(); /** * Generates an extractor that fires when a property of a list value @@ -21,10 +26,21 @@ public class Ansprechpartner */ public static Callback extractor() { - return ansprechpartner -> new Observable[] { - ansprechpartner.vornameProperty(), - ansprechpartner.nachnameProperty(), - ansprechpartner.getOrganisation().nameProperty() }; + return ansprechpartner -> { + List 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()]); + }; } /** @@ -34,23 +50,32 @@ public class Ansprechpartner */ public Ansprechpartner(Organisation organisation) { - if (organisation == null) + this.organisation.set(organisation); + + if (organisation != null) { - throw new IllegalArgumentException( - "Ansprechpartner sollten nicht arbeitslos sein."); + this.organisation.get().addAnsprechpartner(this); } - this.organisation = organisation; - this.organisation.addAnsprechpartner(this); } /** * Getter for property organisation. + * + * @return Returns the organisation. + */ + public ObjectProperty organisationProperty() + { + return organisation; + } + + /** + * Getter for organisation. * * @return Returns the organisation. */ public Organisation getOrganisation() { - return organisation; + return organisation.get(); } /** @@ -61,14 +86,17 @@ public class Ansprechpartner */ public void setOrganisation(Organisation organisation) { - if (organisation == null) + if (this.organisation.get() != null) + { + this.organisation.get().removeAnsprechpartner(this); + } + + this.organisation.set(organisation); + + if (organisation != null) { - throw new IllegalArgumentException( - "Ansprechpartner sollten nicht arbeitslos sein."); + organisation.addAnsprechpartner(this); } - this.organisation.removeAnsprechpartner(this); - this.organisation = organisation; - this.organisation.addAnsprechpartner(this); } /** 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 8f04979..dca4511 100644 --- a/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Organisation.java +++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Organisation.java @@ -11,7 +11,7 @@ import javafx.util.Callback; public class Organisation { - private StringProperty name = new SimpleStringProperty(); + private StringProperty name = new SimpleStringProperty(""); private List ansprechpartner = new ArrayList<>(); diff --git a/src/de/fhswf/in/inf/se/projektthemenvergabe/view/AnsprechpartnerListeController.java b/src/de/fhswf/in/inf/se/projektthemenvergabe/view/AnsprechpartnerListeController.java index 2ae8405..27f85c9 100644 --- a/src/de/fhswf/in/inf/se/projektthemenvergabe/view/AnsprechpartnerListeController.java +++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/view/AnsprechpartnerListeController.java @@ -92,14 +92,26 @@ public class AnsprechpartnerListeController }); organisationsNameTableColumn.setCellValueFactory((data) -> { - data.getValue().getOrganisation().nameProperty() - .addListener((listener) -> { - // Hack, aktualisiert TableView, da Organisation kein Observable + data.getValue().organisationProperty().addListener((listener) -> { + // Hack, aktualisiert TableView, da Organisation kein + // Observable // ist organisationsNameTableColumn.setVisible(false); organisationsNameTableColumn.setVisible(true); }); - return new SimpleObjectProperty<>(data.getValue().getOrganisation()); + if (data.getValue().getOrganisation() != null) + { + data.getValue().getOrganisation().nameProperty() + .addListener((listener) -> { + // Hack, aktualisiert TableView, da Organisation kein + // Observable + // ist + organisationsNameTableColumn.setVisible(false); + organisationsNameTableColumn.setVisible(true); + }); + return data.getValue().organisationProperty(); + } + return new SimpleObjectProperty(new Organisation()); }); organisationsNameTableColumn.setOnEditCommit((data) -> { data.getRowValue().setOrganisation(data.getNewValue()); @@ -203,8 +215,14 @@ public class AnsprechpartnerListeController if (result == null || result.get() == ButtonType.OK) { - main.getAnsprechpartner() - .removeAll(selectedItem.getAnsprechpartner()); + Ansprechpartner[] ansArray = selectedItem.getAnsprechpartner() + .toArray(new Ansprechpartner[selectedItem + .getAnsprechpartner().size()]); + for (Ansprechpartner ans : ansArray) + { + ans.setOrganisation(null); + } + main.getOrganisation().remove(organisationComboBox.getValue()); } } @@ -222,7 +240,7 @@ public class AnsprechpartnerListeController if (selectedItem != null) { - selectedItem.getOrganisation().removeAnsprechpartner(selectedItem); + selectedItem.setOrganisation(null); main.getAnsprechpartner().remove(selectedItem); } } -- cgit v1.2.3-70-g09d2