JavaFX Label in Controller-Klasse in einer anderen Controller-Klasse ändern

[wainmaster]

Hallo zusammen,

Ich habe folgendes Problem: Ich habe mit Hilfe des Scene Builders mehrere Fenster erstellt, die ich alle mit Hilfe von Controller-Klassen bearbeite. Am Anfang des Programms wird ein Login-Fenster aufgerufen, welches ein Hilfe-Fenster bei einem fehlgeschlagenen Login öffnet. Das klappt alles auch soweit. Nun wollte ich das Hilfe-Fenster so gestalten, dass der darin enthaltende Text universell anpassbar ist, da im Laufe des Programms die Möglichkeit besteht mehrere solcher Fenster öffnen zu lassen.
Mein Problem ist nun, dass ich es nicht hinbekomme den Text des Labels durch die Controller-Klasse des Login-Fensters zu verändern. Ich habe bereits versucht diese Änderung durch eine Setter-Methode in der InfoMessageController-Klasse zu realisieren aber irgendwie hat das nicht funktioniert..

Die beiden Controller-Klassen hänge ich mal mit ran:

LoginController: Ist die Controller-Klasse für das Login-Fenster

package com.thwildau.pachisiG2.controller;

import java.net.URL;
import java.util.ResourceBundle;

import javafx.event.ActionEvent;
import javafx.fxml.FXML;
import javafx.fxml.FXMLLoader;
import javafx.fxml.Initializable;
import javafx.scene.Node;
import javafx.scene.Parent;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.Label;
import javafx.scene.control.PasswordField;
import javafx.scene.control.TextArea;
import javafx.scene.control.TextField;
import javafx.scene.layout.VBox;
import javafx.stage.Modality;
import javafx.stage.Stage;

public class LoginController implements Initializable{

	@Override
	public void initialize(URL location, ResourceBundle resources) {		
	}
	
	@FXML
	private Label lblUsername;
	
	@FXML
	private Label lblPassword;
	
	@FXML
	private TextField txtUsername;
	
	@FXML
	private PasswordField pwPassword;
	
	@FXML
	private Button btnLogin;
	
	@FXML
	private Button btnRegister;
	
	@FXML
	private void switchToPassword(ActionEvent event) {
		pwPassword.requestFocus();
	}
	
	@FXML
	private void sendInformation(ActionEvent event) {
		if(txtUsername.getText().equalsIgnoreCase("Test") && pwPassword.getText().equals("1234")) {
			System.out.println("User " + txtUsername.getText() + " logged in");
			try{
				Node node = (Node) event.getSource();
				Stage stage = (Stage) node.getScene().getWindow();
				Parent root = FXMLLoader.load(getClass().getResource("/com/thwildau/pachisiG2/layout/GameSetup.fxml"));/* Exception */
				Scene scene = new Scene(root);
				stage.setScene(scene);
				stage.show();
				} catch(Exception e) {
					e.printStackTrace();
				}
		}
		else {
			try {
				Stage stage = new Stage();			
				Parent root = FXMLLoader.load(getClass().getResource("/com/thwildau/pachisiG2/layout/InfoMessage.fxml"));
				Scene scene = new Scene(root);
				stage.setScene(scene);
				stage.initModality(Modality.APPLICATION_MODAL);
				stage.show();
				} catch(Exception e) {
					e.printStackTrace();
				}
		}
	}
	
	public void login(ActionEvent event) {
		if(txtUsername.getText().equalsIgnoreCase("Test") && pwPassword.getText().equals("1234")) {
			System.out.println("User " + txtUsername.getText() + " logged in");
			try{
				Node node = (Node) event.getSource();
				Stage stage = (Stage) node.getScene().getWindow();
				Parent root = FXMLLoader.load(getClass().getResource("/com/thwildau/pachisiG2/layout/GameSetup.fxml"));/* Exception */
				Scene scene = new Scene(root);
				stage.setScene(scene);
				stage.show();
				} catch(Exception e) {
					e.printStackTrace();
				}
		}
		else {
			//Hier sollte der Text (Label) der InfoMessageController-Klasse geändert werden
			try {
			Stage stage = new Stage();	
		    Parent root = FXMLLoader.load(getClass().getResource("/com/thwildau/pachisiG2/layout/InfoMessage.fxml"));
			Scene scene = new Scene(root);
			stage.setScene(scene);
			stage.initModality(Modality.APPLICATION_MODAL);
			stage.show();
			} catch(Exception e) {
				e.printStackTrace();
			}
		}
	}
	
