Stopper une thread qui tourne à l'infini

Version imprimable

Bonjour je suis en train de faire une application J2EE JSF sous glassfish.
J'ai une boutton pour lancer une thread qui vas boucler a l'infinie et une autre qui vas essayer de stopper cette thread.
Le probléme est que je veux pas toucher a mon code métier mais juste interrompre mon thread par n'importe quel moyen méme dangereuse.voici mon code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
package com.example.beans;
 
import java.util.Random;
import java.util.TimerTask;
 
 import org.apache.commons.lang.RandomStringUtils;
import org.apache.log4j.Logger;
 
import com.example.core.RandomUtils;
 
public class MySimpleRunnableTask implements Runnable {
private Logger logger = Logger.getLogger(MySimpleRunnableTask.class);
@Override
public void run() {
  /* i do not want to touch my code here*/
    int i = 1;
    while (i == 1) {
        String random = RandomStringUtils.random(3);
        logger.info(random);
    }
}
 
  }

Code:

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
 
@ManagedBean(name = "MainView")
@SessionScoped
public class MainView {
private static Thread myThread;
 
@SuppressWarnings({ "unchecked", "rawtypes", "deprecation" })
public String startSimpleThread() throws SecurityException,
        NoSuchMethodException, InterruptedException {
 
    MySimpleRunnableTask mySimpleRunnableTask = new MySimpleRunnableTask();
 
    myThread = new Thread(mySimpleRunnableTask);
           myThread.start();
    return null;
 
}
 
@SuppressWarnings({ "unchecked", "rawtypes", "deprecation" })
public String stopSimpleThread() throws SecurityException,
        NoSuchMethodException, InterruptedException {
    myThread.interrupt();
    return null;
 
}

j'ai partout recherché dans le net mais en vain et je sais bien qu'il existe une solution car je l'ai déja vue dans une application proprietaire.
Merci

28/10/2011, 17h37
adiGuba

Salut,

Dans le code de ton thread, remplaces ton while (i == 1) par ceci :

Code:

while (!isInterrupted())

Ensuite il te suffira d'utiliser interrupt() sur ton thread...

a++
28/10/2011, 18h04
thelvin

En même temps, il a dit qu'il ne voulait pas y toucher, mais que ce n'est pas grave si la méthode est dangereuse.

Je suggère de tuer glassfish avec un kill -9, puis de le redémarrer. Le thread sera ainsi arrêté. (Oui, je connais la méthode dépréciée qui permet de tenter d'arrêter un thread, mais elle rend Java très instable. Arrêter glassfish et le redémarrer est préférable à tout point de vue.)

bonjour tu m'a pas compris adiGuba.je vais t'expliquer
j'ai le code suivant

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
package com.example.beans;
 
import org.apache.commons.lang.RandomStringUtils;
 
public class MyBusinessClass {
	public static void myBusinessMethod() {
		/* this method take a lot of time */
		int i = 1;
		while (i == 1) {
			String random = RandomStringUtils.random(3);
 
			System.out.println(random);
 
		}
	}
}

et voici mon runnable class

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
package com.example.beans;
 
import java.util.Random;
import java.util.TimerTask;
 
import org.apache.commons.lang.RandomStringUtils;
import org.apache.log4j.Logger;
 
import com.example.core.RandomUtils;
 
public class MySimpleRunnableTask implements Runnable {
	private Logger logger = Logger.getLogger(MySimpleRunnableTask.class);
 
	@Override
	public void run() {
		MyBusinessClass.myBusinessMethod();
 
	}
}

j'ai cherché partout sur le net et j'ai rien trouvé :cry::cry:

28/10/2011, 18h09
blacknight1985

bonjour thelvin
non :( je veux pas comme méme tuer le serveur glassfish juste arreter cette merdique thread pas plus.le probléme est que la solution existe et je l'ai vue par mes propres yeux dans une application proprietaire avec JDK 5
28/10/2011, 18h13
adiGuba

La solution c'est de modifier ta "myBusinessMethod" afin qu'elle gère les interruptions (ou un système similaire).

C'est sûrement ce qui est utilisé par ton application propriétaire...

a++
28/10/2011, 19h03
thelvin

Citation:

Envoyé par blacknight1985

bonjour thelvin
non :( je veux pas comme méme tuer le serveur glassfish juste arreter cette merdique thread pas plus.le probléme est que la solution existe et je l'ai vue par mes propres yeux dans une application proprietaire avec JDK 5

Ce dont tu parles cause bien plus de problèmes que juste redémarrer glassfish. De toute façon je croyais que c'était pas grave s'il y a un risque.
"Un risque" est un doux euphémisme pour décrire ce que tu demandes. C'est le chaos total à un moment absolument imprévisible.
28/10/2011, 19h16
andry.aime

Bonsoir,

J'ai pas le lien pour jdk 5 mais c'est le même principe et à toi d'assumer risque:
:fleche: http://download.oracle.com/javase/1....precation.html
La façon la plus propre c'est d'utilisé un flag (comme proposé par adiGuba plus haut)

A+.
28/10/2011, 19h26
blacknight1985

bon la solution est vraiment trés simple.il suffit de faire myThread.stop(); Elle vas tuer le thread méme si stop est deprecated
28/10/2011, 20h37
adiGuba

Comme indiqué plus haut, il est préférable d'arrêter ton serveur plutôt que de faire un stop() sur un thread... Cela pourrait produire des plantages aléatoires et totalement incompréhensible !!!

a++
28/10/2011, 21h34
andry.aime

Citation:

Envoyé par adiGuba

Comme indiqué plus haut, il est préférable d'arrêter ton serveur plutôt que de faire un stop() sur un thread... Cela pourrait produire des plantages aléatoires et totalement incompréhensible !!!

C'est ce qu'il veux :mouarf:

Citation:

Envoyé par blacknight1985

Le probléme est que je veux pas toucher a mon code métier mais juste interrompre mon thread par n'importe quel moyen méme dangereuse.
29/10/2011, 04h01
thelvin

Citation:

Envoyé par blacknight1985

bon la solution est vraiment trés simple.il suffit de faire myThread.stop(); Elle vas tuer le thread méme si stop est deprecated

Tenter de tuer le thread, si c'est à la hauteur de ses moyens et de ceux de l'OS.
Et ouvrir la porte à de nombreuses choses imprévisibles.

Ce n'est pas pour rien si on veut pas citer ça.
29/10/2011, 12h52
adiGuba

Citation:

Envoyé par andry.aime

C'est ce qu'il veux :mouarf:

Je préfère marteler le message... surtout pour des lecteurs qui tomberait là dessus...

a++