Discussion :

[Virgile le chat]

Bonjour,
je suis novice en matière de threads. Si quelqu'un pouvait m'aider... Merci bien :
J'ai créé une application qui lance une animation graphique. Pour cela j'ai créé un thread qui gère cette animation. C'est-à-dire pour la lancer, l'arrêter et la reprendre (en cliquant sur des boutons de ma barre d'outil).
Seulement, quand je lance l'animation ça marche, quand je l'arrête ça marche aussi, mais quand je veux la reprendre ça plante.
Voici le code de ma classe Thread. Les noms des méthodes sont suffisamment explicites pour se passer de commentaires. Encore merci.

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
public class Evolution extends Thread {
 
	// référence à la fenetre principale de l'application
	public Evolution(fenPrinc fp){
		this.fp = fp;
	}
 
	public synchronized void anime(){
		if(this.getState() == State.WAITING)
			notifyAll();
		else
			start();
	}
 
	public synchronized void interromp(){
			try {
				wait();
			} catch (InterruptedException e) {
				e.printStackTrace();
			}
	}
 
 
	public void run() {
 
		while(true)
			try {
				fp.getPanSys().repaint();
				Thread.sleep(10);
			} catch (InterruptedException e) { e.printStackTrace();}
 
	}
 
	private fenPrinc fp;
 
}

[Rayndell]

Fais voir le code des actionPerformed de l'ActionListener de ta barre d'outils.

[lechewal]

utilise la balise code

indente ton code

extendre thread..bof

[Virgile le chat]

Voilà le code des ActionPerformed et actionlistener.
Je ne vois pas bien à quoi ça va avancer mais bon...

Code :

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

1
2
3
4
5
6
7
8
9
 
public void actionPerformed(ActionEvent e) {
 
		if(e.getActionCommand().equals("Lancer"))
			fp.getEv().anime();
 
		else if(e.getActionCommand().equals("Arreter"))
			fp.getEv().interromp();
	}

et la suite

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
//ajout des actions au menu déroulement
		actionArreter = new ActionDeroulement("Arrêter", null, this);
		actionArreter.putValue(AbstractAction.MNEMONIC_KEY, KeyEvent.VK_A);
		actionArreter.putValue(AbstractAction.ACTION_COMMAND_KEY, "Arreter");
		deroulement.add(actionArreter);
 
		actionLancer = new ActionDeroulement("Lancer", null, this);
		actionLancer.putValue(AbstractAction.MNEMONIC_KEY, KeyEvent.VK_L);
		actionLancer.putValue(AbstractAction.ACTION_COMMAND_KEY, "Lancer");
		deroulement.add(actionLancer);

A mon avis, le problème n'est pas là. Ce serait plutôt dans le code du thread.
Il semble que ce soit notifyAll() qui pose problème mais je n'en suis pas sûr.
Merci pour votre attention.
Gilles.

[jamalmoundir]

Salut,
si j'oublie ton interface graphique car je suis pas expert ds la matiére, le fait de faire un notifyAll te reveille tous les threads bloqués sur ton objet, si tu fais wait le thread va juste s'endormir mais il n'est surtout pas mort (tu ne la pas tué).
donc il faut tuer ton thread : le mieu c'est de le faire sortir du while du run, il existe une méthode qui permet de tuer un thread mais si mes souvenir sont bons elle marche pas tres bien (à vérifier).
voila j'espere que ça t'aider un peu
bonne chance

[yannart]

Tu as les appels a wait et notifyAll dans des méthodes synchronized de la même classe, donc tant que le thread est en état d'attente, il aura toujours le cadenas de la classe et aucun autre Thread pourra exécuter l'autre méthode anime().
En plus que je sache c'est pas génial d'utiliser wait et notifyAll sur ta même classe, essaye de créer une autre classe Animation par exemple que tu contrôleras grâce à la classe que tu as déjà.

[sinok]

Ce que j'en dis c'est que tu fais du swing et que tu devrais plutôt te diriger vers un timer Swing et ses méthodes start/stop pour ce genre d'opérations...

[hydraland]

Salut,

Je te conseille deux choses:
La première est te renseigner sur l'EDT, car il n'y a qu'un thread, l'EDT, qui traite les évènements(souris, clavier, affichage, ...). Ce code

Code :

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

1
2
else if(e.getActionCommand().equals("Arreter"))
			fp.getEv().interromp();

va bloquer l'EDT. Donc ton IHM sera figé et tu ne pourras plus rien faire sur ton IHM.
Deuxième chose si tu n'utilise pas la méthode préconisé par sinok, il faut que l'arrêt de ton animation(le fameux wait) soit dans le run du thread et non dans la méthode interromp.

A+
Hydraland

[lechewal]

Salut,
si j'oublie ton interface graphique car je suis pas expert ds la matiére, le fait de faire un notifyAll te reveille tous les threads bloqués sur ton objet, si tu fais wait le thread va juste s'endormir mais il n'est surtout pas mort (tu ne la pas tué).
donc il faut tuer ton thread : le mieu c'est de le faire sortir du while du run, il existe une méthode qui permet de tuer un thread mais si mes souvenir sont bons elle marche pas tres bien (à vérifier).
voila j'espere que ça t'aider un peu
bonne chance

La seule manière d'arrêter efficacement un thread est qu'il est fini d'exécuter son code