	public void runRegistration(ActionEvent event) {
		try{
			Node node = (Node) event.getSource();
			Stage stage = (Stage) node.getScene().getWindow();
			Parent root = FXMLLoader.load(getClass().getResource("/com/thwildau/pachisiG2/layout/Registration.fxml"));/* Exception */
			Scene scene = new Scene(root);
			stage.setScene(scene);
			stage.show();
			} catch(Exception e2) {
				e2.printStackTrace();
			}
	}

}

InfoMessageController: Controller-Klasse für die universell anpassbare InfoMessage

package com.thwildau.pachisiG2.controller;

import java.net.URL;
import java.util.ResourceBundle;

import javafx.event.ActionEvent;
import javafx.fxml.FXML;
import javafx.fxml.Initializable;
import javafx.scene.Node;
import javafx.scene.control.Button;
import javafx.scene.control.Label;
import javafx.stage.Stage;

public class InfoMessageController implements Initializable{
	
	@Override
	public void initialize(URL location, ResourceBundle resources) {
	}
	
	@FXML
	private Label lblInfoMessage;
	
	@FXML
	private Button btnOk;

	public void closeWindow(ActionEvent event) {
		try{
			Node node = (Node) event.getSource();
			Stage stage = (Stage) node.getScene().getWindow();
			stage.close();
			} catch(Exception e) {
				e.printStackTrace();
			}
		
	}

}

Hat vielleicht jemand eine Idee, wie ich mein Vorhaben realisieren kann? Wäre wirklich sehr dankbar!!

 

[dzim]

Gibt ein paar Wege:

