Discussion :

[Anais.serguier]

Bonjour à tous

Tout d'abord je vous explique mon architecture de code :

J'ai crée une classe, et celle-ci utilise les fonctions d'une dll qui a été développé par quelqu'un d'autre, j'ai donc accès au code source mais ne doit pas le modifier.

Dans ma classe, j'ai développé une méthode

Code :

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

void Test_Comm_Event(int event, void * param);

Dans une autre méthode, j'appelle une fonction de la dll :

Code :

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

int Initialize(Comm_Event_Fct* pFct)

Sachant que la déclaration de Comm_Event_Fct est :

Code :

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

typedef void Com_Event_Fct(int event, void * param);

L'argument passée à ma fonction Initialize est donc un pointeur sur une méthode ayant la même déclaration que la méthode évoquée au début que j'ai développé.

Or, ce que j'essaye donc est de passer un pointeur sur ma fonction Test_Comm_Event en argument pour réaliser un appel ressemblant à

Code :

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

n = Initialize(Test_Comm_Event)

J'ai essayé pas mal de chose qui me paraissait logique, malheureusement rien n'a fonctionné...

Si quelqu'un c'est comment réaliser ce passage d'argument, ou aurait une idée je suis preneuse

Merci !

[Bousk]

Tu dois passer un pointeur de fonction, l'opérateur pour récupérer un pointeur/une adresse est &, y compris pour les fonctions. n = Initialize(&Test_Comm_Event)
Fournir l'erreur ne serait pas superflu cela dit, pour les prochaines fois.

[Anais.serguier]

Déjà merci pour ta réponse

Mais... du coup j'ai déjà essayé avec '&'

Et mon erreur est la suivante : int Initiale

Code :

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

C2664	'int Initialize(Comm_Event_Fct (__cdecl *))'*: impossible de convertir l'argument 1 de 'void (__cdecl * )(int,void *)' en 'Comm_Event_Fct (__cdecl *)'

[Médinoc]

Je pense que le problème est la déclaration du typedef.
Généralement, les typedefs de pointeurs de fonction, on les déclares comme ça:

Code C :

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

typedef void (*Com_Event_Fct)(int event, void * param);

D'un autre côté, en C++ moderne, on déclarerait plutôt comme ça:

Code C++11 :

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

1
2
#include <functional>
typedef std::function<void(int, void*)> Com_Event_Fct;

[Anais.serguier]

Ok je note merci !

Pour l'instant, j'ai trouvé la solution en déclarant ma fonction en static, là il accepte une conversion du style :

Code :

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

Comm_Event_Fct* Test_Comm_Event_Ptr = (Comm_Event_Fct*) Test_Comm_Event;

Et donc après je peux passer Test_Comm_Event_Ptr en paramètre de ma fonction Initialize

[Bousk]

en déclarant ma fonction en static, là il accepte une conversion du style :

Code :

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

Comm_Event_Fct* Test_Comm_Event_Ptr = (Comm_Event_Fct*) Test_Comm_Event;

Je pense que ta conversion est un peu cavalière.
Mais les fonctions membres ont une signature qui n'a rien à voir avec avec les fonctions libres ou static.
https://cpp.developpez.com/faq/cpp/?...ur-de-fonction

[Médinoc]

Attention, un cast C-style de pointeur de fonction, ça montre TOUJOURS une erreur quelque part.
cf les deux liens dans ma signature: What can go wrong when you mismatch the calling convention? et A function pointer cast is a bug waiting to happen

[JolyLoic]

D'un autre côté, en C++ moderne, on déclarerait plutôt comme ça:

Code C++11 :

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

1
2
#include <functional>
typedef std::function<void(int, void*)> Com_Event_Fct;

Ça, ce serait le coup en C++ moderne où on a besoin de dynamicité, mais pas de performances. Ici, je n'ai pas l'impression que la dynamicité soit requise, et on écrirait alors plutôt :

Code :

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

1
2
template<class Fct>
int Initialize(Fct fct) {/*...*/}