Discussion :

[DakM]

Bonjour !

Etant en cure intensive de désingletonisation, j'ai besoin de passer plus paramètres à un pthread!
Or, la fonction qui lance les thread n'en prend qu'un seul, donc:

Code :

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

1
2
3
4
5
struct PtrCollection
{
    SessionsCollection* sc;
    Selector* sl;
};

La fonction qui lance le thread:

Code :

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

1
2
3
4
5
6
7
8
9
 
    // Threads
    PtrCollection* pc = (PtrCollection*)malloc(sizeof(PtrCollection*)) ;
    pc->sc = (SessionsCollection*)malloc(sizeof(SessionsCollection*));//sessions;
    pc->sc = sessions;
 
    pc->sl = (Selector*)malloc(sizeof(Selector*));//sessions;
    pc->sl = selector;
    pthread_create(&t_networker, NULL, Networker, pc);

Rien que ici, je me dis que ya un gros problème au niveau de la conception de mon problème...
Et c'est evident que le passage ne marche pas, crash au démarrage du thread.

Cette solution est-elle convenable?
Faudrait-il que je fasse une classe mère à chaque classe (ici: SessionsCollection & Selector) et une classe de collection contenant un vector<classMere>, et que dans mon thread je puisse faire:

Code :

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

1
2
classDeCollection* cdc = reinterpret_cast<classDeCollection*>(argument)
Selector* sl = cdc->get(0); // retourner le vecteur.at(0);

[Neckara]

On est en C++, on utilisera plutôt new et delete plutôt que malloc et free.

Il faudrait que tu nous montre toutes tes sources (surtout les prototypes de tes fonctions).

Sinon, il faudrait vraiment que tu revoies les pointeurs :

Code :

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

1
2
 
pc->sc = sessions;

Soit sessions est un pointeur et dans ce cas là, faire

Code :

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

1
2
pc->sc = (SessionsCollection*)malloc(sizeof(SessionsCollection*));

ne sert à et est complètent faux, on doit allouer un multiple de sizeof(SessionsCollection)

Il faut que tu fasses :

Code :

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

1
2
 
*pc->sc = *sessions;

Soit ce n'est pas un pointeur et tu dois faire :

Code :

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

1
2
 
*pc->sc = sessions;

Tu répètes cette erreur pour

Code :

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

1
2
 
pc->sl = selector;

Ensuite, tu transmet à pthread_create un PtrCollection* et non un void *, il faut donc que tu effectues une conversion (en C ça passe mais en C++, il ne me semble pas).


De plus l'utilisation de phtread n'est pas très C++, on préfèrera souvent utiliser d'autres fonctions d'autres bibliothèques permettant de lancer directement une méthode.

Sinon, tu peux toujours passer un pointeur sur une instance de classe et utiliser ta fonction void machin(void * data) pour lancer une méthode de cette classe.

Après théoriquement tu ne devrais pas avoir de problèmes d'échanges de données.