JavaFX Daten zwischen Controllern austauschen- neue Frage

[ralfb1105]

Hallo zusammen,

in dem Thread https://www.java-forum.org/thema/daten-zwischen-controller-austauschen.183928/ habt Ihr mir hier im Forum seh anschaulich und gut erklärt wie ich von meinem Main Controller weitere Controller aufrufen kann und in diesen dann auf Objekte des Main Controller zugreifen kann --> ControllerInjectable, ControllerFactory und ViewLoader. Dieses Framework habe ich jetzt auch schon erfolgreich in meinem projekt implementiert.

Nun habe ich festgestellt das ich quasi auch noch "die andere Richtung" benötige, soll heißen ich muss von meinem MainController auf Objekte des aufgerufenen Controller zugreifen.

Um es deutlicher zu machen hier mein aktuelles Problem:

Problembeschreibung:
Ich habe einen Controller und in dem wird über einen Button ein weiterer Controller aufgerufen. Hier das Code Beispiel:

// Method for DB Monitor Button
    public void dbMonitorButtonTapped() {
        Parent rootDBMonitor = ViewLoader.load("DBMonitorView.fxml",
                clazz -> ControllerFactory.controllerForClass(clazz, model, db, this));
        Stage stageDBMonitor = new Stage();
        stageDBMonitor.initModality(Modality.NONE);
        stageDBMonitor.setOpacity(1);
        stageDBMonitor.setTitle("DB Monitor View");
        Scene sceneDBMonitor = new Scene(rootDBMonitor);
        sceneDBMonitor.getStylesheets().add(getClass().getResource("application.css").toExternalForm());
        stageDBMonitor.setScene(sceneDBMonitor);
        stageDBMonitor.setOnCloseRequest(event -> {
            LOGGER.info("Close JavaFX Stage: ControllerDBMonitor");
        });
        LOGGER.info("Starting DB Monitor window @ SID " + getDbSid());
        messageTextArea.appendText("Starting DB Monitor window @ SID " + getDbSid() + "\n");
        stageDBMonitor.show();
    }

In dem neuen Controller wird unter anderem eine Datenbank Verbindung in der Initialize() Methode erstellt:

// Create DB connection
        try {
            connSys = db.dbConnect(controller.getConnectString(), controller.getDbSysUser(),
                    controller.getDbSysUserPassword());
        } catch (Exception ex) {
            String msg = String.valueOf(ex);
            controller.messageTextArea.appendText(msg + "\n");
            LOGGER.severe(msg);
            ((Stage) chart.getScene().getWindow()).fireEvent(
                    new WindowEvent(((Stage) chart.getScene().getWindow()), WindowEvent.WINDOW_CLOSE_REQUEST));
        }

Das Fenster hat nur ein AreaChart, sonst nichts.
Wenn ich nun als Benutzer dieses neue Fenster schließe habe ich keine Möglichkeit innerhalb des ControllerDBMonitor die Datenbank Verbindung zu schließen, was natürlich schlecht ist!

Nun habe ich ja in meinem Main Controller in der Button Methode ein Event erstellt, welches ausgeführt wird wenn es ein setOnCloseRequest gibt:

stageDBMonitor.setOnCloseRequest(event -> {
            LOGGER.info("Close JavaFX Stage: ControllerDBMonitor");
        });

Aus meiner Sicht wäre das ja eine gute Stelle um beim schließen des Fenster die Verbindung zu schließen. Nun habe ich aber folgendes Problem:
1. Ich habe keinen Zugriff auf das Connection Objekt aus dem ControllerDBMonitor.
2. Wenn ich mir im MainController eine Instanz von der Klasse ControllerDBMonitor erstelle und dann mit Hilfe von Getter/Setter versuche auf das Objekt zuzugreifen, gibt es eine NPE. Meine Vermutung ist das dieses Objekt dann nicht mehr verfügbar ist .. ??

