Discussion :

[f-k-z]

Bonjour,
j'ai actuellement le code suivant:

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
#include <stdio.h>
#include <stdlib.h>
#include <windows.h>
#include <conio.h>
 
DWORD WINAPI monthread(LPVOID);
 
DWORD WINAPI monthread(LPVOID lpParameter){
	int i;
	for (i=30;i>=0;i--){
		printf("il reste %d seconde(s)\n",i);
		Sleep(1000);
	}
	return 0;
}
 
int creationthread(){
	HANDLE handlemonthread;
	DWORD ID1;
	printf("On cree notre thread\n");
	handlemonthread=CreateThread(NULL,0,monthread,0,0,&ID1);
	while (handlemonthread!=0){
		if (kbhit()){
			return 0;
		}
	}
	return 0;
}
 
int main(){
	printf("essai de la creation d'un thread\n");
	creationthread();
	printf("Fin du programme\n");
	getchar();
	return 0;
}

LE but est de faite un decompte de 30 jusqu'a 0 et si l'utilisateur appui sur une touche le programme arrete le decompte et va se terminer.
Cependant, si j'appui sur une touche pendant le décompte, mon thread reste actif.
Comment faire pour savoir d'une part s'il est actif (car ma boucle while tournera toujours, visiblement il ne faut pas recuperer l'etat de handlemonthread) et d'autre part, comment fermer mon thread?

Cordialement,

F-k-z

[loufoque]

Tuer un thread en cours d'exécution, c'est mal.

[f-k-z]

Oui, je sais, c'est mal de tuer un thread, mais tanpis
Parcontre comment savoir si celui-ci est toujours vivant?
cordialement,

F-k-z

[Médinoc]

WaitForSingleObject() sur le handle du thread: Avec un timeout nul, tu peux tester s'il est toujours actif. Avec un timeout infini, tu peux t'assurer qu'il s'arrête bien avant que la fonction quitte.
Et pour couper le thread, tu peux utiliser un Event.
Si tu remplaces le Sleep() par un WaitForSingleObject() sur l'Event, il te suffira de tester le résultat: Si ce n'est pas un timeout, alors le thread doit s'arrêter.

PS: Je te conseille un timeout non-nul sur le thread principal (celui qui attend une touche), pour éviter une attente active.

[f-k-z]

Yop,
merci pour la réponse mais je vois pas trop comment l'implémenter, ca donnerai quoi pour ma boucle while en fait?
en remplacant mon handlemonthread!=0 par la vérificaiton qu'il est tjs vivant?

F-k-z

[Médinoc]

Je dirais un truc de ce genre:

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
DWORD WINAPI monthread(LPVOID);
 
DWORD WINAPI monthread(LPVOID lpParameter){
	HANDLE hEvent = static_cast<HANDLE>(lpParameter);
	int i;
	for (i=30 ; i>=0 ; i--){
		printf("il reste %d seconde(s)\n",i);
		if(WaitForSingleObject(hEvent, 1000) != WAIT_TIMEOUT)
			break;
	}
	return 0;
}
 
int creationthread(){
	HANDLE hThread, hEvent;
	DWORD ID1;
	hEvent = CreateEvent(NULL, FALSE, FALSE, NULL);
	if(hEvent == NULL)
		return -1;
	printf("On cree notre thread\n");
	hThread=CreateThread(NULL, 0, monthread, hEvent, 0, &ID1);
	if(hThread == NULL)
	{
		CloseHandle(hEvent);
		return -1;
	}
	while (WaitForSingleObject(hThread, 1)==WAIT_TIMEOUT) {
		if (kbhit()){
			SetEvent(hEvent); //On dit au thread de se terminer vite
			break;
		}
	}
	//On attend que le thread soit vraiment terminé avant nettoyage
	WaitForSingleObject(hThread, INFINITE);
	CloseHandle(hEvent);
	CloseHandle(hThread);
	return 0;
}
 
int main() {
	printf("essai de la creation d'un thread\n");
	creationthread();
	printf("Fin du programme\n");
	getchar();
	return 0;
}

Note: Pour être pleinement compliant avec Windows, il faudrait utiliser _beginthreadex() au lieu de CreateThread() et caster son résultat en HANDLE avec reinterpret_cast<>.
Pourquoi? Parce qu'on utilise la bibliothèque runtime C/C++ dans le nouveau thread : On appelle la fonction printf().

[f-k-z]

Yop,
Merci beaucoup pour ta réponse, en effet j'ai pas mal de progrès à faire la-dedans
cordialement,

f-k-z

[loufoque]

À part ça, pourquoi tu n'écris pas du beau code C++ portable ?
Boost.Thread a même un timed_join...

[Médinoc]

Peut-être parce que ça ne sert à rien puisqu'il a besoin d'un kbhit() ?

[loufoque]

Quand bien même, au moins le code ne serait pas immonde.

[Médinoc]

C'est vrai qu'ici, c'est codé "à la C", Windows n'ayant pas de wrapper C++ gratuit de son API.

[f-k-z]

Yop,
effectivement, faudrait peut-être que je me tourne plus vers du C++ portable.
En tout cas merci pour les réponses j'ai plus qu'à tout faire passer en résolu

F-k-z

[loufoque]

La payante n'est pas mieux.