Discussion :

[Drahu]

J'annule un thread en cours d'execution avec

Code :

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

monthread.Abort()

Et voici le message qui m'est retourné:

Le thread a ete abandonne 'CALCUL_ALERTE 3,101'

CALCUL_ALERTE est une procédure stockée.
Ce qui veut dire que le thread a interrompu son execution. Donc je comprend le problème. Et je pense qu'il faut donc mettre un lock

Code :

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

1
2
3
4
lock(this)
{
Calculer_Mon_Alerte()
}

(Avec Calculer_Mon_Alerte() qui se charge de l'éxecution de ma procédure stockée)


Cela ne résoud pas mon pb. Donc je me suis trompé. Mais où ?

[DrQ]

Peux tu expliquer un peu plus l'architecture de ton prog ? Qu'est ce qui est lancé dans le thread ? Que souhaites tu arréter et laisser continuer ?

[Drahu]

Je dois effectuer un long calcul, ainsi, je l'ai mis dans un thread.

J'appuie sur mon bouton "Calculer" : il me démarre mon thread.
J'appuie sur "Annuler" : il me le stoppe. > monthread.Abort()



Mon code:

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
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
 
	public class F_Alertes : System.Windows.Forms.Form
	{
		private void  bCalcul_Click(object sender, System.EventArgs e)
		{
			ThreadCalcul threadCalc = new ThreadCalcul()
			this._treadCalcul = new Thread(new ThreadStart(threadCalc.Calculer));
 
			this._treadCalcul.Start();
 
 
		}
		private void bAnnuler_Click(object sender, System.EventArgs e)
		{
 
			this._treadCalcul.Abort();
 
		}
 
	}
 
 
	public class ThreadCalcul
	{
 
 
 
		// Constructeur	
		public  ThreadCalcul ()
		{
 
			//En fait ici j'ai l'initialisation de tous mes parametres, mais ceci fonctionne bien donc je passe sous silence.
		}
 
		public void Calculer()
		{
 
			//instructions sans importance
 
 
			for(int i = 0 ; i < 100 ; i++)
			{ 
				lock(this)
				{
 
					Calculer_Alertes();
				}
			}
 
			//instructions sans importance
 
		}
 
		private void							Calculer_Alertes()
		{
 
			string strSQL = "CALCUL_ALERTE ";
 
			//data est une classe perso qui fonctionne très bien, le pb ne peux pas venir d'elle
			data dat = new data(this.connect);
 
			dat.Exec(strSQL);
 
 
 
 
		}
 
	}

à quoi sert 'lock' exactement?
L'ai-je utilisé dans un bon contexte?

[Sphax]

Salut,

Le lock permet de synchroniser plusieurs thread pour qu'ils ne puissent pas éxécuter un bout de code en même temps (c'est un Mutex). Dans ton cas je pense que ça ne sert à rien vu qu'il n'y a qu'un seul thread de lancé.

Par contre qque chose m'étonne : il n'y a aucune boucle dans ta fonction Calculer_Alertes, alors je suppose que le traitement long se fait coté serveur sql dans la procédure CALCUL_ALERTE ? Sans ça ton abort ne sert à rien vu que l'exécution de ton thread s'arrete dés le retour de la procédure stockée. Si c'est effectivement le cas et alors ton abort stoppe net l'exécution de la procédure stockée, ce qui est surement trés mauvais ...

[Drahu]

Très juste.
En fait, il y a bien une boucle dans la procédure, mais je l'ai effacée pour rendre une meilleure lisibilité.

Je vais, de ce pas, éditer mon message pour la rajouter. Merci



Comment puis-je faire pour exiger de finir l'execution de ma procédure stockée avant d'annuler?

[Sphax]

Ah ok, dans ce cas c'est mieux. Ce qu'il faut faire dans ton thread c'est pas une boucle infinie mais une boucle avec une condition sur une variable, du genre :

Code :

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

1
2
3
4
5
 
While( ContinuerExcecution )
{
    //Code du thread
}

Et dans ton thread principal, qd tu veux arreter ton thread tu fais :

Code :

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

1
2
 
ContinuerExcecution = false;

Comme ça tu es sûr que le dernier appel à ta procédure stockée sera terminé, tu vois l'idée ? Un thread.Abort n'est pas une bonne chose de maniere générale, c'est trop brutal, tu ne controles pas ce qui se passe, et tu laisses les choses dans un état "inconnu", il faut l'utiliser en cas de dernier recours, comme par exemple N secondes aprés avoir positioné ta variable à false, si ton thread n'est toujours pas terminé, alors il y a un probleme (la procédure bloque) et là tu fais un thread.abort, mais c'est extreme.

[Drahu]

Ok merci du coup de main.

Je commence tout juste les threads depuis aujourd'hui.
C'est vraiment pratique ....