Discussion :

[Gaillac]

Bonjour,

il doit y avoir quelque chose que je n'ai pas compris dans les états des thread et l’exception java.lang.IllegalThreadStateException

Je fais ça

Code :

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

1
2
3
System.out.println("Etat du thread " + this.isAlive());
if(!this.isAlive())
   this.start();

Au premier passage il m'affiche
Etat du thread false
et tout va bien.
Puis je rappelle se bout de code (et pour mi je suis sortie du premier run)
il m'affiche toujours
Etat du thread false
et du coup essaye de re-faire un start (et jusque là c’est ce que je veux), et bam, IllegalThreadStateException.

Il y a quelque chose qui m'échappe du coup.

[thelvin]

Un thread est créé, puis démarré, puis il s'exécute (on dit qu'il est vivant,) puis il s'arrête et meurt.
Quand il est mort il est mort. On ne peut pas l'exécuter une deuxième fois, il faut créer un nouveau thread.

Si c'est un problème, l'approche habituelle est de se créer un pool de thread qui bouclent et attendent du travail sans jamais se terminer, et auxquels on fournit du travail. C'est le principe du ThreadPoolExecutor.

[Gaillac]

OK, merci.

Je vais regarder de ce côté là mais ça veut dors et déjà dire que mon approche initiale est pourrie.

J'ai installé mon système de threads a posteriori sur des classes existante.
En occurrence j'ai fait hériter une classe existence (qui réceptionne les évènements (implémente ActionListener) et en fait plein de trucs formidables avec) et encapsulé un de ses traitements dans le run.

Donc en gros, au premier clic pour se traitement ça marche, au deuxième ça explose.

Je pense que je vais devoir revoir le fonctionnement, utiliser une classe séparée dédiée qui fera le boulot que j'ai mis dans le run (via un paquet d’accesseurs du coup, mais je ferais avec).


Je reviendrais conclure le topic quand j'aurais quelque chose de viable.

[JoeChip]

En fait il suffit de réinstancier ta thread avant le start(). donc à un moment, au lieu de faire "monThread.start()" tu fais juste "new MonThread.start()"

[Gaillac]

thelvin a répondu à ma question.
BenWillard a répondu à mon problème (je ne veux qu'un seul thread de ce type à la fois. Mais je veux qu'il soit interruptible, d'où les threads).
Ça fait un point pour les 2.

J'ai sorti la partie "threadée" de ma classe, et j'ai fait ça :

Code :

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

1
2
3
4
5
if((this.threadDExecution == null) || (!this.threadDExecution.isAlive()))
{
      this.threadDExecution = new threadExecution(this);
      this.threadDExecution.start();
}

J'ai pour l'instant le comportement souhaité, sujet clôt.