Frage:
Wie kann ich die Datenbank Verbindung schließen (connSys.close()) wenn das Fenster geschlossen wird?

Kann ich irgendwie in dem ControllerDBMontor einen "Listener" konfigurieren der beim schließen des Fenster noch Aktionen ausführt?

Gruß
Ralf

 

[dzim]

Nun ja, es ist ja nicht beschränkt, was du in deinen Controller reinreichen kannst. Das Connection-Objekt sollte es natürlich nicht sein, aber du kannst ja in deinem Model ein BooleanProperty (o.ä.) setzen und an geeigneter Stelle auf diese Änderung hören und dann darauf reagieren.

So etwas wie folgender (Pseudo-)Code:

// Controller:
model.setChartWindowClosed(true);

// Listener:
model.chartWindowClosedProperty().addListener((obs, o, n) -> {
  if (n != null && n)
    connection.close();
});

In etwa klar, was ich meine?

 

[ralfb1105]

Hallo Daniel,

leider ist mir nicht 100% klar was Du in Deinem Pseudo-Code machst. Ich hatte mir gestern Abend noch genau das überlegt, was Du vermutlich als schlecht zitierst

Nun ja, es ist ja nicht beschränkt, was du in deinen Controller reinreichen kannst. Das Connection-Objekt sollte es natürlich nicht sein

Ich hatte mir überlegt in meinem Main Controller folgendes zu implementieren:

private Connection connSysMonitor;
private Timeline animationDBMonitor;

public Connection getConnSysMonitor() {
        return connSysMonitor;
    }

    public void setConnSysMonitor(Connection connSysMonitor) {
        this.connSysMonitor = connSysMonitor;
    }
  
    public Timeline getAnimationDBMonitor() {
        return animationDBMonitor;
    }

    public void setAnimationDBMonitor(Timeline animationDBMonitor) {
        this.animationDBMonitor = animationDBMonitor;
    }

Im ControllerDBMonitor setze ich dann beim erstellen der Objekte Connection und Timeline diese im Main Controller über die Setter Methode:

