Discussion :

[Dom_the_quaker]

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 :

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

 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 :

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

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 :

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
 
 
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

[poukill]

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 !

[Dom_the_quaker]

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!

[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".

[Emmanuel Deloget]

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 :

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
 
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).

[Dom_the_quaker]

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 :

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

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

et pas comme ca:

Code :

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

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!!

[Médinoc]

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

[Dom_the_quaker]

c'est pas faux....
merci

[Dom_the_quaker]

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 :

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

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 :

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
 
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

[Médinoc]

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

[Dom_the_quaker]

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

[Médinoc]

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.

[Emmanuel Deloget]

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*

[Dom_the_quaker]

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 :

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

 void recep_loop()

j'ai donc redeclaré en

Code :

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

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

[Médinoc]

C'est bien 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"

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

[Dom_the_quaker]

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