Task Multithread, modifier les enfants d'une HBox

Bonjour,

Tout ce qui est fait dans la méthode call() se déroule dans le Thread de la Task. Autant dire que si tu essaie de modifier un noeud qui est déjà attaché dans une scène, tu cours a la catastrophe. si on omet la première ligne dont je ne sais pas trop ce qu'elle fait la (vu que juste après tu écrase la valeur par celle crée par la tache), ici, tu crée un noeud et le modifie alors qu'il n'est pas attache a une scène, c'est bien !

Ta Task est crée et exécutée par un Service, tu n'as pas a gérer un Thread a mano.

Ton code devrai plutôt être comme ceci : ici le callback onSucceed est appelé durant le JavaFX Application Thread, si la tache s'est correctement terminée.

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
25
26
27
28
29
30
31
32
33
 
final Service<HBox> taskProcessService = new Service<HBox>() {
  @Override
  protected Task<HBox> createTask() {
     return new Task<HBox>() {
    @Override 
        protected HBox call() throws Exception {
        try {
            //Calculs longs et complexe.
        } catch (Exception e) {
            log.error(e);
        }
 
        //reinit hbox
        HBox myHbox = new HBox();
 
                //ajout nodes à ma hbox
                addInfosToHbox(myHbox);    
 
        return myHbox;
     }
       };   
     }
  }
};
taskProcessService.setOnSucceed(new EventHandler<WorkerStateEvent>() {
  @Override
  public void handle(WorkerStateEvent event) {
    HBox hboxResultat = taskProcessService.getValue(); 
    // Attacher ça dans la scène.
  }
};
taskProcessService.start();

Si tu veux modifier des valeurs qui sont déjà dans la scène, il va falloir prendre un peu plus de précautions : pour éviter tout soucis, il faut faire la modif dans le JavaFX Application Thread.

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
25
26
27
28
29
 
final HBox hboxResultat = (HBox) vboxListInput .getChildren().get(indexHbox);
final Service<Void> taskProcessService = new Service<Void>() {
  @Override
  protected Task<Void> createTask() {
     return new Task<Void>() {
    @Override 
        protected Void call() throws Exception {
        try {
            //Calculs longs et complexe.
        } catch (Exception e) {
            log.error(e);
        }
                // ajout nodes à ma hbox
                // Comme la box est deja dans la scene, on met a jour son contenu dans le JavaFX Application Thread.
                Platform.runLater(new Runnable() {
                   @Override
                   public void run() {
                      addInfosToHbox(hboxResultat, <ce qu'il faut pour mettre a jour le contenu>);
                   }
                };    
 
        return null;
     }
       };   
     }
  }
};
taskProcessService.start();

A noter que c'est bien beau de catcher et de loger l'exception dans la méthode call(), mais si elle ne remonte pas plus haut, il n'est pas possible de déterminer si la tache/le service est dans un état failed.
Ca serait mieux de faire :

Code:

1
2
3
4
5
6
taskProcessService.setOnFailed(new EventHandler<WorkerStateEvent>() {
  @Override
  public void handle(WorkerStateEvent event) {
    log.error(taskProcessService.getException());
  }
};