JavaFX -> SocketIO -> Thread -> Update Label

[JNodex]

Heyho,

ich bin neu hier und ich hoffe Ihr könnt mir etwas helfen.
Da ich bisher kaum in Java programmiert habe komm ich an einem bestimmten Punkt nicht mehr weiter.
Ich habe einen Node.js server + Java Desktop Applikation. Ich kann zum Node.js server connecten + eine msg empfangen.
Diese printe ich momentan aus. Soweit passt das alles.
Aber nun zu meinem Problem ^^ wahrscheinlich Simpel aber ich check es nicht. Ich möchte den String den ich momentan habe nicht printen sondern in ein Label setzen.

Mein Socket wird über eine sperate Klasse SocketHandler gesteuert und in der Main Klasse durch Initialisierung ausgeführt:

//Handle thread
    private void sockThread (){
        mSocket.on("SocketID", new Emitter.Listener() {
             public void call(final Object... args) {
                Platform.runLater(new Runnable() {
                    public void run() {
                        JSONObject obj = (JSONObject) args[0];
                        String socketID;
                            try{
                            socketID = obj.getString("SocketID");
                        }catch (Exception e){
                            throw new RuntimeException(e);
                        }
                        System.out.println("Incoming: " + socketID);
                    }
                });
            }
        });
    }

void run() throws URISyntaxException {
        connect();
        sockThread();
    }

und in der Main:

public static void main(String[] args) {
        //launch the application
        SocketHandler socket = new SocketHandler();
        try{
            socket.run();
        }catch (URISyntaxException e){
            e.printStackTrace();
        }

        launch(args);
    }

Nun habe ich auch ein Label und einen Controller dafür:

 <Label fx:id="textView" layoutX="58.0" layoutY="224.0" prefHeight="57.0" prefWidth="548.0" text="Label" textFill="WHITE" />

@FXML
    private Label textView;

  public void setText (String string){
        textView.setText(string);
    }

So ich hoffe jemand kann mir helfen wie ich nun meinen String da rein bekomme ^^
Danke für eure Hilfe.

Gruß
JNodex

 

[Flown]

Dein sockThread braucht eine Referenz auf dein Controller um dann lustig: controller.setText(socketId); aufrufen zu können.

 

[JNodex]

 private Controller controller;

Das habe ich oben hinzugefügt und unter meinem System.out.println habe ich nun folgendes:

controller = new Controller();
controller.setText(socketID);

nun haut er mir aber einen Error raus:

Exception in thread "JavaFX Application Thread" java.lang.RuntimeException: java.lang.NullPointerException
at SocketHandler$1$1.run(SocketHandler.java:36)
at com.sun.javafx.application.PlatformImpl.lambda$null$173(PlatformImpl.java:295)
at java.security.AccessController.doPrivileged(Native Method)
at com.sun.javafx.application.PlatformImpl.lambda$runLater$174(PlatformImpl.java:294)
at com.sun.glass.ui.InvokeLaterDispatcher$Future.run(InvokeLaterDispatcher.java:95)
at com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
at com.sun.glass.ui.win.WinApplication.lambda$null$148(WinApplication.java:191)
at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.NullPointerException
at Controller.setText(Controller.java:32)
at SocketHandler$1$1.run(SocketHandler.java:34)


Ich hoffe ich stell mich grade nicht zu dämlich an ^^

 

[mrBrown]

Das innere des Setters solltest du in ein Platform.runLater kapseln - GUI-Änderungen dürfen nur durch den zuständigen Thread stattfinden, und damit "verschiebst" du es in diesen

Edit: Verlesen, sehe grad das machst du beim Empfangen direkt...

Du erstellst dir einen neunen Controller, der die FXML-Datei nicht kennt, deshalb sind die Felder auch nicht gesetzt - du solltest den entsprechend mit FXMLLoader laden, anstatt ihn mit new zu erzeugen

 

[JNodex]

Das innere des Setters solltest du in ein Platform.runLater kapseln - GUI-Änderungen dürfen nur durch den zuständigen Thread stattfinden, und damit "verschiebst" du es in diesen

