Discussion :

[Sylk]

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

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

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

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

[dinobogan]

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 ?

[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 :

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

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 ?

[dinobogan]

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

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 :

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

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