Discussion :

[BB_17]

Bonjour,

Je suis nouveau sur le forum... il est probable que le sujet ai été débattu, mais la fonction de recherche ne m'a pas permi de cerner rapidement un nombre suffisament restreint de fil à lire. Alors je soumets mon probleme :

Je souhaite modifier un handler (en fait substituer un handler par un autre, du même type) lors de l'excution d'un programme. Celui-ci est developpé avec VS 2005 en utilisant les MFC.

D'après ce que j'ai pu constater, la table des messages MFC est faite pour être statique. Mais j'ai également vu sur le net que des "workaround" était possible... seulement, plusieurs solutions tendent vers la reécriture d'un framework autorisant les tables dynamiques - too much pour moi.

Je veux seulement, pour un message en particulier, pouvoir remplacer un Handler_A associé à ce message par un Handler_B :
Soit une classe CTestAppDialog dérivant de CDialog,
Soit un message ON_EVENEMENT (n'importe quoi, un clic souris, une frappe clavier), on lui associe un Handler OnEvenementProcess_A() dans le
MESSAGE_MAP. Puis on souhaite remplacer, a l'exécution, OnEvenementProcessType_A() par OnEvenementProcessType_B() dans la table.
Comme il s'agit,finalement de pointeurs sur fonction (quelque part dans les structures complexes genre AFX_MSGMAP_ENTRY) ca devrait être possible.

Alors comment faire simplement ?
Merci d'avance
Bernard

[3DArchi]

Bonjour et bienvenu,
Je dirais à priori que c'est une mauvaise idée de vouloir trifouiller dans les structures internes des MFC. Rien ne te garantie une pérennité et tu risques de passer plus de temps en 'bidouille' qu'autre chose. Plusieurs solutions:
-> Sublcasser ta fenêtre ()
-> Récupérer le message dans un OnEvenementProcess qui ensuite invoque OnEvenementProcess_A ou OnEvenementProcess_B via un mécanisme interne que tu mets en place (pointeur de fonction, état, foncteur...).

[BB_17]

Je dirais à priori que c'est une mauvaise idée de vouloir trifouiller dans les structures internes des MFC. Rien ne te garantie une pérennité et tu risques de passer plus de temps en 'bidouille' qu'autre chose.

D'accord avec cela, mais c'est une action "one shoot", pour un developpement particulier, qui n'aura probalement pas de réutilisation ailleurs.

-> Récupérer le message dans un OnEvenementProcess qui ensuite invoque OnEvenementProcess_A ou OnEvenementProcess_B via un mécanisme interne que tu mets en place (pointeur de fonction, état, foncteur...).

En fait, c'est ce que je voulais eviter, car quelque soit le mécanisme, il y a une indirection (un test, l'appel d'une fonction, ...). Je souhaitais changer le pointeur sur fonction pour embrayer sur le bon handler dès reception du message, l'application est critique en temps de calcul, le handler en question est apellé environ 100 x par seconde.

Les evenements proviennent d'un ActiveX, et la "message map" à modifier est "EVENTSINK_MAP". En fait, j'arrive à décripter la table, mais je ne peux pas modifier le pointeur sur fonction. Il me semble que les macros MFC "DECLARE_EVENTSINK_MAP", "BEGIN_EVENTSINK_MAP", etc... utilisent partout le mot clé "const". Mes tentatives de modification de la table se soldent par un "0xC0000005: Violation d'accès lors de l'écriture" :

Code :

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

1
2
3
4
5
6
7
 
AFX_EVENTSINKMAP_ENTRY* my_pMap = (AFX_EVENTSINKMAP_ENTRY*)this->eventsinkMap.lpEntries;
 
// itération à l'interieur de la table des message
if(my_pMap[i].dispEntry.lDispID == 5 ) // c'est la liaison event-handler que je souhaite modifier
    my_pMap[i].dispEntry.pfn = (AFX_PMSG)&CTestAppDlg::OnNouveauHandler; 
// ici ca plante : 0xC0000005: Violation d'accès lors de l'écriture

Il me semblait que "const" servait de verification lors de la compilation ?
Or le code compile sans erreurs ... mais génére une erreur lors de l'accès à l'écriture lors de l'execution. Bizzare !

Any hints ?
Bernard

[3DArchi]

Bon, j'ai retrouvé la définition de BEGIN_EVENTSINK_MAP :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#define BEGIN_EVENTSINK_MAP(theClass, baseClass) \
    const AFX_EVENTSINKMAP* PASCAL theClass::_GetBaseEventSinkMap() \
        { return &baseClass::eventsinkMap; } \
    const AFX_EVENTSINKMAP* theClass::GetEventSinkMap() const \
        { return &theClass::eventsinkMap; } \
    const AFX_EVENTSINKMAP theClass::eventsinkMap = \
        { &theClass::_GetBaseEventSinkMap, &theClass::_eventsinkEntries[0], \
            &theClass::_eventsinkEntryCount }; \
    UINT theClass::_eventsinkEntryCount = (UINT)-1; \
    const AFX_EVENTSINKMAP_ENTRY theClass::_eventsinkEntries[] = \
    { \

Effectivement, le const ici veut dire aussi que le tableau n'étant pas modifiable dynamiquement, il sera placé dans une zone de données en lecture seule. Le système interdit l'écriture sur cette zone mémoire. Conclusion : tu ne peux pas modifier une entrée de ce tableau.

le handler en question est apellé environ 100 x par seconde.

Pas sur que le coût d'une indirection soit si critique par rapport à ce que fait ta fonction...

[Médinoc]

Un truc que tu pourrais faire, c'est contourner les macros BEGIN_MESSAGE_MAP, DECLARE_MESSAGE_MAP etc. pour créer ta propre implémentation de la fonction virtuelle GetMessageMap(), et en faire une fonction qui retourne une copie modifiable de la map. Cette copie pourrait être créée dans le constructeur de ta classe à partir de la vraie message map statique...

[BB_17]

Merci à 3DArchi pour avoir confirmé mes soupçons.

Un truc que tu pourrais faire, c'est contourner les macros BEGIN_MESSAGE_MAP, DECLARE_MESSAGE_MAP etc. pour créer ta propre implémentation de la fonction virtuelle GetMessageMap(), et en faire une fonction qui retourne une copie modifiable de la map. Cette copie pourrait être créée dans le constructeur de ta classe à partir de la vraie message map statique...

Oui mais je ne me sens pas assez costaud en C++/MFC pour faire cela, et on se rapproche de la reecriture d'un framework. En faisant des recherches j'étais tombé sur :
http://blog.emmanueldeloget.com/inde...remiere-partie
Avec une implementation de table de message dynamique.

Et puis je trouve étrange que ce genre de code (re-écritude de GetMessageMap()) ne traine pas sur le net... Les developpeurs utilisant les pointeurs sur fonction et les fans de Qt (avec connect(signal,slot)) on surement dû avoir envie de contourner les tables statiques.
A mon avis, ca doit être assez complexe.

Je pense que je vais laisser tomber et utiliser une indirection supplementaire (un pointeur sur fonction à moi, dans un handler unique)

On peut toutefois laisser le sujet ouvert quelque temps, we never know !!!!!

Bernard

[Médinoc]

Pour moi, c'est pourtant simple: Tu regardes les macros, puis tu fais la même chose en coupant les bouts qui te gênent...
S'il y a un moyen d'obtenir la taille d'une message map (et il y en a forcément un), alors il est aisé de la dupliquer dans ton constructeur, et mémoriser un pointeur vers elle en variable membre. Ensuite, il suffit de faire une fonction qui retourne ce pointeur.

[3DArchi]

Pour moi, c'est pourtant simple: Tu regardes les macros, puis tu fais la même chose en coupant les bouts qui te gênent...
S'il y a un moyen d'obtenir la taille d'une message map (et il y en a forcément un), alors il est aisé de la dupliquer dans ton constructeur, et mémoriser un pointeur vers elle en variable membre. Ensuite, il suffit de faire une fonction qui retourne ce pointeur.

Pour du one-shot, l'indirection me semble quand même la solution la plus rapide à mettre en oeuvre et présente l'avantage de la compréhension immédiate dans 6 mois par le petit nouveau qui viendra prendre le relais...