Thread et Swing

Version imprimable

14/08/2006, 12h30
alladdin

Thread et Swing

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.
14/08/2006, 13h57
xixi31

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

ok, si ca peux aider ....

code du boutton

Code:

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:

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

14/08/2006, 14h06
sinok

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.....
14/08/2006, 14h07
xixi31

le mieux serait que tu effectue ton long traitement dans un autre thread que l'edt. c'est déjà ton cas??
14/08/2006, 14h22
sinok

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

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

Code:

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?

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:

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();                                        }
  }
}

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:

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

14/08/2006, 15h33
sinok

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
14/08/2006, 16h18
alladdin

ok merci j'ai compris le truc :D.
J'ai testé mon appli et ca marche, merci:D
14/08/2006, 19h01
adiGuba

Citation:

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

Citation:

Envoyé par alladdin

ok merci j'ai compris le truc :D.

Penses à utiliser le bouton :resolu: alors ;)

a++
14/08/2006, 19h21
sinok

Code:

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