JavaFX Warum wird hier eine NullPointer-Exception geworfen?

BigMemo007

Aktives Mitglied
Hallo liebes Forum,

ich habe folgenden Code mit fünf Klassen:

Main
Java:
public class RunEinnAusgApp extends Application {

    private Stage stage;
    private ConMain conMain;
    private ConBook conBook;

    
    public RunEinnAusgApp() {
        conMain = new ConMain(this);
        conBook = new ConBook(this);
        
    }
    
    @Override
    public void start(Stage primaryStage) {
        try {
            stage = primaryStage;
            stage.setScene(conMain.getgMain().getSceneGMain());
            primaryStage.show();
            
        } catch(Exception e) {
            e.printStackTrace();
        }
    }
    
    public static void main(String[] args) {
        launch(args);
    }
    
    public Stage getPrimStage() {
        return stage;
    }
    
}//RunEinnAusgApp


Controller 1
[CODE lang="java" highlight="17"]public class ConMain implements EventHandler<ActionEvent>{

private GMain gmain;
private RunEinnAusgApp runApp;


public Control(RunEinnAusgApp runApp) {
gMain = new GMain(this);
this.runApp = runApp;

}

@Override
public void handle(ActionEvent event) {
Object o = event.getSource();
if (o == gMain.getBtnBook()) {
runApp.getPrimStage().setScene(runApp.getConBook.getScenegBook());
System.out.println("Button Buchen geklickt");
}

}//handle

public GMain getgMain() {
return gMain;
}
[/CODE]


Controller 2
Java:
public class ConBook implements EventHandler<ActionEvent>{

    private GBook gBook;
    private RunEinnAusgApp runApp;
    
    
    public Control(RunEinnAusgApp runApp) {
        gBook = new GBook(this);
        this.runApp = runApp;
        
    }

    @Override
    public void handle(ActionEvent event) {
        Object o = event.getSource();
        if (o == gMain.getBtnBack()) {
            runApp.getPrimStage().setScene(runApp.getConMain.getScenegBook());
            System.out.println("Button ZURÜCK geklickt");
        }
        
    }//handle

    public GBook getgBook() {
        return gBook;
    }

Gui Main
Java:
public class GMain {
    BorderPane layout;
    Scene sceneGMain;
    
    private Button btnAnalyze;
    private Button btnBook;
    private Button btnEdit;
    private Control conMain;
    
    public GMain(Control conMain) {
        layout = new BorderPane();
        sceneGMain = new Scene(layout, 800,600);
        this.conMain = conMain;
        
        btnAnalyze = new Button(ConVar.CVMain.btnTxtAnalyze);
        btnBook = new Button(ConVar.CVMain.btnTxtBook);
        btnBook.setOnAction(this.conMain);
        btnEdit = new Button(ConVar.CVMain.btnTxtEdit);
    
        setLayout();
                
    }//constr
    
    
    public void setLayout () {
        HBox btnHBox = new HBox(30);
        btnHBox.getChildren().addAll(btnAnalyze, btnBook, btnEdit);
        layout.setBottom(btnHBox);
        
    }

    public Scene getSceneGMain() {
        return sceneGMain;
    }

    public Button getBtnBook() {
        return btnBook;
    }

}//class

Gui Book
Java:
public class GBook {

    private BorderPane layout;
    private Scene sceneBook;
    private Control conMain;
    private final Button btnBack;   
    
    public GBook(Control conBook) {
        this.conBook = conBook;
        
        layout = new BorderPane();
        sceneBook = new Scene(layout, 800,600);

        Label l = new Label("Seite Gui Book");
        l.setFont(Font.font(100));
    
        layout.setCenter(l);
        
        btnBack = new Button("zurück zum Hauptfenster");
        btnBack.setOnAction(this.conMain);
        layout.setBottom(btnBack);
        BorderPane.setAlignment(btnBack, Pos.CENTER);   
        
    }//constr
    
    public Scene getScenegBook() {
        return sceneBook;
    }

    public Button getBtnBack() {
        return btnBack;
    }
    
}//GBook

Es wird eine NullPointer-Ecxeption ausgeworfen im Controller 1 Zeile 17, bei dem Wechselversuch der Scene.
Ich nehme an, dass es an der Erstellung der Objekte liegt. Aber ich bin irgendwie der Meinung, dass die Reihenfolge korrekt ist und eigentlich auch die Scenen erstellt sein müssten, bevor die handle-Methoden aufgerufen werden.

Wenn ich mein Programm umstelle und einen Controller nehme, statt wie hier zwei, dann klappt auch der Wechsel der Scnene wunderbar. Dort werden die beiden GUI-Objekte einwandrei erstellt und die getScene klappt wunderbar.

Aber wenn ich vom Controller 1 aus ein Objekt zurückgehe, also zur RunEinnAusgApp zurück und von dort Controler 2 aufrufe um vom dessen GUI-Objekt die Scene zu getten, dann gibt es einen NullPointer. Aber in der RunEinnAusgApp habe ich ja im Konstruktor die beiden Contoller erstellt und innerhalbe der Controller werden ja die GUI-Klassen-Objekte erstellt und darin jeweils die Scene's. Wo ist mein Denkfehler?
 
Ich habe eben die Controller über die init()-Methode in der RunEinnAusgApp-Klasse (meine Java-FX Main-Klasse) generieren lassen und es gibt keine Exception. Hmm... Wann wird was ausgeführt in der FX-Main? Ist folgende Reihenfolge richtig?

Konstruktor()
main()
init()
start()
stop()

Wenn ja, müsste doch oben auch alles richtig sein. Zuerst werden die Controller im Konstruktor erstellt und dann im Start ausgeführt und verändert. Oder werden die Objekte, die über die Konstruktoren erstellt werden, nicht vollständig erstellt? Etwas checke ich da grad überhaupt nicht.
 
Da ist aber eine NPE das geringste Problem - der Code ist ja schon syntaktisch falsch: ConBook/ConMain hat einen "Control"-Konstruktor, runApp.getConMain wird schwierig, es gibt in der Klasse keinen getConMain-Member.
Danke für die Infos. Das sind aber nur Fehler hier in meiner Frage im Forum. Ich habe die Klassen leider aus dem Kopf und schnell texten müssen. In meinem Eclipse hatte ich Syntaktisch und inhaltlich alles richtig. Sprich: Konstruktoren, Setter, Getter und alles andere war richtig. Trotzdem kam die NPE. Da ich nun alles auf einem Controller aufbaue, habe ich den alten Code nicht mehr. Deswegen aus dem Kopf.
 
Zuletzt bearbeitet:
Ok. Dann stell ich mal die Frage anders. Die Erstellung meiner conMain und conBook Objekte. Wären die 1zu1 identisch, wenn sie zum einen im Konstruktor der Klasse RunEinnAusgApp erstellt würden und zum anderen in der überschrieben init()-Methode?
 
Also da du nicht mit fxml zu arbeiten scheinst, wird ggf. einiges funktionieren. Aber ich würde klar dazu raten, sich das JavaFX richtig anzusehen und die jeweiligen Dinge im jeweiligen Event zu machen. Einiges gehört meiner Meinung nach einfach nicht in den Konstruktor.

Dann empfinde ich dein Design als recht fragwürdig. Da hängt alles voneinander ab. Deine App Klasse nutzt die zwei ConXXX Klassen. Diese haben aber auch eine klare Abhängigkeit zur App Klasse .. wobei es ja nicht dabei bleibt, denn die App Klasse kennt sogar die Details dieser Klassen, denn es wird ja auf das Element gMain zugegriffen. Und da das so ja nicht reicht, nutzen sich die beiden Controller gegenseitig....

Sorry, aber das ist in Meinen Augen erst einmal kein sauberes Design und sollte etwas überarbeitet werden....
 
Ok. Ich habe es wohl blöd gelöst. Mein Ziel ist es, nach Button-Klicks die Scene wechseln zu können. Und ich habe versucht die MVC umzusetzen (die Models habe ich noch nicht eingefügt). Der Button "Buchen" ist erstmal Test. Wie lasse ich in einem sauberen Code die Scene entsprechend in der primaryStage wechseln zu lassen? Ich habe 4 GUIs, d.h. eben auch 4 Scenes.
 

Zurück
Oben