Plusieurs parametres d'une fonction à passer dans un thread

Salut,

Je suis un tuto sur les thread et il est dit que quand on cree un thread avec cette fonction :
int pthread_create(pthread_t* thread, pthread_attr_t* attr, void*(*start_routine)(void*), void* arg);

on ne peut specifier qu'un seul parametre. Hors la fonction que je voudrais inscrire dans un nouveau thread comprend 4 parametres.

Comment dois-je m'y prendre ?

Tu peux créer un objet de type structure (une structure qui contiendra les données dont a besoin ta fonction), et tu passes son adresse comme argument. D'où le type void* (on peut lui passer ce qu'on veut).

Tu crées une structure avec 4 champs ou un tableau de 4 valeurs
tu alloues cette structure, tu initialise les champs avec tes paramètres
tu crées ton thread an passant le pointeur de cette structure
le thread est lancé, il reçoit le pointeur de la structure, récupère les 4 paramètres et n'oublie pas de libérer la structure

[Edit] Damned :aie: grillé par jeroman

Merci pour l'astuce!!

Vous me conseillez d'installer pthread ou bien d'utiliser CreateThread() de l'api win32 ?

Salut,

Pouvez vous m'expliquer comment passer un pointeur dans mon thread ?
Dans un premier temps j'essaye juste d'incrémenter un nombre dans mon thread que je passe en parametre, et que je veux renvoyer dans mon main.

J'ai cette fonction la :

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

DWORD WINAPI MaFonction(LPVOID Valeur) { int a;   a = *(int *)Valeur; printf("a = %d",a);   return 0; }     int main(int argc, char *argv[]) { HANDLE handleMonThread; DWORD dwGenericThread; int nombre=3;   handleMonThread = CreateThread(NULL,0,MaFonction,&nombre,0,&dwGenericThread); WaitForSingleObject(handleMonThread,INFINITE);   printf("\nfin, nombre = %d\n",nombre);   system("PAUSE"); return 0; }

---

Elle n'est pas correcte mais au moins elle compile bien. Si j'essaye d'incrémenter Valeur, le compilateur me fait une erreur du type :
ISO C++ forbids incrementing a pointer of type `void*'

Citation:

---

Envoyé par theclem35

ISO C++ forbids incrementing a pointer of type `void*'

---

Déjà, si tu codes en C, pourquoi utilises-tu un compilateur C++ ?

Code C -> Compilateur C
Code C++ -> Compilateur C++

Pour incrémenter, tu peux écrire :

Code:

---

(*(int *)Valeur)++;

---

Euh ben ouais j'avoue c'est bizarre je code toujours en C et je pensais que ca compilait toujours en C (enfin je pense)...

Les parentheses j'avais deja essayé et ca me faisai la meme erreur, j'ai refait un nouveau projet et ca fonctionne .........

Dans le meme esprit, comment passer une structure en entree/sortie dans une fonction gérée par un thread ?

Ce que je veux dire c'est comment écrire le prototype et récuperer ma valeur à l'intérieur de ma fonction ?? A moins que ce soit deja un pointeur ?

Quand on écrit (*(int *)Valeur) :
- (int *) cela veut dire que on transforme Valeur en int car precedemment c'était un (void *) de part le thread ?
- et le *(.....)Variable signifie donc que accede à la valeur contenue dans Valeur, c'est un pointeur quoi ?

=> Dans ce cas la, si on a obligatoirement une variable de type (void *) comment appelle t-on cela (pas un pointeur ?)??

Merci d'avoir lu mes multiples questions qui symbolisent l'état de mon cerveau à la vue des étoiles...

Par exemple ceci ne fonctionne pas :

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 26 27 28 29 30 31 32 33 34 35 36 37 38 39

typedef struct INFOSPARTIE { char NomPartie[23]; char IPServeur[16]; char IPClient[16]; char Port[6]; char Niveau[3]; char PseudoJ1[13]; char PseudoJ2[13]; } InfosPartie;       DWORD WINAPI MaFonction(LPVOID Valeur) { *(InfosPartie *)Valeur[0].IPClient = "TheIPClient"; *(InfosPartie *)Valeur[0].PseudoJ2 = "pseudoJ2"; return 0; }     int main(int argc, char *argv[]) { HANDLE handleMonThread; DWORD dwGenericThread; InfosPartie MaPartie[1];   MaPartie[0].NomPartie = "Ma super partie!!!"; MaPartie[0].IPServeur = "128.182.123.134"; MaPartie[0].PseudoJ1 = "Moi";   handleMonThread = CreateThread(NULL,0,MaFonction,&MaPartie,0,&dwGenericThread); WaitForSingleObject(handleMonThread,INFINITE);   printf("\nLe pseudo de J2 est = %s\n",MaPartie[0].PseudoJ2);   system("PAUSE"); return 0; }

