Aufruf einer Methode einer anderen Klasse

[didi577]

Hallo,

ich grübele jetzt schon lange über diesem Problem:

Ich habe eine JavaFx GUI mit 2 Fenstern. Ich möchte aus Fenster 1 einen String an Fenster 2 übergeben. Der Code aus Fenster 1:

twc.setWidgets(lvEmploye.getSelectionModel().getSelectedItem()
                   .getFirstname());

Der Code aus Fenster 2 wo der Wert ankommen soll:

public void setWidgets(String name) {

       lblNameTarget.setText(name);

   }

Am Ende soll der Wert:

lvEmploye.getSelectionModel().getSelectedItem()
                   .getFirstname()

aus Fenster 1 im Label aus Fenster 2 stehen:

lblNameTarget.setText(name)

wer kann mir weiter helfen?

 

[mrBrown]

wer kann mir weiter helfen?

Wobei denn? Der Code steht da ja schon fertig...

 

[didi577]

wenn ich die Methode aufrufe erhalte ich eine NPE mit Verweis auf diese Stelle

    twc.setWidgets(lvEmploye.getSelectionModel().getSelectedItem()
                   .getFirstname());

 

[Robat]

Dann am besten mal schauen ob twc oder lvEmploye null sind bzw sicherstellen dass das ganze nur aufgerufen wird wenn auch wirklich ein Item in der Liste ausgewählt ist. Sonst kann auch lvEmploye.getSelectionModel().getSelectedItem() null werden.

 

[didi577]

twc

scheint das Problem zu sein. Ich habe in der aufrufenden Klasse diese Zeile:

private TargetWindowController twc;

TargetWindowController ist die Klasse in der die Methode:

public void setWidgets(String name) {

       lblNameTarget.setText(name);

   }

steht

 

[Robat]

private TargetWindowController twc;

Wenn du nur das drinne stehen hast ist klar dass er die NPE wirft..
Du initialisierst es ja nirgends.

 

[didi577]

ok, ich habe jetzt in der initialize Methode die Zeile:

TargetWindowController twc = new TargetWindowController();

die NPE ist damit weg, Danke

jetzt habe ich noch eine NPE mit Verweis auf diese Zeile:

lblNameTarget.setText(name);

in der Methode:

public void setWidgets(String name) {

       lblNameTarget.setText(name);

   }

??

 

[JStein52]

dasselbe in grün vermutlich. lblNameTarget vermutlich null bzw. nicht initialisiert.

 

[didi577]

der String "name" wird jetzt mit dem gewünschten Wert belegt

die NPE bezieht sich also auf das Label dass scheinbar zum Zeitpunkt noch nocht da ist

wo muss ich jetzt noch umbauen?

 

[VfL_Freak]

Moin,

dasselbe in grün vermutlich. lblNameTarget vermutlich null bzw. nicht initialisiert.

VG Klaus

 

[didi577]

lblNameTarget vermutlich null bzw. nicht initialisiert.

das es sich hier um eine FXML Anwendung handelt wird das Label "lblNameTarget" doch beim Fensterstart initialisiert?! Wenn ich die oben beschriebenen Methoden zum setzen des Textes auskonmmentiere ist das Label beim Fensterstart mit dem default Wert aus dem Scene Builder auch vorhanden. Hat noch jemand ei e Idee??

 

[VfL_Freak]

Moin,

wird das Label "lblNameTarget" doch beim Fensterstart initialisiert

wird wirklich initialisiert oder nur deklariert??
Poste doch einfach den entsprechenden Code ....

VG Klaus

 

[Robat]

Kann es sein, dass du die @FXML Annotation vergessen hast?
Wie sieht denn die fx:id des Labels aus?

 

[didi577]

Wie sieht denn die fx:id des Labels aus?

lblNameTarget

Kann es sein, dass du die @FXML Annotation vergessen hast?

@FXML
private Label lblNameTarget;

die Zeile aus der fxml Datei:
<Label fx:id="lblNameTarget" text="Name" GridPane.columnIndex="1" />

 

