Discussion :

[lvr]

Bonjour,
J'ai une méthode qui crée un gros objet data et son pendant graphique et qui retourne le pendant graphique.
Pour faire ça j'ai créé une sous-classe du SwingWorker avec une variable public pour pouvoir récupérer mon objet graphique.
Donc dans le doInBackground() je fais la création de l'objet data et dans le done() je fais la création de l'objet graphique sur base de l'objet data récupéré via get().

Quand je lance le execute() je voudrais pouvoir être sûr d'attendre la complète exécution du doInBackground et du done afin de pouvoir récupérer mon objet graphique.

Donc la question: est-ce que le execute() équivaut à un invokeLater ou à un invokeAndWait ? Si c'est à un invokeLater, comme forcer d'attendre la fin de l'exécution du done() (ou son annulation) ?

Merci.

[tchize_]

le swingworker est destiné à faciliter la création de tache longues de calcul qui mettent à jour occasionellement l'UI. Il tourne dans son propre thread, pas via le thread de l'EDT!. Si t'as besoin d'attendre sa fin, il faut se questionner sur l'utilité alors du swingworker. Tu peux aussi bien tout faire dans ton thread courant en faisant tes update UI via invokeLater ou invokeAndWait.

[adiGuba]

Donc la question: est-ce que le execute() équivaut à un invokeLater ou à un invokeAndWait ? Si c'est à un invokeLater, comme forcer d'attendre la fin de l'exécution du done() (ou son annulation) ?

Aucun des deux ! invokeLater et invokeAndWait servent à exécuter du code dans l'EDT, et c'est l'inverse que l'on souhaite faire avec un SwingWorker.

Le execute() lance le SwingWorker, c'est à dire :

• La méthode doInBackground() qui est exécutée dans un thread à part.
• Puis une fois fini, la méthode done() est exécutée dans l'EDT via invokeLater()...

Tu veux afficher ton objet graphique à la fin du traitement c'est ca ? Dans ce cas il suffit tout simplement de le faire dans la méthode done()...


a++

[lvr]

En fait j'ai une méthode qui duplique une fenêtre de données.
Il y a plusieurs moyen de l'utiliser (via un menuitem, via drag&drop+touche CTRL, ...).
Donc ce que je fais c'est une méthode qui duplique mais n'affiche pas, à charge du menuDuplication_actionPerformed(..), itemDropped(..) de l'appeller, de récupérer l'instance de la fenêtre et de la positionner là où ils veulent.

Cette duplication est composée de deux "gros" traitements: un traitement data (duplication des datas) que je fais dans le doInBackground et un traitement graphique (création d'une fenêtre avec pas mal de composants) que je fais dans le done. Après quand les deux process sont terminés, j'aimerais récupérer l'instance de la fenêtre pour la retourner à l'appellant.

Au début j'avais créé une Thread pour la partie data, qui à la fin lance le SwingUtilies.invokeAndWait pour la partie graphique. Je trouvais le SwingWorker plus propre et plus lisible, d'où mon changement de cap. Si retourne vers ce code d'origine, je ferais alors

Code :

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

1
2
3
4
 
mythread.run();
mythread.join();
return mythread.fenetredupliquee;

mais ça veut dire que je dois faire remonter "fenetredupliquee" du runnable d'invokeAndWait au runnable de la Thread, ... (
Qu'est ce que vous pensez de cette solution ?

[tchize_]

En fait j'ai une méthode qui duplique une fenêtre de données.
Il y a plusieurs moyen de l'utiliser (via un menuitem, via drag&drop+touche CTRL, ...).
Donc ce que je fais c'est une méthode qui duplique mais n'affiche pas, à charge du menuDuplication_actionPerformed(..), itemDropped(..) de l'appeller, de récupérer l'instance de la fenêtre et de la positionner là où ils veulent.

Cette duplication est composée de deux "gros" traitements: un traitement data (duplication des datas) que je fais dans le doInBackground et un traitement graphique (création d'une fenêtre avec pas mal de composants) que je fais dans le done. Après quand les deux process sont terminés, j'aimerais récupérer l'instance de la fenêtre pour la retourner à l'appellant.

C'est pas possible avec un swingworker pour plusieurs raisons. D'abord ca veut dire que tu bloque l'appelant pour faire ton traitement lourd, hors l'appelant est sur le thread de l'EDT puisque c'est une réaction à un event -> tu va givrer ton affichage. Ensuite le done d'un swingworker est fait aussi sur le thread de l'EDT que tu occupe, ce qui pose problème, car il ne sera exécuté que *après* que tu aie retourné à l'appelant (car traité en invokeLater).

Ta structure, si c'est à l'appelant de positionner la fenetre, devrait etre ceci:
1) création de la fenetre, lien vers des données vide
2) retour à l'appelant, en parallèle, le swingworker démarre
3) l'appelant positionne la fenetre, en parallèle, tes données sont dupliquée
4) l'appelant a fini le boulot, un peu plus tard ton done() rempli la fenetre avec les données dupliquées et la rend visible.

[lvr]

C'est pas possible avec un swingworker pour plusieurs raisons. D'abord ca veut dire que tu bloque l'appelant pour faire ton traitement lourd, hors l'appelant est sur le thread de l'EDT puisque c'est une réaction à un event -> tu va givrer ton affichage. Ensuite le done d'un swingworker est fait aussi sur le thread de l'EDT que tu occupe, ce qui pose problème, car il ne sera exécuté que *après* que tu aie retourné à l'appelant (car traité en invokeLater).

Ok J'avais pensé les chose sous cet angle.
Ta suggestion remet pas mal de chose en question dans ma manière très séquentialisée de procéder, mais cela me semble le plus logique. Si je n'arrive pas à le faire comme tu suggères sans trop démonter mon code, je pense que passer un paramètre ou petite classe à appeller dans le done() quand la duplication de la fenêtre est faite ferait l'affaire aussi.

Donc si je comprends bien:
1) création de la fenetre, lien vers des données vide
Fait indirectement dans mon menu_ActionPeformed => fait dans l'EDT => ok
2) retour à l'appelant,
3) l'appelant positionne la fenetre,
Fait dans mon menu_ActionPeformed => fait dans l'EDT

2) en parallèle, le swingworker démarre
3) en parallèle, tes données sont dupliquée
Lancé depuis l'EDT dans une thread en dehors de l'EDT => ok
4) l'appelant a fini le boulot, un peu plus tard ton done() rempli la fenetre avec les données dupliquées et la rend
Retour à l'EDT => ok