---

Citation:

---

Envoyé par theclem35

Dans le meme esprit, comment passer une structure en entree/sortie dans une fonction gérée par un thread ?

---

Cela me parait bizarre comme question car pour moi, dans un monde idéal, le thread reçoit ses paramètres et n'a besoin de rien d'autre pour se réaliser et il ne retourne rien (je n'ai pas dit qu'il n'avait pas le droit d'informer son parent). Tous les paramètres forment un "contexte" et c'est tout.

Mais bon, ta question a une solution.

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 26 27 28 29 30 31 32 33 34 35

typedef struct { int a; char param2[40]; } StructParametre;   DWORD WINAPI MaFonction(void * ptr_Param) { StructParametre * Param = (StructParametre *)ptr_Param;   printf("Thread Name %s, a = %d\n", Param->param2, Param->a);   // incremente la parametre Param->a++;   printf("Thread Name %s is exiting\n", Param->param2); return 0; }   int main(int argc, char *argv[]) { HANDLE handleMonThread; DWORD ThreadId;   StructParametre param; param.a = 3; strcpy(param.param2, "ThreadName");   handleMonThread = CreateThread(NULL, 0, MaFonction, &param, 0, &ThreadId); WaitForSingleObject(handleMonThread, INFINITE);   printf("fin, nombre = %d\n", param.a);   system("PAUSE"); return 0; }

---

Attention, ce code n'a pas été testé, il y a peut être des coquilles. De plus, ce code ne sert qu'a montrer comment un thread pourrait retourner une info à son parent (en plus ce n'est qu'une méthode parmi tant d'autres).

Citation:

---

Quand on écrit (*(int *)Valeur) :
- (int *) cela veut dire que on transforme Valeur en int car precedemment c'était un (void *) de part le thread ?
- et le *(.....)Variable signifie donc que accede à la valeur contenue dans Valeur, c'est un pointeur quoi ?

---

Pour que l'on puisse accéder à l'objet, il faut que le compilateur sache de quel type d'objet il s'agit, puisqu'il a affaire ici à un pointeur sur void (c'est-à-dire qu'il peut s'agir de n'importe quel type, le compilateur ne peut donc pas accéder à l'objet). Par exemple, on ne lit/écrit pas un objet de type float ou double de la même manière qu'un int. Pareil pour manipuler des structures. Le compilateur doit savoir de quoi il s'agit.

Donc, comme on a à l'origine un pointeur sur void, on doit faire savoir au compilateur qu'il doit manipuler un int. D'où le cast (int*), qui indique au compilateur qu'à cette adresse se trouve un objet de type int.
Le *(...) accède ensuite à l'objet en question.
Et le ++ postfixé incrémente sa valeur de 1.

Citation:

---

=> Dans ce cas la, si on a obligatoirement une variable de type (void *) comment appelle t-on cela (pas un pointeur ?)??

---

Le fait que l'argument soit de type void* permet de faire passer ce qu'on veut à la fonction, aussi bien un objet ayant un type de base qu'une structure quelconque. C'est un pointeur générique, sur lequel on ne peut pas travailler directement (car il pointe sur un type inconnu justement). C'est pour ça qu'il faut caster.

Plus d'info sur le pointeur générique : http://rperrot.developpez.com/articles/c/genericite/

COucou,

C'est vraiment super interessant!

Du coup j'ai essayé de faire un peu "à l'imitation" et j'ai qu'UNE seule erreur ;)
Un petit miracle...

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 25 26 27 28 29 30 31 32 33 34 35 36 37

typedef struct INFOSPARTIE // Cette structure va servir a contenir toutes les infos des parties trouvees sur le reseau { char NomPartie[23]; // On met tout en char a cause des conflit de int char IPServeur[16]; // qui dependent du bus de données et qui sont donc char IPClient[16]; // pas forcement compatibles pour une transmission entre 2 PC. char Port[6]; char Niveau[3]; char PseudoJ1[13]; char PseudoJ2[13]; } InfosPartie;   DWORD WINAPI MaFonction(void * thPartie) // Avec un thread on a un type void * (on peut mettre ce qu'on veut) { InfosPartie * PartieCree = (InfosPartie *)thPartie; // on fait un cast pour indiquer qu'on veut travailler sur une structure InfosPartie   // mon code strcpy(PartieCree.NomPartie, "Ma Super PArtie!");   return 0; }   int main(int argc, char *argv[]) { HANDLE handleMonThread; DWORD ThreadId; InfosPartie MaPartie;   strcpy(MaPartie.IPServeur, "192.168.10.1");   handleMonThread = CreateThread(NULL, 0, MaFonction, &MaPartie, 0, &ThreadId); WaitForSingleObject(handleMonThread, INFINITE);   printf("Fin du thread, IPServeur = %s, Nom de la partie = %s\n",MaPartie.IPServeur,MaPartie.NomPartie);   system("PAUSE"); return 0; }

---

Le compilateur me dit "request for member `NomPartie' in something not a structure or union " dans MaFonction.

Je comprend pas j'ai bien fait le cast, j'ai bien dit que ma variable était de type InfosPartie :calim2:

Sinon petite question. Si j'ai bien compris la fonction WaitForSingleObject(handleMonThread, INFINITE); permet que les fonctions suivantes ne s'éxécutent pas en parallele de mon thread, c'est à dire qu'il attend qu'il soit terminé, sans que ca fasse freezer tout le programme ?

Citation:

---

Envoyé par theclem35

Le compilateur me dit "request for member `NomPartie' in something not a structure or union " dans MaFonction.

Je comprend pas j'ai bien fait le cast, j'ai bien dit que ma variable était de type InfosPartie :calim2:

---

Le problème est ici :

Code:

---

strcpy(PartieCree.NomPartie, "Ma Super PArtie!");

---

NomPartie est un membre de la structure InfosPartie. Or, PartieCree est un pointeur sur un objet de type structure, pas l'objet en lui-même. Il faut donc le déréférencer pour pouvoir accéder aux membres de la structure :

Code:

---

strcpy((*PartieCree).NomPartie, "Ma Super PArtie!");

---

ou

Code:

---

strcpy(PartieCree->NomPartie, "Ma Super PArtie!");

---

C'est la même chose en fait. Les deux lignes suivantes sont équivalentes :

Code:

---

1 2	(*a).b a->b

---

Citation:

---

Sinon petite question. Si j'ai bien compris la fonction WaitForSingleObject(handleMonThread, INFINITE); permet que les fonctions suivantes ne s'éxécutent pas en parallele de mon thread, c'est à dire qu'il attend qu'il soit terminé, sans que ca fasse freezer tout le programme ?

---

Je suis pas super compétent en threads, mais il me semble effectivement que INFINITE comme second argument oblige à mettre en pause le thread dans lequel la fonction WaitForSingleObject se trouve, jusqu'à ce que le thread, identifié comme premier argument (ici : handleMonThread), se termine. Si je me trompe, on me corrigera.

EDIT :

Code:

---

InfosPartie * PartieCree = (InfosPartie *)thPartie;

---

Ce cast, en rouge, n'est pas obligatoire. Les conversion void* vers type* et vice versa sont implicites en C.

Ok merci pour toutes ces infos !

Pour les flèches ce sont des notions qu'on a pas vraiment apprises je prefere dans un premier temps ne pas les utiliser.

Du coup mon code devient :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

DWORD WINAPI MaFonction(void * thPartie) // Avec un thread on a un type void * (on peut mettre ce qu'on veut) { InfosPartie * PartieTemp = thPartie; // on fait un cast pour indiquer qu'on veut travailler sur une structure InfosPartie InfosPartie PartieCree;   PartieCree = *PartieTemp;   // mon code strcpy(PartieCree.NomPartie, "Ma Super PArtie!");     // fin du code   *PartieTemp = PartieCree;   return 0; }

---

C'est un peu moins joli mais plus compréhensible pour moi, comme j'ai dis dans un premier temps :)
Et puis c'est surtout que ca m'évite de revoir tout mon code que je dois mettre dans MaFonction ;)

