Discussion :

[sebastien61]

Bonjour,

Je suis débutant en multi-threading, je programme en Java.
J'ai d'un coté 1 interface graphique, et d'autre coté un processus utilisateur. Dans l'interface graphique il y a 1 JProgressBar qui indique la progression du calcul utilisateur.

Je me suis fortement inspiré de l'exemple suivant :
http://rom.developpez.com/java-swingworker/

Pour faire simple, j'ai :

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
 
class MaClasse extends JPanel {
      public void okActionPerformed(ActionEvent e)
            MonSwingWorker sw = new MonSwingWorker();
            sw.execute();
 
            wait();
 
            // ici : le infos doivent etre collectées !
            Info informations = sw .getInformations();
 
            System.out.println(informations.size());
      }
 
     public class MonSwingWorker() extends SwingWorker {
           public void scan(fichier) // scan d'un fichier ligne par ligne
           {
                 for(int i=0 ; i<nbLigne ; i++)
                        setProgress(i/nbLigne);
           }
 
          protected Void doInBackground(){
                 scan(fichier);
                 notify();
          }
 
          protected void done() {// appelée à la fin
		 System.out.println("Done");
          }
     }
}

Alors mon problème est qu'il faut que sw ai collecté toutes les infos du fichier avant de continuer. Si je ne mets pas de notify et wait , la tâche est trop longue et les infos n'ont pas le temps d'etre calculées (informations.size() = 0)

Quand je mets notify et wait, la tâche calcule bien les info ("Done" affiché), mais le processus principal ne reprend pas la main (pas d'affichage de "informations.size()...". J'ai essayé avec un notifyAll(), mais le problème reste le même.

Avez vous une idée d'ou cela peut provenir ? Ca ne doit pas etre hyper compliqué, mais ça m'échappe...
J'ai trouvé pas mal de topic sur ce type de problème, mais peu sont dans le même cas que moi (parcours du fichier en entier indispensable pour poursuivre l'EDT)

Merci d'avance pour votre aide.

[DjGonk]

Au lieu de wait, est ce que tu as essayer une boucle while?

Code :

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

1
2
3
4
5
 
while(sw.isDone() == false){
      //TODO : A voir si tu fais un wait ou pas
      wait(10);
}

[fr1man]

Je ne comprends pas vraiment ton problème.
Si tu dois attendre que ton swingworker se termine pour utiliser tes Infos, pourquoi n'utilises tu pas la méthode done pour faire ton traitement dans l'EDT.
C'est fait pour cela.

[sebastien61]

Tout d'abord merci pour vos réponses.

@DjGonk : Oui j'avais essayé de faire ça, mais 2 cas surviennent :
1 - Si je mets le wait dans le while : Exception levée (e.getMessage = null !)
2 - Si je ne le mets pas, boucle infinie dans l'EDT et le swingWorker n'avance pas

@fr1man : Quand tu dis "La méthode done" c'est celle de SwingWorker ? Il faut que je mette a jour la barre d'avancement. Pourrais tu me donner davantage de détails stp ?

Désolé si je pose des questions idiotes... Merci d'avance en tout cas.

[adiGuba]

Salut,

Alors mon problème est qu'il faut que sw ai collecté toutes les infos du fichier avant de continuer. Si je ne mets pas de notify et wait , la tâche est trop longue et les infos n'ont pas le temps d'etre calculées (informations.size() = 0)

Quand je mets notify et wait, la tâche calcule bien les info ("Done" affiché), mais le processus principal ne reprend pas la main (pas d'affichage de "informations.size()...". J'ai essayé avec un notifyAll(), mais le problème reste le même.

Ton problème vient du fait que tu veux faire du traitement séquentiel, ce qui est une erreur en Swing (et avec la plupart des UIs évolué).

Bien sûr tu vas me dire qu'ici tu utilises bien un thread et que donc tu es en parallèle et non pas en séquentiel...
Le problème c'est que tu attends que le thread soit terminé pour continuer ton travail, donc cela revient à faire du séquentiel : a quoi te sert le thread puisque tu n'as pas de traitement en parallèle !!!



La solution c'est de faire de l'évenementielle. Dans ton listener tu ne doit pas attendre la fin de la tâche. Tu dois juste te contenter de mettre à jour l'affichage avant le lancement de la tâche (par exemple grisé le bouton ou affiché une progress bar).

Dans ton SwingWorker tu vas effectuer les traitements dans un nouveau thread, ce qui va permettre à l'EDT de continuer, de sortir du listener et de faire ce qu'il a a faire (mettre à jour l'affichage, traiter les autres evenements, ...)

Enfin, lorsque ton traitement est terminé, tu peux mettre à jour l'affichage via la méthode done().


Exemple :

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
40
41
42
class MaClasse extends JPanel {
 
	public void okActionPerformed(ActionEvent e) {
 
		// 1 - On est dans l'EDT
		// => On peut mettre a jour l'affichage avant le début de la tâche
 
		new SwingWorker<Info,Object>() {
			@Override
			protected Info doInBackground() throws Exception {
				// 2 - On est dans un thread 'en tâche de fond'
				// => On peut effecuter nos traitements lourds
 
				// ...
				scan(fichier);
 
				return getInformations(); // et eventuellement renvoyer un résultat 
			}
 
			@Override
			protected void done() {
				// 3 - On est dans l'EDT, et le traitement est fini
				// => On peut mettre à jour l'affichage avec les données
				try {
					// On récupère le résultat
					Info result = get();
 
					// Le traitement s'est bien passé
					// On peut afficher les résultats dans l'interface graphique
 
				} catch (InterruptedException e) {
					// Le thread a été interrompu
					// TODO
				} catch (ExecutionException e) {
					// La méthode doInBackground a généré une exception
					// TODO
				}
			}
		}.execute();
	}
 
}

a++

[sebastien61]

Ah ok, merci adiGuba, je viens de comprendre... Et ça marche !
Je savais que c'était pas super approprié ce que j'avais fait, mais je croyais que c'était indispensable pour pouvoir mettre à jour la progressbar. (le découpage en classes)
Ca fait un sacré bloc maintenant, mais bon du moment que ça marche...

Merci beaucoup à tous en tout cas.