Discussion :

[arsenik7]

le voici :

A l'intérieure des methodes Cycle() j'utilise testFin() quelques fois pour arreter mon thread quand je clic sur un bouton qui me met ma variable stopThread à true.

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
public class Histo extends Thread {
 
	private int nb=0, nbDD=0;
	private Date date;
	private GetSNMP gsnmp;
	private Poster mail;
	private String[] elements;
	private Elements el;
	private PingElement pe; 
	private String[] tabR;
	private long idElement;
	private Locale locale;
	private DateFormat dateFormat;
	private DataBas db;
	private ResultSet rs;
	private long temps;
	private boolean stopThread = false;
private boolean stopThread = false;
 
	public Histo(String tps)
	{
	temps=Long.parseLong(tps);
	this.start();
	}
 
public synchronized void arret() {
   stopThread = true;
}
 
public void run() 
{
	try {
	while (true){
 
		el = new Elements();
		Cycle("sgsf_mon",0);	//Switch et Routeurs
		Cycle("sgsf_mon",1);	//Serveurs et Flexnet
		Cycle("sgsf_mon",2);	//Points acces
		Thread.sleep(temps*1000);
	}	
		}catch( InterruptedException e ) {
			System.out.println(e);
		} 
} 
 
public synchronized void testFin() throws InterruptedException {
    if( stopThread ) {
    	throw new InterruptedException();
    } 
}

Lors d'un clic sur ce bouton , la methode tesFin() s'execute car j'obtiens dans la console : java.lang.InterruptedException. Problème le thread ne s'arrete po

[arsenik7]

J'aimerai ne pas utiliser une exception pour arreter le thread , mais je ne vois pas comment terminer la fonction run en appuyant sur un bouton qui mettrai une variable a true.

Mes fonctions cycles sont super longues et je ne doit pas attendre qu'une finisse pour terminer le thread. Je voudrais que mon thread se termine a l'interieur d'une des metodes cycle.

Merci

[sozie9372]

il faut pas faire un .stop ?

[sinok]

Ou pas, stpo est deprecated, et à bannir de ce fait.
Pour interrompe un Thread tu peux passer par sa méthode interrupt()

[le y@m's]

il faut pas faire un .stop ?

la méthode stop() est Deprecated.
Pour plus d'infos : Why are Thread.stop, Thread.suspend and Thread.resume Deprecated?.

[edit] grilled [/edit]

[NeptuS]

rhaaa lalaaa .. la seule vraie méthode pour arrêter 1 Thread (on vous l'a dit et répété) c'est d'arriver à l fin de la méthode run

quand je clic sur un bouton qui me met ma variable stopThread à true

Cette variable devrait se trouver dans ton Thread, et ta méthode run devrait être :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
public void run() 
{
	while (!stopThread){
 
		el = new Elements();
		Cycle("sgsf_mon",0);	//Switch et Routeurs
		Cycle("sgsf_mon",1);	//Serveurs et Flexnet
		Cycle("sgsf_mon",2);	//Points acces
		Thread.sleep(temps*1000);
	}	
}

[sozie9372]

désolé...
En même temps je n'ai jamais eu à l'utiliser car je suis plutôt adepte de la méthode de NeptuS

[arsenik7]

Je ne voudrais pas attendre qu'il finisse un while complet avant de s'arreter .
Chaque Cycle et une opération longues que je voudrais stopper à tous moments

[NeptuS]

t'as la solution du break, mais c'est crade.

[NeptuS]

à priori, je dirai que ton programme est trop pensé linéairement ... et pas assez d'un point de vue évènementiel :

A l'intérieure des methodes Cycle() j'utilise testFin() quelques fois pour arreter mon thread quand je clic sur un bouton qui me met ma variable stopThread à true.

Cette structure est trop lourde et te fait perdre une boucle entière avant que le Thread ne se termine.

il y a 2 ou 3 choses que je ne saisis pas dans ton programme, comme la raison pour laquelle tu recréée un "Elements" à chaque début de boucle car il n'est -- a priori, pas modifié.

Si "Cycle" est une méthode, elle devrait commencer par une minuscule.
tu peux très bien remplacer :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
public void run() 
{
	while (!stopThread){	
		el = new Elements();
		Cycle("sgsf_mon",0);	//Switch et Routeurs
		Cycle("sgsf_mon",1);	//Serveurs et Flexnet
		Cycle("sgsf_mon",2);	//Points acces
		Thread.sleep(temps*1000);
	}	
}

par :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public void run() 
{
	while (!stopThread){
		el = new Elements();
		for(int i=0 ; i<3 ; i++)
		     if(!stopThread) Cycle("sgsf_mon", i);
		Thread.sleep(temps*1000);
	}	
}

Ensuite, si tu veux stopper le chargement du cycle au milieu, tu modifie ta fonction "Cycle" en faisant des blocs insécables (des blocs logiques de if(!stopThread){ ....} )

[Regis.C]

à priori, je dirai que ton programme est trop pensé linéairement ... et pas assez d'un point de vue évènementiel :

Cette structure est trop lourde et te fait perdre une boucle entière avant que le Thread ne se termine.

il y a 2 ou 3 choses que je ne saisis pas dans ton programme, comme la raison pour laquelle tu recréée un "Elements" à chaque début de boucle car il n'est -- a priori, pas modifié.

Si "Cycle" est une méthode, elle devrait commencer par une minuscule.
tu peux très bien remplacer :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
public void run() 
{
	while (!stopThread){	
		el = new Elements();
		Cycle("sgsf_mon",0);	//Switch et Routeurs
		Cycle("sgsf_mon",1);	//Serveurs et Flexnet
		Cycle("sgsf_mon",2);	//Points acces
		Thread.sleep(temps*1000);
	}	
}

par :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public void run() 
{
	while (!stopThread){
		el = new Elements();
		for(int i=0 ; i<3 ; i++)
		     if(!stopThread) Cycle("sgsf_mon", i);
		Thread.sleep(temps*1000);
	}	
}

Ensuite, si tu veux stopper le chargement du cycle au milieu, tu modifie ta fonction "Cycle" en faisant des blocs insécables (des blocs logiques de if(!stopThread){ ....} )

C'est beau, quand même, l'optimisation de code source...non ?
Moi en tout cas j'aime bien !
Ca permet de voir une autre façon de coder, de la part d'un collègue qui est totalement détaché du projet, ce qui lui permet d'analyser d'un plus haut niveau même un ridicule petit bout de code.

Ca encourage à développer en binome ça !
Vivement que je connaisse une exeprience en eXtreme Programming !!

[adiGuba]

Salut,

Cette structure est trop lourde et te fait perdre une boucle entière avant que le Thread ne se termine.

Pas forcément : si la vérification de l'état du thread est effectué à l'intérieur de la méthode Cycle() (c'est pas beau un nom de méthode avec une majuscule) et que cette dernière renvoit une InterruptedException, on sort directement de la boucle...

Maintenant si tu dis que le texte du catch() est affiché dans la console le thread devrait s'arrêter puisque tu arrives à la fin de la boucle... à moins qu'il ne s'agissent d'un autre catch()...

a++

[NeptuS]

adiGuba, si c'est une erreur qui fait s'arrêter le programme, je suis d'accord pour que ça lance une exception.
En revanche, si c'est une condition normale de sortie de Thread, je ne vois pas la "propreté" de lancer une Exception : un simple boolean est acceptable.

Regis.C :

ce qui lui permet d'analyser d'un plus haut niveau même un ridicule petit bout de code

Le "ridicule petit bout de code" qui a été changé a permis de répondre à une partie de la question posée, qui était :

Je ne voudrais pas attendre qu'il finisse un while complet avant de s'arreter

et le ' if ' inséré permettait de répondre à cette question.
La boucle n'est qu'une simple fioriture qui permet de ne pas réécrire 3 fois les 2 mêmes lignes.

Et personnellement, le codage en binome, j'ai testé et ça ne donne que de la m**** : la seule solution pour coder à plusieurs, c'est une analyse sans faille et coder (si possible) des packages différents.

[adiGuba]

adiGuba, si c'est une erreur qui fait s'arrêter le programme, je suis d'accord pour que ça lance une exception.
En revanche, si c'est une condition normale de sortie de Thread, je ne vois pas la "propreté" de lancer une Exception : un simple boolean est acceptable.

Un simple booléen peut faire l'affaire, je n'ai pas dit le contraire...

Toutefois l'utilisation d'une exception peut s'avérer utile dans certain cas, car cela permet de stopper le thread au milieu d'une méthode. On ne connait pas la méthode Cycle() et cette dernière peut être plus ou moins longue. Il peut être souhaitable de stopper le thread avant la fin du traitement complet de cette méthode. Dans ce cas une InterruptedException est approprié, puisque elle indique que la méthode ne s'est pas terminé correctement...

De plus on peut éviter de s'embêter à coder des méthodes pour arrêter le thread et vérifier l'état du booléen : on peut directement utiliser interrupt() et isInterrupted().

Enfin interrupted() permet de sortir de certaines méthodes bloquantes (sleep(), wait(), read()/write() avec les InterruptibleChannel, etc.) en renvoyant une exception...

a++