comprendre une partie de code

Version imprimable

Voir 40 message(s) de cette discussion en une page

Salut à tous,

J'ai rucupéré une partie de code et je ne comprends pas l'interet du Runnable runner. Je ne vois pas à quoi il sert...

Si quelqu'un comprends:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
    public class BuildThread extends Thread {
         public BuildThread() {
        }
 
        public void run() {
            Runnable runner= new Runnable() {
                public void run() {
                }
            };
            try {
                SwingUtilities.invokeLater(runner);
                if (isMode1()) {
                    build();
                }                                 
                }
            }
            catch (InterruptedException ignoredException) {
            }
            catch (InvocationTargetException ignoredException) {
            }
 
        }
    }

09/08/2006, 17h28
arasium

Lol je ne vois pas trop non plus. D'autant plus que ce Runnable ne fait rien du tout (je pense, g peut être raté une accolade)! Tu l'as chopé ou ce code?
09/08/2006, 17h36
divxdede

EDIT: me suis fait avoir sur ces accolades... ca ne sert a rien...
09/08/2006, 17h43
XSeb74

à mon avis il manque des bouts dans le code !
c'est un exemple qui te montre comment lancer un Thread qui fait des choses sans bloquer l'affichage
09/08/2006, 17h45
menuge

Citation:

Envoyé par XSeb74

à mon avis il manque des bouts dans le code !
c'est un exemple qui te montre comment lancer un Thread qui fait des choses sans bloquer l'affichage

oui, c'est tout à fait ça.
Mais je ne comprends pas pourquoi est ce qu'il faut faire un Runnable qui ne fait rien! je ne vois pas la logique... A moins que je sois ....
09/08/2006, 17h46
menuge

En fait, je ne vois pas en quoi ce thread ne bloque pas l'affichage
09/08/2006, 18h47
XSeb74

cette commande :
SwingUtilities.invokeLater(runner);

dit de lancer le thread sans bloquer l'affichage.
La il fait rien mais tu peux mettre des choses dans le run() et tout de suite ça devient plus interessant.

si tu ne fait pas un invokelater, les affichage sont gelés tant que le thread lancé est pas fini et ça peux poser problème en cas de multifenêtrage.
09/08/2006, 19h14
®om

Citation:

Envoyé par XSeb74

cette commande :
SwingUtilities.invokeLater(runner);

dit de lancer le thread sans bloquer l'affichage.

Non, pas vraiment...

Ca veut dire que ça met en file d'attente dans l'EventDispatchThread (qui s'occupe de l'affichage) le Runnable que tu lui donnes en paramètre.

Donc au contraire, si le Runnable est long à s'exécuter, il bloquera l'affichage...

L'intérêt est que si son appel est fait à partir d'un autre Thread que l'EDT, et que le Runnable met à jour un affichage, sa mise à jour est "thread-safe" (respecte la règle du Thread unique pour la mise à jour de l'affichage).

Mais là, bien sûr, avec un Runnable qui ne fait rien, ça ne sert à rien...
10/08/2006, 15h23
menuge

Donc, en fait, le

Code:

SwingUtilities.invokeLater(runner)

ne sert à rien! c'est ça?
10/08/2006, 15h33
arasium

Si, si une mise à jours de l'affichage est en cours, alors il ne lancera le thread que lorsque celle ci sera terminé. Cela evite de bloquer la mise à jour du dessin.

EDIT: euh après relecture, j'ai pas l'impression d'avoir été clair 8O
10/08/2006, 15h44
menuge

Citation:

Envoyé par arasium

Si, si une mise à jours de l'affichage est en cours, alors il ne lancera le thread que lorsque celle ci sera terminé. Cela evite de bloquer la mise à jour du dessin.

EDIT: euh après relecture, j'ai pas l'impression d'avoir été clair 8O

J'ai pas compris! Désolé :lol:
10/08/2006, 15h50
arasium