Dans ma fonction appellé par mon thread, j'ai besoin de savoir s'il y a eu une erreur (en plus des messagebox).
Pour chaque erreur possible je met une variable à -1.

A l'origine je faisai un return de cette variable mais cette fois ci avec le thread il me semble que c'est impossible le return ne donnera rien ??

Suis-je donc obligé de mettre aussi ma variable de code de retour dans ma structure en parametre du thread ?

Citation:

---

A l'origine je faisai un return de cette variable mais cette fois ci avec le thread il me semble que c'est impossible le return ne donnera rien ??

---

Tu peux sortir du thread avec la fonction ExitThread, avec, comme argument, le code retour, puis le récupérer dans un autre thread grâce à la fonction GetExitCodeThread.

EDIT:
Je viens d'essayer avec un return. Cela marche aussi mais je ne sais pas si c'est garanti.

Si je comprends bien pour chaque erreur possible dans ma fonction, il faut que j'appelle autant de fois qu'il peut y avoir d'erreur :

TerminateThread(th, -1);

A ce moment la dans mon main, après l'appel de mon thread je regarde la valeur de retour avec :

GetExitCodeThread(th, &code);

Et si le code est -1 alors la je peux reboucler ce que je demandais à l'utilisateur. Dans le cas inverse, je continue l'execution normale de mon programme?