Discussion :

[CobolProgrammator]

bonjour, dans ma classe Controleur, j'ai défini une méthode valider() qui est invoquée quand on veut valider les données saisies depuis x écrans (classes MonEcran1, MonEcran2 ...).

Elle lance un swingWorker, qui a une méthode Void doInBackGround() qui contient une méthode abstraite traitementLent(), laquelle est implémentée différemment selon les écrans.
Dans chaque écran, traitementLent() manipule des objets différents.
Et chaque écran implémente aussi une méthode miseAJourEdt(), exécutée depuis done() une fois traitementLent() terminé, et aura besoin de données calculées dans traitementLent().

Question : les objets calculés par traitementLent() doivent ils forcément être retournés par traitementLent() (+ ajout de get()) si je veux les utiliser dans le done() ? dans le cas MonEcran1, panel.add(graphe) plante parfois en null pointer.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
34
35
36
37
38
39
class Controleur{
   void valider(){
      new Worker(Void, Void).execute();
   }
}
 
public class Worker extends SwingWorker(Void,Void){
   @overide
   Void doInBackground(){
      traitementLent();
   }
   @overide
   Void done(){
      miseAJourEdt();
   }
}
 
public class MonEcran1 {
   jpanel Panel,JFreeChart graphe;
   @overide
   void traitementLent(){
      graphe = fabriqueGraphe();// traitement lent
   }
   @overide
   void miseAJourEdt(){
      panel.add(graphe);// traitement EDT
   }
}
public class MonEcran2 {
   ...
   @overide
   void traitementLent(){
      ...()// traitement lent
   }
   @overide
   void miseAJourEdt(){
      ...// traitement EDT
   }
}

[tchize_]

Question : les objets calculés par traitementLent() doivent ils forcément être retournés par traitementLent() (+ ajout de get()) si je veux les utiliser dans le done() ? dans le cas MonEcran1, panel.add(graphe) plante parfois en null pointer.

Tu fais comme tu veux, mais si tu ne retourne pas la valeur depuis le doInBackground() pour y accéder via get(), n'oublie pas que c'est alors à toi de gérer le fait que deux thread différent y accèdent. Nottement le fait que le thread de l'EDT, au moment de l'exécution du done() ne vois pas nécessairement les valeurs que doInBackground() a créé si elle ne sont pas volatile et si il n'y a pas eu de point de synchronisation.


au passage, done() retourne void, pas Void.

[CobolProgrammator]

merci Tchize_,

donc pas la peine d'appeler get() si je déclare volatile les variables calculées dans le traitement lent. Et je dois faire un point de synchronisation =>
1 - tu peux me dire en quoi ça consiste ? (exemple)
2 - quel est le fin mot de l'histoire : si on dit sur SwingWorker que le done() s'exécute dans l'EDT quand doInBackGround() est fini, alors on peut se passer du get() si l'objet "graphe" est déclaré au niveau de la classe (donc à l'extérieur de doInBackground()) non ?

je précise bien sûr que dans miseAJourEdt() qui est appelé depuis le done() je ne mets pas à jour l'objet "graphe" , je ne fais que le lire ou l'ajouter à un champ du panel.

[tchize_]

merci Tchize_,

donc pas la peine d'appeler get() si je déclare volatile les variables calculées dans le traitement lent. Et je dois faire un point de synchronisation =>

"Ou" pas "et"

1 - tu peux me dire en quoi ça consiste ? (exemple)

Tout ce qui invalide le cache mémoire du thread. Typiquement: les sleep, les synchronized, les io bloquants. Dans ton cas, un synchronized sur la methode.

2 - quel est le fin mot de l'histoire : si on dit sur SwingWorker que le done() s'exécute dans l'EDT quand doInBackGround() est fini, alors on peut se passer du get() si l'objet "graphe" est déclaré au niveau de la classe (donc à l'extérieur de doInBackground()) non ?

le done s'exécute dans le thread EDT, le doInBackground s'exécute dans un autre thread. Tu dois avoir la garantie que ce qu'a fait cet autre thread est visible du thread EDT. Les champs de classe non volatile ne le sont pas d'office. C'est du au optimisation au niveau de la mémoire entre les threads. Il faut un évènement qui dise au thread (ici EDT) attention, on va aller chercher des trucs qu'un autre thread a peut être modifié (ton champ graphe), donc au revoir les optimisation de cache, je veux la valeur qui sorte de la mémoire centrale, pas du cache CPU ou d'un registre.

[CobolProgrammator]

bon finalement en déclarant graphe volatile, j'ai eu un cas où ça m'a refait null pointer.
J'ai abandonné cette solution.
Puis j'ai ajouté un Thread.sleep(1) , comme tu suggérais pour synchroniser, à la fin du doInBackGround() , et pour l'instant ça n'a pas refait de null pointer.
Merci.

[tchize_]

nan mais le thread.sleep c'est n'importe nawak. Utilise le get alors