En fait, des que Java veut modifier une interface Swing, il le fait via l'EDT (le thread dédié aux modif graphiques). Donc si toi dans ton thread, par exemple le main (qui à toujours son propre thread) tu demande de modifier un élément graphique, il peut y avoir un conflit car il se peut que lorsque tu demande la modif, l'EDT soit déja en train de modifier l'élément!

:) Je pense que c un peu mieu la! Mais je ne suis pas un expert, ce te raconte ce que je pense avoir compris :)
10/08/2006, 16h01
adiGuba

Salut,

Citation:

Envoyé par arasium

En fait, des que Java veut modifier une interface Swing, il le fait via l'EDT (le thread dédié aux modif graphiques). Donc si toi dans ton thread, par exemple le main (qui à toujours son propre thread) tu demande de modifier un élément graphique, il peut y avoir un conflit car il se peut que lorsque tu demande la modif, l'EDT soit déja en train de modifier l'élément!

C'est tout à fait ca !

Toutes les modifications des composants Swing devraient être effectué dans l'EDT (a quelques exceptions près, car certaines méthodes sont thread-safe et peuvent donc être utilisées depuis d'autre thread - voir la doc pour plus de détail).

Pour exécuter du code dans l'EDT, on utilise SwingUtilities.invokeLater() qui mettre l'objet Runnable en file d'attente, et sera exécuté au plus tôt par l'EDT...

A noter enfin que la plupart des méthodes des listener Swing sont appellé via l'EDT (actionPerformed(), mouseclicked(), etc.)

Plus de détail : Threads et performance avec Swing par Romain Guy

a++

[edit] PS : mais dans ce cas l'utilisation d'un Runnable vide est inutile... d'où sors-tu ce code ???
10/08/2006, 16h12
menuge
OK, je comprends mieux!!!!

Mais j'ai juste une petite question, supposons que je sois dans ce cas:
J'ai un bouton et lorsque je clique, il m'exécute une méthode toto() qui faitune action et qui fait "tourner" une progressBar durant le travail:
Code:

1 2 3 4 5 6 7 8 9 10 toto(){ //MISE EN ROUTE PROGRESS BAR //ACTION .... //ARRET PROGRESS BAR }
Comment faire cela???
Je fais
Code:

1 2 3 4 5 6 SwingUtilities.invokeLater(new Runnable(){ void run(){ toto(); } } )
10/08/2006, 16h22
arasium
Dans ce cas, lors du clic sur le bouton, c'est la méthode actionPerformed qui est appelé (elle meme dans l'edt). Donc le problème est que toto va être appéle dans l'edt et donc toutes les modifs que tu vas demander vons être appelés aussi dans l'edt, mais ne se lanceront qu'apres.
Par exemple, on suppose que toto affiche la barre, la fait avancer jusqu'a la moitié puis l'efface. Tu auras dans l'ordre:

clic sur le bouton declenche dans l'edt:

