menu #43

Merged
Mat_02 merged 20 commits from menu into master 2023-05-09 15:52:14 +02:00
Showing only changes of commit af3489d078 - Show all commits

View File

@ -24,14 +24,15 @@ public class Controller extends Application {
root = new MenuAcceuil();
stage = primaryStage;
stage.setTitle("ROAD TO MASTER");
stage.setScene(new Scene(root));
switchRoot(root);
Review

tu pourrais faire un appel à #L33 (switchRoot) dans un premier temps ca ne changer rien mais si on change la façon de changer de scene un moment ca sera alors plus cohérent!

tu pourrais faire un appel à #L33 (switchRoot) dans un premier temps ca ne changer rien mais si on change la façon de changer de scene un moment ca sera alors plus cohérent!
Review

en gros au lieu de faire

        stage.setScene(new Scene(root));

Tu pourrais faire

        switchRoot(root);

Ca ne changerait pas grand chose pour l'instant mais si plus tard dans le code on a besoin de changer le fonctionnement de Scene par expemple, alors ca serait plus simple à changer.

en gros au lieu de faire ```java stage.setScene(new Scene(root)); ``` Tu pourrais faire ```java switchRoot(root); ``` Ca ne changerait pas grand chose pour l'instant mais si plus tard dans le code on a besoin de changer le fonctionnement de Scene par expemple, alors ca serait plus simple à changer.
stage.show();
}
public static void switchRoot(Parent root){
stage.setScene(new Scene(root));
Scene scene = new Scene(root);
stage.setScene(scene);
}
public static void main(String[] args) {
tonitch marked this conversation as resolved
Review

pourquoi ?

pourquoi ?
launch();