[Robat]

Okay das scheint zu passen.
Wie lädst du denn deinen FXML Datei? Kann es sein dass du 2 Instanzen deiner Controller-Klasse hast und die Variable daher null ist?

 

[didi577]

in der Klasse MainWindowController wird bei Klick auf einen Button folgendes ausgeführt:

btnAddTarget.setOnAction(ev -> {

           myTeam.TargetWindow();
           
           twc.setWidgets(lvEmploye.getSelectionModel().getSelectedItem()
                   .getFirstname());
           
       });

myTeam.TargetWindow(); lädt die FXML:

public void TargetWindow() {

       try {
           FXMLLoader loader = new FXMLLoader(
                   MyTeam.class.getResource("/view/TargetView.fxml"));
           GridPane pane = loader.load();

           Scene scene = new Scene(pane);
           Stage targetWindowStage = new Stage();
           TargetWindowController targetWindowController = loader
                   .getController();
           targetWindowController.setMyTeam(this, targetWindowStage);

           targetWindowStage.setScene(scene);
           
           targetWindowStage.setTitle("Zielvereinbarungen");
           targetWindowStage.initOwner(primaryStage);
           targetWindowStage.initModality(Modality.APPLICATION_MODAL);
           targetWindowStage.show();

       } catch (IOException e) {
           // TODO Auto-generated catch block
           e.printStackTrace();
       }

   }

twc.setWidgets(lvEmploye.getSelectionModel().getSelectedItem()
.getFirstname()); übergibt den String an das Label in der Klasse TargetWindowController:

public void setWidgets(String name) {
       
       lblNameTarget.setText(name);
       System.out.println(name);
   }

die Variable name hat den richtigen Wert

 

[didi577]

kann es sein dass ich lblNameTarget.setText(name); irgendwie in der @FXML inizialize() Methode unterbringen muss ??

 

[didi577]

kann mir hier noch jemand helfen??

 

[mrBrown]

Du solltest nirgendwo mit new einen neuen TargetWindowController erstellen, sondern den in TargetWindow initialisierten benutzen.

 

[didi577]

ok, was heißt das für meinen Code? Was kann ich tun?

 

[mrBrown]

TargetWindowController als Instanzvariable setzten und einen Getter dafür bereit stellen, und diesen dann in der anderen Klasse aufrufen

 

[didi577]

ich habe jetzt in der MainWindowController:

private TargetWindowController twc;
...
...
btnAddTarget.setOnAction(ev -> {

           myTeam.TargetWindow();
          
           getTwc().setWidgets(lvEmploye.getSelectionModel().getSelectedItem()
                   .getFirstname());
           
           
       });

in der TargetWindowController:

public void setWidgets(String name) {
       
       lblNameTarget.setText(name);

   }

ich bekomme weiter eine NPE in dieser Zeile:

getTwc().setWidgets(lvEmploye.getSelectionModel().getSelectedItem()
                   .getFirstname());

 

[didi577]

ich glaube du meintest es anders, TargetWindowController:

private TargetWindowController twc;
...
...
public TargetWindowController getTwc() {
       return twc;
   }

MainWindowController:

private TargetWindowController twc;
...
...
twc.getTwc().setWidgets(
                   lvEmploye.getSelectionModel().getSelectedItem()
                           .getFirstname());

trotzdem die NPE:

twc.getTwc().setWidgets...

 

[mrBrown]

Ja, ich mein was anderes. In deinem Code benutzt du ja auch nirgendwo Getter, sondern führst nur neue Variablen ein, die logischerweise null sind.

Aber bei den kurze Codestückchen kann man nicht wirklich was sagen...

 

[didi577]

hier mal mein Ziel wo ich hin möchte:


aus der Listview des Hauptfensters soll der markierte Name "Patrick S" den Labeltext "Name aus LV" im Fenster "Zielvereinbarungen" ersetzen, das selbe dann für das Jahr "2017" im Hauptfenster nach "Jahr aus cb" . Das Fenster "Zielvereinbarungen" öffnet sich wenn ich im Hauptfenster auf den Button "Ziel erfassen" klicke