actionPerformed
toto
allumerBar (appelé par toto mais mise à la suite dans l'EDT)
avancerBar (idem)
effacerBar (idem)

En fait, la plupart des actions de modification des composants swing font appel à invokeLater, donc si ta méthode est déja dans l'edt, les modification seront lancé après que ta méthode soit terminée.

Le mieu est de faire dans actionPerformed:
Code:

1 2 (new Thread(new Runnable(){public void run(){toto();}})).start();
10/08/2006, 16h36
menuge

Et le simple fait de mettre toto dans le thread, permettra à la barre de progression de continuer même durant l'ACTION.
10/08/2006, 16h38
arasium

Normalement oui car dans ce cas TOTO sera dans un nouveau thread, mais quant tu fera avancer ta barre avec progressBar.setValue(int i), la fonction setValue lancera les modification a faire dans l'EDT (un autre thread que celui de TOTO). Donc dans ce cas les deux thread tourne en parallèle! Pas de problème, l'affichage suit ce que demande TOTO.

Si tu veut g un bout de code?
10/08/2006, 16h42
menuge

Citation:

Envoyé par arasium

Normalement oui car dans ce cas TOTO sera dans un nouveau thread, mais quant tu fera avancer ta barre avec progressBar.setValue(int i), la fonction setValue lancera les modification a faire dans l'EDT (un autre thread que celui de TOTO). Donc dans ce cas les deux thread tourne en parallèle! Pas de problème, l'affichage suit ce que demande TOTO.

Si tu veut g un bout de code?

Ca je veux bien! Merci

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
 
public void actionPerformed(ActionEvent actionEvent)
  {
	    String actioncommand = actionEvent.getActionCommand();
	    if (actioncommand == "Generer matrice")
	    {
	    	// l'action à effectuée est lancée sous forme de thread car
	    	// elle est longue à effectuer et peux perturber l'affichage
	    	new Thread() {
	            public void run() {genereMatrice();}
	        }.start();
	    }
  }
 
private void genereMatrice()
  {
	ensembleBalisesAmont = new EnsembleBalises();
	ensembleBalisesAval = new EnsembleBalises();
	inversion=true;
 
    bureauMatrix.setWaitCursor(true);
 
    ensemblePourGeneration = bureauMatrix.getEnsemblePourGeneration();
 
    if(ensemblePourGeneration != null)
    {
      // un file pour récuperer le repertoire de travail
  	  File fileSauvegarde = ensemblePourGeneration.getDestination();
 
  	  urlFichierColonneDroite = fileSauvegarde.getParent()  
                                               +File.separator+nomFichierColonneDroite;
  	  urlFichierColonneGauche = fileSauvegarde.getParent()
                                               +File.separator+nomFichierColonneGauche;
      traitement();
      statusBarMatrix.clearProgressBar();
    }
    bureauMatrix.setWaitCursor(false);
  }
 
private boolean traitement()
  {
    EnsembleBalises ensembleBalisesTemp;
 
    statusBarMatrix.setProgressBar(ensemblePourGeneration.
                                              getGroupeAmont().size()+
  		           ensemblePourGeneration.getGroupeAval().size()); 
 
    // récupération de l'ensemble des balises en amont puis en aval
    for(FichierMatrix fichier:ensemblePourGeneration.getGroupeAmont())
    {
      statusBarMatrix.setLabel(fichier.getName());
      if(!ensembleBalisesAmont.ajouter(fichier)) return false;
      statusBarMatrix.incrementeProgressBar();
    }
    for(FichierMatrix fichier:ensemblePourGeneration.getGroupeAval())
    {
      statusBarMatrix.setLabel(fichier.getName());
      if(!ensembleBalisesAval.ajouter(fichier)) return false;
      statusBarMatrix.incrementeProgressBar();
    }
 
    if(!ensembleBalisesAmont.verification()) return false;
    if(!ensembleBalisesAval.verification()) return false;
 
    // génération des liens
    listLiaison=ensembleBalisesAmont.lier(ensembleBalisesAval);
 
    // on parcours la liste pour voir si il y a effectivement des liens
    for(Integer lien:listLiaison)
      if(lien != null)
      {
          inversion=false;
          break;
      }

Si tu as quelque questions sur le code hésite pas car y a pas de commentaire :)

02/09/2006, 17h00
zulot

Ce post est fort interessant mais moi j'ai un probleme.

Dans un thread je dois appeler une fonctione avec des parametres qui sont définis hors du thread.

Dans ce cas la je fais comment pour appeler ma fonction qui vient d'une autre classe.

En fait ma fonction copie un fichier par tranche de 1Mo et je voudrais donc creer une progresse bar qui avance pendant que le fichier se copie.

Hors ma fonction à 2 parametre le fichier à copier et la fenetre, et la gros probleme erreur sur les variables.

Voir 40 message(s) de cette discussion en une page