SwingWorker et Exception

Version imprimable

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:

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.

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:

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++

22/08/2007, 09h57
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é :yaisse2:

A +
Merci. Philippe.