Thread et Swing

**alladdin** · 14/08/2006, 12h30

Bonjour,

J'ai un problème au niveau de mon apllication, qui fait un traitement long (plusieurs heures) quand on appuie sur un boutton, et pour cela il faut le mettre dans un thread.

J'ai donc lu ce tuto
http://gfx.developpez.com/tutoriel/j...ing-threading/

et je l'ai appliqué à mon application.

Effectivement maintenant le boutton n'apparait plus comme enfoncée (bon signe) mais les autres bouttons de mon applications ne repondent toujours pas (notamment la croix pour fermer l'application ...).

Quelqu'un serait pourquoi?

Merci.

**xixi31** · 14/08/2006, 13h57

salut,

Le plus simple serait que tu fournisses un peu de code pour illustrer, parce que comme ca, ca va pas être évident de t'aider...

xixi

**alladdin** · 14/08/2006, 14h04

ok, si ca peux aider ....

code du boutton

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
begin.addActionListener(new ActionListener() {
				public void actionPerformed(ActionEvent e) {
					new Thread(new Runnable() {
						public void run() {
							SwingUtilities.invokeLater(new Runnable() {
								public void run() {//traitement	}
							});
						}
					}).start();
				}
			});

exemple de code que je voudrais qui marche en parralèle

Code :

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

1
2
3
4
5
6
 
	f.addWindowListener(new WindowAdapter() {
				public void windowClosing(WindowEvent e) {
					System.exit(0);
				}
			});

**sinok** · 14/08/2006, 14h06

Fais voir le code du run de ton thread, j'ai comme l'impression que tu fais les traitements dans l'EDT alors qu'il faudrait que tu les fasse en dehors.....

**xixi31** · 14/08/2006, 14h07

le mieux serait que tu effectue ton long traitement dans un autre thread que l'edt. c'est déjà ton cas??

**sinok** · 14/08/2006, 14h22

C'est le cas

Grosso modo le SwingUtilities.invokeLater() ne s'emploie que quand tu dois faire des modifications sur ton interface graphique, et surtout pas pour effectuer des calculs

**alladdin** · 14/08/2006, 14h46

bon voilà un bout de code du traitement (le reste c'est du même ordre )

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
 
while(true){
										try{
											wm un = new wm();
											un.analyse(new URL(httpLink.getText()), lieux.getSelectedIndex()+3);
											if(miniPowa.getText().compareTo("") == 0){
											}
											else{
												Object resu =un.findMoshu(Integer.parseInt(miniPowa.getText()),Integer.parseInt(maxiPowa.getText()));
												if(resu != null){
													Runtime.getRuntime().exec(lecteurPath.getText() + " " + sonPath0.getText());
													JOptionPane.showMessageDialog(null, "Alerte");
												}
 
}catch(Exception d){
											d.printStackTrace();
										}
}

Sinon si je mets du code dans un autre thread sans utiliser le invokeLater je vais pas respecter la règle du thread unique et qui d'après le tutoriel que j'ai lu est déconseillé ( car ca sera plus "thread safe"). C'est bien ca, non?

**sinok** · 14/08/2006, 15h30

J'ai comme l'impression que tu n'a pas saisi toute la subtilité de bouzin....

La règle du thread unique ne s'applique qu'aux modifications directes de l'interface, (encore heureux, sinon quelle serait l'utilité de créer de threads pour effectuier des calculs lourds....)

Grosso merdo tu dois procéder de la façon suivante:

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
 
 
while(true){
  try{
    wm un = new wm();
    un.analyse(new URL(httpLink.getText()), lieux.getSelectedIndex()+3);
    if(miniPowa.getText().compareTo("") == 0){
    }
    else{
            Object resu =un.findMoshu(Integer.parseInt(miniPowa.getText()),Integer.parseInt(maxiPowa.getText()));
        if(resu != null){
            Runtime.getRuntime().exec(lecteurPath.getText() + " " + sonPath0.getText());
            SwingUtilities.invokeLater(new Runnable(){ public void run(){JOptionPane.showMessageDialog(null, "Alerte");});
                }
    }
  }catch(Exception d){
    d.printStackTrace();                                        }
  }
}

**adiGuba** · 14/08/2006, 15h32

Salut,

La règle du thread unique ne concerne que l'accès aux composants de ton interface graphique, mais tous tes traitements long doivent être effectué dans un autre thread.

Si tu utilises invokeLater() ton traitement long sera effectué dans l'EDT qui ne pourra plus mettre à jours l'affichage... Dans ton code tu utilises un Thread qui se contente de passer le traitement à l'EDT via invokeLater(), ce qui bloque toujours l'EDT, et ton Thread ne fait rien !!! La seule différence c'est que l'EDT aura le temps de finir de traiter l'évenement "onclick" sur le bouton et qu'il pourra donc le redéssiner dans son état normal avant de passer à sà tâche suivante : le traitement long que tu lui a passer via l'EDT...

Et pendant qu'il effectuera ce traitement ton interface paraitra figé, puisqu'il ne pourra plus s'en occuper...

Donc il faut bien que ce soit ton thread qui fasse le traitement long, et tu ne doit utiliser invokeLater() QUE pour mettre à jour l'affichage (et donc des traitements courts), par 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
bbegin.addActionListener(new ActionListener() {
    public void actionPerformed(ActionEvent e) {
        new Thread(new Runnable() {
            public void run() {
 
                // traitement long ici
 
                final String value = ...; // resultat du traitement
 
                // On n'utilise invokeLater() que si on a besoin
                // de mettre à jours un composant graphique
                SwingUtilities.invokeLater(new Runnable() {
                    public void run() {
                        monJLabel.setText( value );
                    }
                });
            }
        }).start();
    }
});

a++

**sinok** · 14/08/2006, 15h33

Oups pardon j'avais oublié la création du runnable pour le invokeLater je viens de m'en rendre compte...
Con de mime

Euh par contre, pour le choix de l'exemple adiguba, ce n'est pas le top car le setText() est une des rares méthodes présentes dans Swing qui soit thread-safe

**alladdin** · 14/08/2006, 16h18

ok merci j'ai compris le truc

.
J'ai testé mon appli et ca marche, merci

**adiGuba** · 14/08/2006, 19h01

Envoyé par sinok

Euh par contre, pour le choix de l'exemple adiguba, ce n'est pas le top car le setText() est une des rares méthodes présentes dans Swing qui soit thread-safe

Euh... Peut-être (je ne vois rien de tel dans la doc). Mais en général je préfère ne pas me poser de question et le faire dans l'EDT dans tous les cas...

Envoyé par alladdin

ok merci j'ai compris le truc

.

Penses à utiliser le bouton

alors

a++

**sinok** · 14/08/2006, 19h21

Code :

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

 Euh... Peut-être (je ne vois rien de tel dans la doc)

Au temps pour moi, elle est safe pour les JtextComponent et classes filles, par pour un JLabel

Thread et Swing

AWT/Swing Java

Discussions similaires

Partager

Partager