Fermer l'interface courante à l'execution d'une autre interface

Version imprimable

Bonjour,
J'ai crée deux interface en JavaFX. Je voudrais fermer l'interface courante quand je clic sur un bouton que ouvre l'autre interface.
J'ai utilisé ça:

Code:

register_btn.setOnAction(e -> primaryStage.close());

mais je reçois l'erreur: java.lang.NullPointerExceptionVoila mon code:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 @FXML private Button register_btn; public void start(Stage primaryStage) { try { //BorderPane root = new BorderPane(); Parent root = FXMLLoader.load(getClass().getResource("Main.fxml")); Scene scene = new Scene(root,1118,628); scene.getStylesheets().add(getClass().getResource("application.css").toExternalForm()); primaryStage.setScene(scene); primaryStage.show(); register_btn.setOnAction(e -> primaryStage.close()); } catch(Exception e) { e.printStackTrace(); } }
Pouvez-vous m'aider svp?
Merci

08/03/2017, 23h11
bouye
Si register_btn est null, ce que tu aurais pu quand même vérifier par toi-même avec un débogueur, c'est probablement que :
1. tu as oublié de déclarer son fx:id dans le FXML (ou que tu l'as mal écrit en faisant une faute et donc qu'il contient autre chose que "register_btn").
2. Ou alors c'est que tu as deux instances du contrôleur : celle qui est lancée par la méthode Application.launch() et celle créée automatiquement par FXMLLoader.load(). Or l'injection des variables n'a eut lieu que dans la seconde instance.
Ici ça semble être l'option 2 (ton application est ton contrôleur donc tu as deux instances créées donc la 1ere a le bouton qui est null puisque l'injection n'a jamais eut lieu).
08/03/2017, 23h56
zsoufianz

Merci pour ta réponse.
Oui, la valeur de register_btn est null. le bouton fonctionne bien j’exécute la deuxième fenêtre mais j'arrive pas à fermer l'ancienne.
Je ne sais pas comment résoudre ce problème ...

Comme je te lai dit en option 2 tu as sans doute 2 instances de ton contrôleur. Il te faut donc modifier ton code pour n'en avoir qu'une seule. Déjà ça implique de ne plus utiliser la méthode statique (cette méthode est MAL) pour charger le FXML mais d'utiliser la méthode d'instance a la place (cette méthode est BIEN) :

Soit injecter l'application comme contrôleur dans le FXML loader :

Code:

1
2
3
4
FXMLLoader fxmlLoader = new FXMLLoader(getClass().getResource("Main.fxml"));
fxmlLoader.setController(this);
Parent root = fxmlLoader.load();
[...]

Soit découpler la partie application de la partie contrôleur. Et fournir au contrôleur un callback qu'il invoquera lorsque l'utilisateur clique sur le bouton. C'est un plus au niveau de l'architecture de l'application de bien découpler les rôles même si ça demande un peu plus de boulot.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
public class A extends Application {
    public void start(Stage primaryStage) {
        try {
            FXMLLoader fxmlLoader = new FXMLLoader(getClass().getResource("Main.fxml"));
            Parent root = fxmlLoader.load();
            B controller = fxmlLoader.getController();
            controller.setOnExit(() -> primaryStage.close());
            Scene scene = new Scene(root,1118,628);
            scene.getStylesheets().add(getClass().getResource("application.css").toExternalForm());
	    primaryStage.setScene(scene);
            primaryStage.show();		
        } catch(Exception e) {
            e.printStackTrace();
        }
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
public class B {
    @FXML
    private Button register_btn;
 
    private final ObjectProperty<Runnable> onExit = new SimpleObjectProperty(this, "onExit");
 
    public final void setOnExit(Runnable value) {
        onExit.set(value);
    }
 
    public final Runnable getOnExit() {
        return onExit.get();
    }
 
    public final ObjectProperty<Runnable> onExitProperty() {
        return onExit;
    }
 
    @FXML
    private void onRegister_btn() {
        Optional.ofNullable(getOnExit())
            .ifPresent(Runnable::run);
    }
}

Code:

1
2
fx:controller="B">
    <Button id="register_btn" fx:id="register_btn" onAction="#onRegister_btn">

merci pour ta réponse.
est ce que je peux affecter deux actions au boutton "register_btn"? (par ce que il ouvre déjà la deuxième fenêtre)
J'ai un erreur au niveau du fichier FXML quand j'ai ajouté la ligne que tu m'avais dis:
Code:

1 2 3 4 5 6 7 8 <Button fx:id="register_btn" layoutX="377.0" layoutY="391.0" mnemonicParsing="false" onAction="#NewDoctor" prefHeight="39.0" prefWidth="193.0" styleClass="register_btn" text="REGISTER"> <font> <Font name="System Bold" size="18.0" /> </font> </Button> fx:controller="B"> <Button id="register_btn" fx:id="register_btn" onAction="#onRegister_btn"> </Button>
Je reçois:
javafx.fxml.LoadException:
.....
Caused by: java.lang.IllegalArgumentException: Unable to coerce fx:controller="B"> to class javafx.scene.Node.

09/03/2017, 09h59
bouye

Ben oui c'est normal ton FXML n'est plus un fichier XML valide du coup. L'attribut fx:controller est un attribut de l'élément racine du fichier.

Non tu peux affecter une seule action à un bouton (donc une seule fonction dans le contrôleur). Mais cette action peut elle invoquer plusieurs fonctions / callback bien sur (autant que tu veux, c'est toi le programmeur, tu fais comme bon te semble).
09/03/2017, 11h09
zsoufianz

Alors, c'est quoi la solution d'après toi pour fermer la fenêtre ancienne?
10/03/2017, 00h25
bouye

Ben invoquer la méthode appropriée depuis la méthode qui permet de faire réagir le bouton (ou depuis le callback qui est associée a cette dernière) :weird:
C'est quoi le soucis ??????????????????