Discussion :

[darioo2]

Bonjour,
Je voudrais construire une classe qui puisse initialiser un Hook et tout un tas d'autre choses, de manière à pouvoir la réutiliser dans mes applis par la suite.
Le problème est que quand je met ce code dans le main:

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
22
23
#include <QtCore/QCoreApplication>
#include <QDebug>
#include <Windows.h>
 
HHOOK hHook = NULL;
 
using namespace std;
LRESULT CALLBACK MyLowLevelKeyBoardProc(int nCode, WPARAM wParam, LPARAM lParam)
{
    qDebug() << "Test Clavier!";
    return CallNextHookEx(hHook, nCode, wParam, lParam);
}
 
int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);
 
    hHook = SetWindowsHookEx(WH_KEYBOARD_LL, MyLowLevelKeyBoardProc, NULL, 0);
    if(hHook == NULL)
        qDebug() << "Hook failed !";
 
    return a.exec();
}

tout fonctionne bien, alors que quand je veut le créer dans une classe (avec séparation conventionelle dans le .h et .cpp ça bloque.

Voici mon .h:

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
22
23
24
25
 
#ifndef LOWLEVELHOOK_H
#define LOWLEVELHOOK_H
 
#include <QtCore>
#include <Windows.h>
#include <QDebug>
 
class LowLevelHook : public QObject
{
    Q_OBJECT
 
public:
    LowLevelHook();
 
private:
    // Attributs:
    HHOOK hookClavier;
 
    // Méthodes:
    LRESULT CALLBACK MyLowLevelKeyBoardProc(int nCode, WPARAM wParam, LPARAM lParam);
 
};
 
#endif // LOWLEVELHOOK_H

Et mon .cpp:

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
 
#include "lowlevelhook.h"
 
LowLevelHook::LowLevelHook() : hookClavier(NULL)
{
    hookClavier = SetWindowsHookEx(WH_KEYBOARD_LL, MyLowLevelKeyBoardProc, NULL, 0);
    if(hookClavier == NULL)
        qDebug() << "Hook Clavier Failed !";
}
 
LRESULT LowLevelHook::MyLowLevelKeyBoardProc(int nCode, WPARAM wParam, LPARAM lParam)
{
    qDebug() << "Test Clavier!";
 
    return CallNextHookEx(hookClavier, nCode, wParam, lParam);
}

Erreur sur la ligne 6 du .cpp:

error: argument of type 'LRESULT (LowLevelHook::)(int, WPARAM, LPARAM)' does not match 'LRESULT (*)(int, WPARAM, LPARAM)'

[ternel]

error: argument of type 'LRESULT (LowLevelHook:(int, WPARAM, LPARAM)' does not match 'LRESULT (*)(int, WPARAM, LPARAM)'

C'est un problème de statisme.
tu dois donner un pointeur de fonction à SetWindowsHookEx(), hors, tu donne un pointeur de fonction membre.

Si f est un void (*)(int), alors f(1); est un appel valable
Au contraire, si f est un void (C::*)(int), alors f(1); n'est pas un appel valable, parce que f est utilisable sur un objet du type C (ou héritier).

Ici, tu peux envisager soit hookClavier = SetWindowsHookEx(WH_KEYBOARD_LL, this->MyLowLevelKeyBoardProc, NULL, 0);, soit de rendre MyLowLevelKeyBoardProc statique, mais tu y perds tout l'intéret.

[darioo2]

Bonjour et merci pour ta réponse,
Pour ce qui est de:

Code :

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

hookClavier = SetWindowsHookEx(WH_KEYBOARD_LL, this->MyLowLevelKeyBoardProc, NULL, 0);

ça ne fonctionne pas, ça me donne exactement la même erreur.
Et comme tu le dis, en rendant statique MyLowLevelKeyBoardProc je perd tout l'intérêt et ça implique pas mal de modifications derrière.

C'est quelque chose que j'ai réussit a faire sans trop de problèmes en C# mais là pour le coups, je suis complètement bloqué

Est ce qu'il n'y a aucune autre solution ?

[darioo2]

Bon, j'ai essayé de mettre MyLowLevelKeyBoardProc en static, mais du coups il n'accepte plus le hookClavier dansl e return, et si je le met aussi en static, ça me fais une belle série d'erreurs.

J'ai essayé de faire une fonction à l'extérieur de la classe, mais il y a un problème avec le hookClavier, je ne sais vraiment plus comment faire, quelqu'un à une solution svp je suis vraiment bloqué
Merci d'avance !

[Médinoc]

J'ai bien peur que tu aies besoin d'une variable globale/statique/singleton pour relier les données de ton hook à la fonction (qui elle, doit être statique pour être acceptée par SetWindowsHookEx()).

Personnellement, j'envisagerais une classe mère template CRTP pour gérer le static/singleton, et faire le lien entre la fonction statique et celle réimplémentée dans la classe fille (virtuelle ou CRTP).