IllegalThreadStateException

Version imprimable

31/07/2008, 00h33
Sylk

IllegalThreadStateException

Bonjour,

J'ai developpe une application et j'essaye de rajouter une couche graphique. Pour cela, je me suis interesse au framework MVC comme on me l'avait conseille sur ce forum.

J'ai donc cree mon interface graphique, avec 2 boutons (start et stop). Le bouton start lance mon classe "Model" (qui derive de Thread afin de ne pas bloquer mon interface graphique). Cette classe fonctionne en permanence (elle va lire des Queues MQ toutes les 30 secondes), et pour arreter proprement mon programme, un bouton stop permet de modifier une variable static de ma classe model afin d'indiquer que l'utilisateur souhaite arreter le processus.

Quand je clique sur stop, le thread s'arrete proprement, mais j'ai une erreur si j'appuie une seconde fois sur Start

Citation:

Exception in thread "AWT-EventQueue-0" java.lang.IllegalThreadStateException
at java.lang.Thread.start(Thread.java:595)
at MQSplitterController.startMQSplitter(MQSplitterController.java:30)
at mqsplitter.JFrameMQSplitter.actionPerformed(JFrameMQSplitter.java:64)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
at java.awt.Component.processMouseEvent(Component.java:6041)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3265)
at java.awt.Component.processEvent(Component.java:5806)
at java.awt.Container.processEvent(Container.java:2058)
at java.awt.Component.dispatchEventImpl(Component.java:4413)
at java.awt.Container.dispatchEventImpl(Container.java:2116)
at java.awt.Component.dispatchEvent(Component.java:4243)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4322)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3986)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3916)
at java.awt.Container.dispatchEventImpl(Container.java:2102)
at java.awt.Window.dispatchEventImpl(Window.java:2440)
at java.awt.Component.dispatchEvent(Component.java:4243)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:273)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:183)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:173)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:168)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:160)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:121)

Quelqu'un sait'il pourquoi j'ai cette erreur ?

Merci
31/07/2008, 10h26
dinobogan

On ne peut pas faire deux fois "start" sur un Thread.
La pause du Thread doit se gérer dans la méthode "run" à l'aide d'un booléen et de "wait". Le redémarrage doit se gérer par "notify". Regarde la javadoc à ce sujet, qui explique pourquoi "stop" est deprecated et donne le code à utiliser à la place, y compris pour faire une pause et un redémarrage.
31/07/2008, 17h32
Sylk

Je n'utilise pas de Thread.Stop pour arreter mon Thread, mais une variable statique (private static boolean stopThread) qui est consulte dans mon Thread. Si cette variable est egale a True, je termine normalement mon thread.
31/07/2008, 17h49
dinobogan

Citation:

Envoyé par Sylk

Je n'utilise pas de Thread.Stop pour arreter mon Thread, mais une variable statique (private static boolean stopThread) qui est consulte dans mon Thread. Si cette variable est egale a True, je termine normalement mon thread.

Je n'ai pas parlé de "stop" mais de "start" ;)
J'imagine que tu appelles "start" sur ton Thread lors de l'appuie sur le bouton ? Et sur la même instance de Thread ?
31/07/2008, 18h21
Sylk
Desole, je parlais du Thread.Stop par rapport au lien que tu m'avais donne.

Effectivement, j'utilise un start sur la meme instance, donc ca signifie que si je cree un nouveau thread au moment du click, ca devrait passe non ?
Code:

1 2 3 4 5 6 7 public void actionPerformed(ActionEvent e) { new Thread(new Runnable() { public void run() { getController().startMQSplitter(); } }).start(); }
Plus de probleme avec ce code non ?
01/08/2008, 09h33
dinobogan
Citation:

Envoyé par Sylk

Desole, je parlais du Thread.Stop par rapport au lien que tu m'avais donne.

Ah ok. Mais le lien que je t'ai donné ne parle pas que de ça. Il explique également, avec le code associé, comment mettre un Thread en pause, puis le redémarrer. C'est pour cette raison que je te l'ai donné.
Citation:
Effectivement, j'utilise un start sur la meme instance, donc ca signifie que si je cree un nouveau thread au moment du click, ca devrait passe non ?

Code:

1 2 3 4 5 6 7 public void actionPerformed(ActionEvent e) { new Thread(new Runnable() { public void run() { getController().startMQSplitter(); } }).start(); }

Plus de probleme avec ce code non ?
Normalement, c'est bon, effectivement. Mais c'est vraiment moche d'instancier un nouveau Thread. Imagine : si on clique deux fois de suite sur démarrer, tu vas devoir vérifier si un Thread tourne déjà. Alors qu'avec une seule instance, tu vas jouer avec un booléen contenue dans l'unique Thread, et appelé "notify".
01/08/2008, 16h32
Sylk

Citation:

Envoyé par dinobogan

Ah ok. Mais le lien que je t'ai donné ne parle pas que de ça. Il explique également, avec le code associé, comment mettre un Thread en pause, puis le redémarrer. C'est pour cette raison que je te l'ai donné.

Normalement, c'est bon, effectivement. Mais c'est vraiment moche d'instancier un nouveau Thread. Imagine : si on clique deux fois de suite sur démarrer, tu vas devoir vérifier si un Thread tourne déjà. Alors qu'avec une seule instance, tu vas jouer avec un booléen contenue dans l'unique Thread, et appelé "notify".

En fait, mon bouton Start se transforme en bouton Stop, donc un second click mettra fin au thread

Merci pour tout tes conseils et ton aide !