Discussion :

[Chop_chop]

Bonjour, j'ai un problème sans doute pas compliqué à résoudre mais sur lequel je cale depuis un moment.

Dans mon main, je souhaite lancer plusieurs threads d'une classe de la façon suivante :

main.cpp

Code :

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

1
2
3
4
5
6
7
8
9
...
for(i=0;i<nb_elem;i++)
 {
    if (fee[i]->GetType()=="File_feeder")
	 {
	  DWORD ThreadID;
 	  CreateThread (NULL , NULL, File_feeder::Thread_File,(LPVOID) fee[i]  ,NULL,&ThreadID);
	 } 
 }

avec ma thread déclarée de la façon suivante dans ma classe:
file_feeder.h

Code :

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

1
2
3
4
5
6
7
8
9
10
class File_feeder : public Feeder
{
   public:
            ...
            DWORD WINAPI Thread_File(LPVOID lpParam)
              {
                /*code*/
                return 0;
              }
 }

Hélas, j'ai une erreur sur le CreateThread que je n'arrive pas à résoudre meme en forcant le param 3 en mettant (LPTHREAD_START_ROUTINE) devant :

Code :

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

error C2664: 'CreateThread' : impossible de convertir le paramètre 3 de 'DWORD (__stdcall File_feeder::* )(LPVOID)' en 'LPTHREAD_START_ROUTINE'

et je ne réussi pas à la résoudre Si un connaisseur en threads a une piste, il est le bienvenu

[Chop_chop]

Autant pour moi,

en mettant static devant la déclaration du thread dans la classe, ça résout le problème.

Désolé du dérangement !

[Médinoc]

Rappel: Utiliser _beginthreadex() plutôt que CreateThread(), sauf dans des programmes faits spécialement pour ne pas utiliser la C/C++ Run-Time Library (et ces programmes sont l'exception, pas la règle).

De plus, le cast en LPVOID est inutile: Tous les pointeurs sont implicitement convertibles en void*, sauf les const void * (utiliser un const_cast<> dans ce cas) et les pointeurs de fonction.

PS: La valeur de retour de CreateThread() et _beginthreadex() est un HANDLE, qui doit absolument être fermé avec CloseHandle() avant d'etre "oublié"...

[Chop_chop]

Merci pour l'info !!