wie baue ich das am besten?

 

[didi577]

hier mein aktueller Code mit Bezug zum Problem:

public class MainWindowController {
   private String s;
@FXML
   private void initialize() {
btnAddTarget.setOnAction(ev -> {

           myTeam.TargetWindow();
           setS(lvEmploye.getSelectionModel().getSelectedItem()
                   .getFirstname());
System.out.println(s);
}

public String getS() {
           
       return s;
   }

   public void setS(String s) {
       this.s = s;
   }
}

der String soll als Text im Label der folgenden Klasse landen:

public class TargetWindowController {

   @FXML
   private Label lblNameTarget;

private MainWindowController mwc;

@FXML
   private void initialize() {

       setTextlbl();
   }
public void setTextlbl() {
       MainWindowController mwc = new MainWindowController();
       lblNameTarget.setText(mwc.getS());
   }

es gibt keinen Fehler aber "mwc.getS()" liefert null und somit ist der Text im Label leer

"System.out.println(s);" gibt den Namen richtig aus

vielleicht kann mir jetzt jemand helfen

 

[VfL_Freak]

Moin,
wenn das ist , wie du schreibst, dann klingt das für mich so, als würde "getS" in "setTextlbl" VOR dem 'initialize' in 'MainWindowController' aufgerufen!
Versuch' mal den String zu initialisieren:

private String s = "Test";

VG Klaus

 

[didi577]

Hi,

public class MainWindowController {
   
   private String s = "Test";
...

mwc.getS() liefert jetzt "Test"

 

[Robat]

Ausgehend von Post#16 und Post#7 kann man sagen dass der Fehler immer noch darin liegt, dass du 2 Instanzen deiner Controller Klasse hast.
In deiner TargetWindow() Methode lädst du die FXML Datei und gleichzeitig auch den Controller. Die Instanz existiert also. Gleichzeit sagst du aber, dass du in der initialize() Methode mittels TargetWindowController twc = new TargetWindowController(); eine weiter Instanz erstellst. Das sollte man niemals tun! Entfern die Zeile aus der initialize() Methode und ruf die setWidgets(String s) Methode aus deiner TargtWindow() Methode auf. Irgendwie so:

public void TargetWindow()
{
     FXMLLoader loader = ...
     ...
     TargetWindowController targetWindowController = loader.getController();
     ....
     targetWindowController.setWidgets(lvEmploye.getSelectionModel().getSelectedItem().getFirstname());
     ...
}

 

[didi577]

Das sollte man niemals tun!

die habe ich nicht mehr drin, also auch nicht aus den Posts von heute

 

[Robat]

Sorry das hatte ich übersehen.
Dennoch solltest du deine anfängliche Idee mMn nicht verwerfen.
In deiner TargetWindowController Klasse hast du Methoden um die Werte in die Labels zu schreiben. (bspw. setWidgets(String s)).
Sobald du das FXML File geladen hast holst du dir dein Controller (wie du es bereits in Post#16 gemacht hast) und rufst die Methode setWidgets(..) auf. Erst danach darfst du die show() Methode aufrufen. Hier mal ein Beispiel aus einen meiner Projekte:

try
{
    FXMLLoader loader = new FXMLLoader();
    loader.setLocation(Manager.class.getResource("view/PlayerEditDialog.fxml"));
    AnchorPane page = (AnchorPane) loader.load();

    Stage dialogStage = new Stage();
    dialogStage.setTitle("Spieler bearbeiten");
    dialogStage.initModality(Modality.WINDOW_MODAL);
    dialogStage.initOwner(primaryStage);
    Scene scene = new Scene(page);
    dialogStage.setScene(scene);

    PlayerEditDialogController c = loader.getController();
    c.setDialogStage(dialogStage);
    c.setPlayer(player); // <-- das könnte deine setWidgets(String s) Methode sein

    dialogStage.showAndWait();

    return c.isOkClicked();
}
catch(IOException e)
{
    e.printStackTrace();
    return false;
}

 

[didi577]

ok Danke ich probier das mal

 

[didi577]

ich glaube ich habe es

ich rufe

myTeam.TargetWindow(lvEmploye.getSelectionModel().getSelectedItem()
                   .getFirstname());

mit dem gewünschten String als Parameter auf

public void TargetWindow(String name) {

       try {
           FXMLLoader loader = new FXMLLoader(
                   MyTeam.class.getResource("/view/TargetView.fxml"));
           GridPane pane = loader.load();
           
           Scene scene = new Scene(pane);
           Stage targetWindowStage = new Stage();
           TargetWindowController targetWindowController = loader
                   .getController();
           targetWindowController.setMyTeam(this, targetWindowStage);
           Label lblNameTarget = new Label();
           lblNameTarget.setText(name);
           pane.add(lblNameTarget, 1, 0);
           targetWindowStage.setScene(scene);
           
           targetWindowStage.setTitle("Zielvereinbarungen");
           targetWindowStage.initOwner(primaryStage);
           targetWindowStage.initModality(Modality.APPLICATION_MODAL);
           targetWindowStage.show();

       } catch (IOException e) {
           // TODO Auto-generated catch block
           e.printStackTrace();
       }

   }

in der Methode erzeuge ich ein neues Label, setze den Text aus dem Parameter und platziere das Label an die richtige Stelle, das Ergebnis passt

keine Ahnung ob das so "legal" ist

 

[Robat]

MMn eher ein Workaround als eine Lösung. So kannst du ja jetzt nicht mehr im Controller auf das Label zugreifen.

 

[didi577]

das merke ich auch gerade

 

[didi577]

Label lblNameTarget = new Label();

das muss ich also ersetzen und auf das vorhandene Laben bezug nehmen

hast du eine Idee?

 

[Robat]

hast du eine Idee?

Ja in dem du es so machst wie in #31 geschrieben. Einfach in deine TargetWindowController eine Methode die den Text des Labels verändert und die dann wie oben gezeigt aufrufen. (siehe setPlayer(..) Methode).

 

[didi577]

ok, habe umgebaut:

public class TargetWindowController {
...
...
public void setWidgets(String name) {
       
       lblNameTarget.setText(name);
   }
}

und

public class MyTeam extends Application {
...
...
public void TargetWindow() {

       try {
           FXMLLoader loader = new FXMLLoader(
                   MyTeam.class.getResource("/view/TargetView.fxml"));
           GridPane pane = loader.load();

           Scene scene = new Scene(pane);
           Stage targetWindowStage = new Stage();
           TargetWindowController targetWindowController = loader
                   .getController();
           targetWindowController.setMyTeam(this, targetWindowStage);
           targetWindowController.setWidgets(LvEmploye().getSelectionModel().getSelectedItem().getFirstname());

           targetWindowStage.setScene(scene);

           targetWindowStage.setTitle("Zielvereinbarungen");
           targetWindowStage.initOwner(primaryStage);
           targetWindowStage.initModality(Modality.APPLICATION_MODAL);
           targetWindowStage.show();

       } catch (IOException e) {
           // TODO Auto-generated catch block
           e.printStackTrace();
       }

   }

beim Ausführen:
Exception in thread "JavaFX Application Thread" java.lang.Error: Unresolved compilation problem:
The method LvEmploye() is undefined for the type MyTeam

at controller.MyTeam.TargetWindow(MyTeam.java:181)

klar, lvEmploye kommt aus der Klasse MainWindowController

habe probiert:

public MainWindowController mwc;
...
targetWindowController.setWidgets(mwc.getLvEmploye().getSelectionModel().getSelectedItem().getFirstname());

aber NPE auf targetWindowController.setWidgets(mwc.getLvEmploye().getSelectionM...

ich denke wir sind nah am Ziel...was fehlt noch

 

[Robat]

Übergib doch lvEmploye als Parameter so wie den String in Post#33

 

[didi577]

Danke für deine Hilfe und Geduld...es funktioniert jetzt