public void initialize()  {
        controller.setAnimationDBMonitor(animation);
...
...
// Create DB connection
        try {
            connSys = db.dbConnect(controller.getConnectString(), controller.getDbSysUser(),
                    controller.getDbSysUserPassword());
controller.setConnSysMonitor(connSys);
...
...

Im Main Controller wird dann beim setOnCloseRequest Event folgendes ausgeführt:

stageDBMonitor.setOnCloseRequest(event -> {
            LOGGER.info("Close JavaFX Stage: ControllerDBMonitor");
            try {
                getConnSysMonitor().close();
                getAnimationDBMonitor().stop();
            } catch (SQLException e) {
              
            }
        });

Bei einem ersten kurzen Test scheint das im Prinzip auch zu funktionieren, vermute aber, auf Grund Deines Kommentars, das man es es genau so NICHT macht

Somit würde ich gerne verstehen was Du in Deinem Paseudo Code machst ...

Ich müsste also im Model ein BooleanProperty Objekt erstellen und eine Methode setChartWindowClosed:

public BooleanProperty chartWindowClosedProperty = new SimpleBooleanProperty(false);

public void setChartWindowClosed(boolean value) {
        chartWindowClosedProperty.set(value);
    }

Die Zeile zu dem Listener ist mir im Detail

(obs, o, n) -> {
    if (n != null && n) {

nicht wirklich klar
Ich habe es mal folgendermaßen versucht zu implementieren:
== Main Controller ==

stageDBMonitor.setOnCloseRequest(event -> {
            LOGGER.info("Close JavaFX Stage: ControllerDBMonitor");
            model.setChartWindowClosed(true);
        });

== ControllerDBMonitor ==

public void initialize() {      
        model.chartWindowClosedProperty.addListener((obs, o, n) -> {
            if (n != null && n) {
                animation.stop();
                try {
                    connSys.close();
                } catch (SQLException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
            }
        });
...
...

Ein erster schneller Test sieht vielversprechend aus

Ist es das was Du mit Deinem Pseudo-Code ausdrücken wolltest?
Ist meine Implementierung so in etwas OK?

Danke auf jedem Fall schon mal für deine Hilfe!!

Gruß
Ralf

 

[dzim]

Genau!
Haste doch richtig was draus gemacht!

Warum ich die connection angesprochen habe: Um sauber zu programmieren, solltest du bestimmte Logik nicht vermischen. Das UI sollte nicht auf so spezielle Sachen wie DB-Connection schliessen zugreifen. Denn eigentlich hat man so etwas wie ein DAO (DataAccessObject) über das man DB-Zugriffe händelt, dass aber die konkrete Implementierung verbirgt. Es ist hier aber schon ok, so etwas wie #close oder #dispose also Methode zu haben.

Ich denke, du hast aus meinen wenigen Stichworten das Beste gemacht! Tip Top.

 

[ralfb1105]

Hallo Daniel,

Danke für Dein Kompliment und die zusätzlichen Erklärungen.
Leider habe ich nach dem ich es implementiert habe bei den Tests ein Problem festgestellt
Wenn ich in meinem UI den Button für den Monitor drücke wird die Connection aufgebaut und für diese Connection auch der Listener hinzugefügt. Schließe ich das Monitor Fenster wird die Verbindung auch geschlossen -> Kontrolle in der DB . Wenn ich nun in meinem UI noch einmal den Button für den DB Monitor drücke wird dieser wieder angezeigt und eine neue Verbindung aufgebaut -> unterschiedliche Session ID in der DB.
Problem:
Wenn ich das Fenster jetzt schließe wird die Verbindung leider nicht abgebaut. Ich vermute es liegt daran das ich schon einen Listener habe, bzw. es jetzt zwei Listener gibt ??

Hast Du eine Idee was ich falsch mache oder wo der Gedankenfehler liegt .. muss ich beim schließen den Listener wieder entfernen?

Gruß

Ralf

 

[ralfb1105]

Hallo Daniel,

ich habe mal ein bisschen probiert und sysout's hinzugefügt und verstehe es nicht wirklich.
Ich lasse mir beim addListener() den Wert für 'obs' ausgeben.
Das Ergebnis:
1. Starte UI
2. Drücke Button "Monitor"
Ausgabe: BooleanProperty [value: true]
3. Schließe Fenster "Monitor" -> DB Verbindung und Animation werden gestoppt.
4. Drücke Button "Monitor"
Ausgabe: -Keine-
5. Schließe Fenster "Monitor" -> DB Verbindung beliebt offen und Animation läuft weiter!?

Es sieht für mich so aus als wenn der Listener beim 2. Mal gar nicht hinzugefügt wird ... die initialize() Methode wird ja auf jeden Fall gestartet, denn sonst ginge ja gar nichts in dem Fenster.

Bis jetzt verstehe ich das nicht ...

Gruß
Ralf

 

[ralfb1105]

Hallo Daniel,

Typischer Anfängerfehler ich hatte vergessen das BooleanProperty beim starten der Stage auf false zu setzen. Da ich ja nur eine Model Instanz habe, was auch gut ist blieb der BooleanProperty nach dem schließen des Fenster auf true und somit wurde nichts ausgeführt

if (n != null && n)

Ich habe jetzt in der Methode zum starten des Monitor noch folgendes hinzugefügt:

model.setDBMonitorWindowClosed(false);
stageDBMonitor.show();

und jetzt sieht es nach den ersten Tests gut aus

Gruß
Ralf

 

[dzim]

Ich habe es erst jetzt gelesen und wollte gerade sagen, dass du das Flag auch "resetten" musst... Hast du aber schon wieder in der Zwischenzeit selbst hinbekommen.