Hole dir den Controller, nachdem du mit dem FXMLLoader die UI geladen hast (dazu musst du aber eine Instanz des FXMLLoader anlegen, anstelle von der Nutzung der statischen #load-Methode), dann kannst du mit einem Setter arbeiten (nach dem du das Objekt, das du bekommst, auf deine Controller-Klasse gecastet hast). Eine Variation wäre, den Controller nicht im FXML anzugeben, sondern selbst eine Instanz anzulegen und dem FXMLLoader zu übergeben (oder man kann noch irgendwie anders sich in den Prozess einklinken - mit der dem Callback-Interface, das man im FXMLLoader per #setControllerFactory angeben kann).

        else { // Zeile 94 bei dir
            //Hier sollte der Text (Label) der InfoMessageController-Klasse geändert werden
            try {
            Stage stage = new Stage();  
            FXMLLoader fxmlLoader = new FXMLLoader();
            Parent root = fxmlLoader.load(getClass().getResource("/com/thwildau/pachisiG2/layout/InfoMessage.fxml"));
            LoginController lc = (LoginController) fxmlLoader.getController();
            ls.updateInfoLabel("Supertolle Hilfe!");
            Scene scene = new Scene(root);
            stage.setScene(scene);
            stage.initModality(Modality.APPLICATION_MODAL);
            stage.show();
            } catch(Exception e) {
                e.printStackTrace();
            }
        }

Alternativ kannst du auch Events verwenden. Ist nicht das sauberste, man gerät vielleicht auch in Gefahr den Überblick zu verlieren, aber das ist mein favorisierter Weg.
Ich habe dazu immer einen (statischen) "Event Dispatcher", bei dem man einen Listener (also ein zu implementierendes Interface) registriert, welcher nur auf bestimmte Events hört (kann man ja per Strings, wie PropertyChangeEvents oder per Integers, festlegen). In dem Controller hörst du dann auf das Event und passt daraufhin den Inhalt des Labels an.
In deinem Code würde das vielleicht so etwas wie:

        else { // Zeile 94 bei dir
            //Hier sollte der Text (Label) der InfoMessageController-Klasse geändert werden
            try {
            Stage stage = new Stage();  
            Parent root = FXMLLoader.load(getClass().getResource("/com/thwildau/pachisiG2/layout/InfoMessage.fxml"));
            Scene scene = new Scene(root);
            stage.setScene(scene);
            stage.initModality(Modality.APPLICATION_MODAL);
            stage.show();
            // vielleicht auch vor dem stage.show(); möglich
            EventDispatcher.getInstance().fireEvent("updateText.infoLabel", "Supertolle Hilfe!");
            } catch(Exception e) {
                e.printStackTrace();
            }
        }

Irgendwie so :-D

 

[wainmaster]

        else { // Zeile 94 bei dir
            //Hier sollte der Text (Label) der InfoMessageController-Klasse geändert werden
            try {
            Stage stage = new Stage();  
            FXMLLoader fxmlLoader = new FXMLLoader();
            Parent root = fxmlLoader.load(getClass().getResource("/com/thwildau/pachisiG2/layout/InfoMessage.fxml"));
            LoginController lc = (LoginController) fxmlLoader.getController();
            ls.updateInfoLabel("Supertolle Hilfe!");
            Scene scene = new Scene(root);
            stage.setScene(scene);
            stage.initModality(Modality.APPLICATION_MODAL);
            stage.show();
            } catch(Exception e) {
                e.printStackTrace();
            }
        }

Das gefällt mir persönlich eigentlich ganz gut. Ich muss doch aber in der Zeile

 "LoginController lc = (LoginController) fxmlLoader.getController();

eigentlich meinen InfoMessageController nehmen oder irre ich mich da? Wenn ich das dann mache bekomme ich aber leider eine NullPointerException :cry:

 

[dzim]

Hey - ich kenn deine Controller nicht, hab einfach den erstbesten genommen der mir über den Weg lief - also ja: Wird wohl der sein, den du meinst.
Und die Methode musst du natürlich noch in dem entsprechendem Controller implementieren - die hat er ja nicht per se...

 

[wainmaster]

Die setter-Methode habe ich ja auch schon in der InfoMessageController-Klasse eingefügt aber trotzdem bekomme ich nur die NullPointerException..

So sieht bei mir die Setter-Methode in der InfoMessageController-Klasse aus:

public void setInfo(String message) {
  //Ändere den Labeltext
  lblInfoMessage.setText(message);
}

und den Code in der anderen Controller-Klasse habe ich für mich angepasst:

                                Stage stage = new Stage();
				FXMLLoader fxmlLoader = new FXMLLoader();
				Parent root = fxmlLoader.load(getClass().getResource("/com/thwildau/pachisiG2/layout/InfoMessage.fxml"));
				InfoMessageController ic = (InfoMessageController) fxmlLoader.getController();
				ic.setInfo("Testmeldung");
				Scene scene = new Scene(root);
				stage.setScene(scene);
				stage.show();

Oder blicke ich grade einfach nicht was du meinst?^^

 

[dzim]

Wo genau hast du die Exception?

 

[wainmaster]

Die Exception ist in der LoginController-Klasse und hat anscheinend was mit

ic.setInfo("Testmeldung");

zu tun.

 

[Joose]

hat anscheinend was mit ... zu tun.

Warum nur anscheinend? Du siehst die Exception und deren StackTrace.
Der StackTrace kann dir auf die Zeile genau sagen wo das Problem auftritt.

Wenn du eine Exception bekommst und nicht weiter weißt musst du Exception UND StackTrace posten.

 

[wainmaster]

Sorry für meine ungenaue Ausdrucksweise..
Es ist diese Zeile die Probleme macht.

 

[dzim]

Ich habe irgendwie das Gefühl, das entweder dein Cast auf InfoMessageController falsch ist (wenn z.B. im FXML ein anderer Controller steht), oder du im FXML überhaupt keinen Controller verlinkt hast.

Sorry, aber ich verwende diese Methode ein FXML zu laden in OSGi um einen IEclispeContext in den Controller zu injecten und das klappt immer.

Mach mal einfach folgendes [c]System.err.println(fxmlLoader.getController());[/c] und sag uns mal, was da für ein Ergebnis kommt. Ist es null, dann hast du keinen Controller im FXML verlinkt. Ist es was anderes, musst du hier mal schauen, welcher Controller es wirklich ist!





BTW: Nutzt du in deinem FXML ResourceBundles (also internationalisierte Zeichenketten)?