Classe et thread

Bonjour,
j'ai créé une classe qui lors de son instenciation crée un thread lui correspondant. Or j'ai un probleme a la compilation:

Code:

---

error: argument of type 'void* (Trs232::)(void*)' does not match 'void* (*)(void*)'

---

Il semble que ce probleme soit lié au faite que la bibliotheque pthread (posix) soit ecrite en C et que les threads posix ne gerent pas le C++ (c'est ce que j'ai lu sur un forum anglophone) Dans ce meme forum, il etait conseillé de déclarer une fonction globale pour appeler le thread.
Mon probleme: La fonction qui appelle le thread est appelée par le constructeur, donc celle la je peux la mettre en globale avec une declaration d'amitié dans la classe, mais le contenu de mon thread, vu qu'il doit etre particulier a chaque instence de classe, je fais comment?

mon code:

Code:

---

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

int Trs232::init_thread() ///Renvoie O si création du thread ok, 1 si Erreur. {     int erreur; int tid=1; // ca c'est juste un id que j'ai mis a 1 pour l'instant je vais le faire //correspondre au nombre d' instencie par la suite   erreur = pthread_create (&thread_com, NULL, recep_loop, (void*)tid);     return erreur; }

---

La fonction threadée:

Code:

---

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

void* Trs232::recep_loop(void *threadid) { int Result; char* pChaine; int tid; // inutile pour le moment tid = (int)threadid;   do { Result=Hard_Rx(pChaine); if (Res > 0) {   Write_Buffer_Rx(pChaine); } } while (true);   }

---

ben merci pour vos lumieres :o)
++ Dom

Si tu veux une lib qui fasse les threads en C++, il y a Boost.Thread ! ;)

Boost.Thread + Boost.Bind est un ensemble assez puissant ! :D

Enorme !!! :-)

Ca c'est bon, je viens de regarder, en plus il est directement telechargeable via apt-get :-)
donc tout benefs, du coup ca va etre plus "souple"
et bien merci du conseil et zou au boulot.
A+
bon mon probleme sera contourné donc du coup je mets resolu!

hello,
Tu ne peux pas passer à pthread_create() l'adresse d'une fonction membre d'une classe. Il faut sortir ta fonction recep_loop() de la classe ou la rendre "static".

Citation:

---

Envoyé par jmv

hello,
Tu ne peux pas passer à pthread_create() l'adresse d'une fonction membre d'une classe. Il faut sortir ta fonction recep_loop() de la classe ou la rendre "static".

---

Ou créer une fonction statique qui prends en paramètre un pointeur sur une instance de l'objet thread:

Code:

---

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

int Trs232::init_thread() { int erreur; int tid=1; // ca c'est juste un id que j'ai mis a 1 pour l'instant je vais le faire //correspondre au nombre d' instencie par la suite   erreur = pthread_create (&thread_com, NULL, &Trs232::static_recep_loop, (void*)this); return erreur; } // cette fonction est statique à la classe void* Trs232::static_recep_loop(void *param) { Trs232* thread = (Trs232*)param; if (thread) return thread->recep_loop(); return NULL; }

---

C'est généralement comme ça qu'on fait (même boost.thread fait comme ça en fait, dès lors qu'il passe par une API native pour créer la thread et lui adjoindre une routine de traitement).

Bien le bonjour,
j'ai pu me rendre compte que ca ne marchait pas plus avec boost, il me fait des erreurs de pointurs aussi, du coup merci pour vos réponses! ca me premet de comprendre un peu mieux plutot que d'y aller a "tatons".

Emmanuel, pouruoi ta fonction statique est elle déclaré comme ceci:

Code:

---

1 2	void* Trs232::static_recep_loop(void *param)

---

et pas comme ca:

Code:

---

1 2	static void* Trs232::static_recep_loop(void *param)

---

c'est juste un oubli, ou c'est comme ca et la je ne comprends pas ... :(
le compilateur le sait comment?
merci d'avance!!

On ne met pas "static" dans la définition d'une fonction membre statique, seulement dans sa déclaration.

c'est pas faux.... :oops:
merci ;)

1- C'est encore moi, donc j'ai pris le temps avant de copier le code betement (meme si bon je ne sais pas comment vous pensez a faire des trucs pareils!!) d'essayer de comprendre, est ce que vous pouvez m'expliquez, pourquoi je ne peux pas appeler un thread directement dans ma classe? j'ai bien compris le mecanisme de la "pirouette" pour contourner ce probleme, mais la raison pour laquelle on fait ca n'est pas claire.

2- Lorsque je compile, j'ai une erreur:

Code:

---

1 2	Trs232.cpp:271: error: void value not ignored as it ought to be

---

et ca pointe sur l'appel de la fonction threadée:

Code:

---

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

int Trs232::init_thread() { int erreur; int tid=1; // ca c'est juste un id que j'ai mis a 1 pour l'instant je vais le faire //correspondre au nombre d' instencie par la suite   erreur = pthread_create (&thread_com, NULL, &Trs232::static_recep_loop, (void*)this); return erreur; } // cette fonction est statique à la classe void* Trs232::static_recep_loop(void *param) { Trs232* thread = (Trs232*)param; >>> if (thread) return thread->recep_loop(); <<<< return NULL; }

---

merci :)

C'est apparemment un warning du au fait que tu fais un return sur l'appel d'une fonction retournant void.

salut
Ben non... c'est pas un warning, le compilo bloque dessus et c'est ma seule erreur :?

C'est peut-être une erreur, mais la raison est la même: Tu cherches à "return" quelque chose, mais ce quelque chose, c'est void.

En C++, il me semble que ce genre de chose ne marche que si la fonction contenant le return possède elle-même le type de retour void.

Citation:

---

Envoyé par Dom_the_quaker

salut
Ben non... c'est pas un warning, le compilo bloque dessus et c'est ma seule erreur :?

---

Vérifie: ta fonction retourne void alors qu'elle devrait retourner void* :)

Je pense que mon probleme n'en est plus un ;)
merci a Medinoc et Emmanuel,
Dites moi si je me trompe:
ma fonction renvoyait void car je renvoyais thread->recep_loop() (en fait une fonction)
et que recep_loop () etait déclaré

Code:

---

void recep_loop()

---

j'ai donc redeclaré en

Code:

---

void* recep_loop()

---

et ca compile.
mais le fait que ca compile ne veut pas dire que ce que j'ai fait est correct. Est ce que c'est la bonne solution?
honnetement, je me suis juste dit je renvois une fonction alors qu'il faut un pointeur sur une fonction, donc j'ai declaré la fonction concernée de façon "adequate"
mais la reflexion a avoir n'est elle pas plus poussée? je sais que tout est lié du fait du thread et qu'il m'est impossible d'appeler la fonction membre dans ce thread, donc on passe, comme vous me l'avez montrer, par differents cast et une fonction static, mais la suite ne serait qu'une simple consequence de cette manip?
en tous les cas merci, et a bientot
dom

C'est bien la bonne solution.

Citation:

---

Envoyé par Dom_the_quaker

honnetement, je me suis juste dit je renvois une fonction alors qu'il faut un pointeur sur une fonction, donc j'ai declaré la fonction concernée de façon "adequate"

---

Tu ne retournes pas une fonction ni un pointeur de fonction, tu retournes un pointeur retourné par une fonction.

c'est vrai, ce sont des nuances, mais d' importance capitale pour comprendre le mecanisme. encore merci! :)