Discussion :

[pverley]

Bonjour,

Je ne sais pas comment gérer proprement les exceptions dans un SwingWorker. Je vous dis comment je fais et je soumets à vos critiques.

Situation : j'exécute une tâche de fond qui throws ses Exceptions.
Si aucune exception n'est levée pendant l'excécution de la tâche de fond je veux que s'exécute Action1 dans l'EDT, Action2 sinon.
La méthode get() me renvoie les exceptions générées dans doInBackGround() donc je le met en début de la procédure done() avec un bloc try/catch

Ca ressemble à ça:

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
 
class MonSwingWorker extends SwingWorker {
       @Override
       public Object doInBackground() throws Exception {
           // Execution d'une tâche qui fait remonter les exceptions.
           tache();
           return null;
       }
 
       @Override
       protected void done() {
           try {
              // Appelle la méthode get() pour voir si une exception a été levée
              get();
              // Si pas d'exception => Action1 
              Action1;
           } catch (Exception e) {
                  //Exception thrown => Action2
                  Action 2;
           }           
       }
   }

Est-ce là une utilisation correcte du SwingWorker et des Exceptions ?
Merci d'avance pour vos éclaircissments.

Philippe.

[adiGuba]

Salut,


Cela me parait tout à fait correct !

Perso je me suis même fait une classe fille qui gère tout cela et propose deux méthode onSuccess() et onFailure() à la place de done() :

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
public abstract class MySwingWorker<T, V> extends SwingWorker<T, V> {
 
	/*
	 * La méthode done() est final afin qu'elle ne soit pas redéfinie.
	 * On implémentera plutôt les méthodes onSuccess() et onFailure()
	 */
	@Override
	protected final void done() {
		try {
			onSuccess( get() );
		} catch (InterruptedException e) {
			onFailure(e);
		} catch (ExecutionException e) {
			onFailure(e.getCause());
		}
	}
 
	/**
         * Méthode appelée dans l'EDT lorsque la tâche s'est terminé correctement.
         * @param result La valeur de retour de la méthode doInBackground()
         */
	protected void onSuccess(T result) {
 
	}
 
	/**
         * Méthode appelée dans l'EDT lorsque la tâche a généré une exception.
         * @param result L'exception renvoyé par la méthode doInBackground()
         */
	protected void onFailure(Throwable t) {
 
	}
 
}

a++

[pverley]

Bonjour,

Merci pour ta réponse.
Ca me plait, c'est bien clair et très propre avec ces deux méthodes onSuccess() et onFailure() ... adopté

A +
Merci. Philippe.