Discussion :

[wilv8]

Bonjour,

Pb : je ne passe jamais dans le bloc "catch(InterruptedException)".
Info : toutes les méthodes sous-jacentes à run() ne traitent pas cette interruption mais la renvoie.
Question : Pourquoi l'interruption n'est pas attrapée dans ma méthode run() ?

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
public class BalayageDlg extends JDialog implements Runnable 
{
.
.
private void initialize() {
.
.
      this.addWindowListener(new java.awt.event.WindowAdapter() {
			public void windowClosing(java.awt.event.WindowEvent e) {
 
				if(con!=null)
				{
					//fermer la connexion et rendre le port com
					if(comThread!=null && comThread.isAlive())
					{
						//ordre d'interruption, doit être attrapé par le thread
						comThread.interrupt();
						//fermeture de la connexion et des listeners
						con.setOpened(false);
					}
				}
			}
		});
 
	}
 
 
	public void run() {
		comThread = Thread.currentThread();
		try
		{
		if (this.isHF) 
			balayageHf();
		else
			balayageFilaire();
		}
		catch(InterruptedException iex)
		{
			Thread.currentThread().interrupt();
		}
		parent.getBT_DetecterParc().setText(Hemolog.RES.getString("Detecter")+"("+parent.getNbActif()+")");
		close();
	}
.
.
}

Merci d'avance

[bygui]

Bonjour,
L'interruption de Thread semble bizarre. Et tu sur que ce soit la meilleur solution dans l'algorithme que tu as créé?
Sinon, essais de rajouter :

Code :

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

1
2
 
if (Thread.interrupted()) {throw new InterruptedException();}

[wilv8]

Disons que j'ai 2 solutions :

Soit j'empêche l'utilisateur de quitter l'appli pendant la phase gérer par mes méthodes "balayageHf()" et "balayageFilaire()", auquel cas la question ne se pose plus.

Soit l'utilisateur peut quitter mais le thread gérant ces deux méthodes doit impérativement se terminer sans quoi il pourrait continuer d'utiliser la liaison série quelques secondes de trop. De la à dire que c'est la meilleure solution... c'est du moins la seule que je connais si je tiens à mettre en oeuvre cette 2eme alternative.

Quand à la solution que tu proposes, je ne vois pas trop comment ni ou la mettre en oeuvre...

Pour info mes méthodes "balayage()" appellent plusieurs autre méthodes diffusants toutes des InterruptedException...

[darkxan]

Je ne comprends pas exactement ton code. En effet dans la partie Listener, tu sembles vouloir interrompre le thread donc si tu l'interromps alors tu n'as rien à faire... De plus tu essaies de récupérer l'exception pour l'interrompre alors que sauf s'il y a des Thread en dessous, il sera déjà interrompu de manière naturelle.

Il faut savoir que l'exception InterruptedException n'est pas lancée à chaque fois que le Thread est interrompu. Cette exception arrive uniquement s'il est dans un état d'attente et donc qu'il a été interrompu durant celui-ci :

If this thread is blocked in an invocation of the wait(), wait(long), or wait(long, int) methods of the Object class, or of the join(), join(long), join(long, int), sleep(long), or sleep(long, int), methods of this class, then its interrupt status will be cleared and it will receive an InterruptedException.

EDIT: Il faudrait vérifier le comportement mais si tu fermes ta fenêtres et que tu as mis comme opération de sortie EXIT_ON_CLOSE, je pense que les Threads sont interrompus.

[djsnipe]

Si le problème est de faire le ménage proprement à la sortie, tu peux utiliser un shutdownHook (voir ici), qui exécutera du code en fin de programme.

[bygui]

Utilise un bloque finally qui ferme correctement ton application...

[wilv8]

Bonjour,

Darkxan :
Tout à fait, dans le listener je cherche à interrompre le thread dont j'ai récupéré l'instance dans la méthode run().

La piste du EXIT_ON_CLOSE, que j'avais déjà essayée se solde par une :
"Exception in thread "AWT-EventQueue-0" java.lang.IllegalArgumentException: defaultCloseOperation must be one of: DO_NOTHING_ON_CLOSE, HIDE_ON_CLOSE, or DISPOSE_ON_CLOSE".

En effet il faut que le thread soit dans un état bloqué (wait, sleep, join) pour être interrompu... Je vais quand même pas faire une boucle infinie qui balance un ordre d'interruption jusqu'à tomber sur un moment ou le thread est bloqué et donc bien receptif !

djsnipe : merci pour le lien

Je vous tient au courant

[bygui]

pourquoi ne termines tu pas le thread tout simplement alors?

Code :

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

1
2
3
4
5
6
7
 
public void run() {
 while (!_shouldStop)
    {
       //ton traitement 
    }
}

[wilv8]

Parceque "mon traitement" consiste en l'appel d'une méthode une et une seule fois. D'ou l'inutilité de boucler dessus.

[bulbo]

Ce que je ferais ce serait une méthode stopThread qui fermerait les flux et autre ressources utilisées par la/les méthodes lancées dans le run().

1 - Tu fais le ménage proprement et c'est ce que tu veux j'ai l'impression

2 - Si tes méthodes ont besoin de ces ressources elles vont plantées (donc s'interrompre) avec une exception. Si la méthode stopThread a été appelée au préalable, tu ignores simplement l'exception qui est dans ce cas normale.

Bulbo

[wilv8]

Merci Bulbo,

C'est le genre d'approche qui colle à mes besoins, j'ai mis en place une solution similaire à ce que tu dis, ça marche.

Merci à tous de votre participation...