Das hab ich doch:

 private void sockThread (){
        mSocket.on("SocketID", new Emitter.Listener() {
             public void call(final Object... args) {
                Platform.runLater(new Runnable() {
                    public void run() {
                        JSONObject obj = (JSONObject) args[0];
                        String socketID;
                            try{
                            socketID = obj.getString("SocketID");
                        }catch (Exception e){
                            throw new RuntimeException(e);
                        }
                        System.out.println("Incoming: " + socketID);
                        controller = new Controller();
                        controller.setText(socketID);
                    }
                });
            }
        });
    }

Oder meinst du was anderes ?!

 

[mrBrown]

Das hab ich doch:

Ist auch schon editiert gewesen - habs nicht da erwartet und deshalb auch nicht danach geguckt

 

[JNodex]

Ist auch schon editiert gewesen - habs nicht da erwartet und deshalb auch nicht danach geguckt

Okay ja ^^ gesehen
Ich hab dein Rat befolgt und folgendes gemacht:

 FXMLLoader loader = new FXMLLoader(
                                getClass().getResource(
                                        "sample.fxml"
                                )
                        );
                        try {
                            Pane pane = loader.load();
                            Controller controller = loader.<Controller>getController();
                            controller.setText(socketID);
                        } catch (IOException e) {
                            e.printStackTrace();
                        }

Nun bekomm ich gott sei dank keine Error meldung mehr aber es passiert auch nichts mit dem Label ^^

Also er geht in die funktion setText rein aber dort macht er scheinbar nichts

 

[mrBrown]

Das Label ist auch noch überhaupt nicht sichtbar, oder?

 

[JNodex]

Das Label ist auch noch überhaupt nicht sichtbar, oder?

Doch das Label ist sichtbar.
Fett und groß ^^

 <Label fx:id="textView" layoutX="58.0" layoutY="224.0" prefHeight="57.0" prefWidth="548.0" text="Label" textFill="WHITE" />

Wenn ich in die funktion:

  public void initialize () {
        // initialization here, if needed...
    }

in meinem controller gehe und dort setText("Hallo"); ausführen lasse funktioniert es komischer weise.

 

[mrBrown]

Du lädst die FXML auch direkt schon zum Programmstart, oder?

 

[JNodex]

Du lädst die FXML auch direkt schon zum Programmstart, oder?

Das ist die Main Class die schlussendlich alles ausführt:

public class Main extends Application {

    @Override
    public void start(Stage stage) throws Exception {
        // get the fxml file --> Frontend !
        Parent root = FXMLLoader.load(getClass().getClassLoader().getResource("sample.fxml"));
        stage.initStyle(StageStyle.UNDECORATED);
        Scene scene = new Scene(root);
        stage.setScene(scene);

        //render the scene
        stage.show();
    }

    public static void main(String[] args) {
        //launch the application
        SocketHandler socket = new SocketHandler();
        try{
            socket.run();
        }catch (URISyntaxException e){
            e.printStackTrace();
        }
        launch(args);
    }
}

und im SocketHandler lade ich die FXML im sockthread dort wo vorher System.out.println(); stand

 

[mrBrown]

Du lädst die FXML zwei mal (und damit auch zwei Controller) - angezeigt wird das in Main geladene, das in deinem SocketHandler geladene wird nie angezeigt, und deshalb siehst du die Änderungen an der Instanz auch nicht.

Du musst die in Main geladene Instanz des Controllers an den SocketHandler übergeben

 

[JNodex]

Du lädst die FXML zwei mal (und damit auch zwei Controller) - angezeigt wird das in Main geladene, das in deinem SocketHandler geladene wird nie angezeigt, und deshalb siehst du die Änderungen an der Instanz auch nicht.

Du musst die in Main geladene Instanz des Controllers an den SocketHandler übergeben

Okay, könntest du mir evtl nen tipp geben wie ?

 

[JNodex]

Okay passt hab es danke euch

Lösung:

 FXMLLoader loader = new FXMLLoader(getClass().getClassLoader().getResource("sample.fxml"));
        Parent root = loader.load();
        Controller controller = loader.getController();