Discussion :

[romuluslepunk]

Bonjour,

Je developpe un programme style traitement de texte.
J'ai des classes commande qui etendent SwingWorker pour gérer les événement de l'interface graphique :
- OuvrirFichierCommande extends SwingWorker
- EnregistrerFichierCommande extends SwingWorker

Dans doInBackground, je fais le traitement, par exemple ouvrir le fichier et le meme en memoire. Et dans done, le raffraissement de l'interface graphique.
Jusque là je pense que je fais celà correctement et que j'ai bien compris le principe de SwingWorker.

J'aimerai pouvoir lancé EnregistrerFichierCommande depuis le doInBackground de OuvrirFichierCommande si le fichier en cours d'édition n'a pas été enregistré.
Vu que je débute en thread et swingWorker je me pose une question :
Le fait de lancer un thread SwingWorker depuis un autre dans doInBackground ne poserat-il pas problème ?
EnregistrerFichierCommande vat-il attendre que OuvrirFichierCommande aille finit (donc la fin de done) ou continuer en parrallèle ?
Je ne peut pas utiliser non plus invokeAndWait car celà va lancé EnregistrerFichierCommande dans l'EDT et son doInBackground aussi...

Merci de m'éclairer.

[gifffftane]

Non, cela ne pose pas de problèmes de lancer un autre SwingWorker depuis un doInBackground.

On ne sait pas quand va commencer la partie doInBackground de ton SwingWorker suivant, ni même quand est-ce qu'elle se terminera. Le second peut très bien se terminer avant le premier, et donc le second done être lancé avant le premier.

La seule chose que l'on sache est que les done seront exécutés successivement, dans le EDT, mais on ne sait pas dans quel ordre.

[romuluslepunk]

Ok, donc sa va posé problème car il faut que le fichier soit enregistrer complètement avant qu'il ne commence à ouvrir l'autre...

Comment je peut attendre la fin de l'enregistrement avant de continuer ?
J'avais pensé à invokeAndWait, celà sert bien a lance un thread et attendre sa fin pour continuer ?
Mais si je l'utilise, cela va lancé le doInBackground de mon 2eme SwingWorker dans l'EDT non ?

[gifffftane]

Tu peux profiter de ce que le SwingWorker soit un Runnable et un Future pour utiliser un Executors, par exemple le newSingleThreadExecutor.

[adiGuba]

Salut,

On ne sait pas quand va commencer la partie doInBackground de ton SwingWorker suivant, ni même quand est-ce qu'elle se terminera. Le second peut très bien se terminer avant le premier, et donc le second done être lancé avant le premier.

En effet par défaut les SwingWorker utilisent un pool de 10 threads max. Tu peux donc avoir 10 tâches en parallèle (si tu en ajoutes d'autres elles seront mises en attente).

Comment je peut attendre la fin de l'enregistrement avant de continuer ?

Ce que tu décris ici ne correspond pas à deux tâches parallèles mais à deux actions séquentielles fortement lié. Pourquoi utiliser 2 thread si le premier devra attendre la fin du second ???
Tu dois faire les 2 dans le même thread, et donc dans le même SwingWorker :

1. Tu enregistres le fichier courant
2. Tu ouvres le nouveau fichier

J'avais pensé à invokeAndWait, celà sert bien a lance un thread et attendre sa fin pour continuer ?

Non invokeAndWait() permet de faire un traitement dans le thread graphique (comme par exemple récupérer l'état d'un composant). Il ne faut donc surtout pas l'utiliser pour des traitements lourds...

a++

[romuluslepunk]

Ce que tu décris ici ne correspond pas à deux tâches parallèles mais à deux actions séquentielles fortement lié. Pourquoi utiliser 2 thread si le premier devra attendre la fin du second ???
Tu dois faire les 2 dans le même thread, et donc dans le même SwingWorker :

1. Tu enregistres le fichier courant
2. Tu ouvres le nouveau fichier

C'est ce que j'ai finit par faire.
J'ai regroupe les commandes qui sont lié dans le même SwingWorker, voici +- à quoi sa ressemble :

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
43
44
45
46
47
48
49
50
51
52
53
54
public class Executeur extends SwingWorker<Boolean, Boolean>
	{
	private String action = null;
 
	private Executeur(String action)
		{
		this.action = action;
		}
 
	@Override
	protected Boolean doInBackground() throws Exception
		{
		// si l'utilisateur demande à ouvrir un fichier
		if (this.action.equals("ouvrir"))
			return this.ouvrirFichier();	
 
		// si l'utilisateur demande a enregistrer le fichier
		else if (this.action.equals("enregistrer"))
			return this.enregistrerFichier();
 
		return false;
		}
 
	protected void done()
		{
		// si l'utilisateur demande à ouvrir un fichier
		if (this.action.equals("ouvrir"))
			this.ouvrirFichierDone();	
 
		// si l'utilisateur demande a enregistrer le fichier
		else if (this.action.equals("enregistrer"))
			this.enregistrerFichierDone();
		}
 
	private boolean ouvrirFichier() throws Exception
		{
		return true;
		}
 
	private void ouvrirFichierDone()
		{
 
		}
 
	private boolean enregistrerFichier() throws Exception
		{
		return true;
		}
 
	private void enregistrerFichierDone()
		{
 
		}
	}

De cette manière ouvrirFichier peut appelé enregistrerFichier en cas de besoin et je suis sur qu'il attendra la fin du traitement pour continuer.
Mes méthode retourne true si l'action à été effectué, false si l'utilisateur à annulé donc je peut facilement voir si il à annulé l'enregistrement et donc annulé aussi l'ouverture. En cas d'erreur, c'est stoppé grace à throw...
Les commandes qui n'ont aucune interaction avec d'autre commande on leur propre SwingWorker.
Je pense avoir trouvé une bonne conception.


Je vais tout de même regarder aux executor, même si je l'utilise pas cette fois, celà me servira certainement à l'avenir.

Merci