Y'a quelque chose qui m'échappeRésolu

Question

Bonjour,



&#231;a faisait longtemps que je n'avais pas fait du win32 pur et dur (merci wxWidgets).



Voulant un peu structurer mon code, j'ai voulu essayer de faire le
maximum avec des classes, et je me suis cass&#233; les dents d&#232;s le d&#233;but:

Voici en gros le code :



class CApp

{

public:

    CApp(HINSTANCE hInst);

    ~CApp(void);

    bool Initialize(WNDPROC WndProc);

private:

    HINSTANCE p_hInst;

    HWND p_hWnd;

};



CApp::CApp(HINSTANCE hInst)

{

    if (hInst==NULL)

        p_hInst=GetModuleHandle(NULL);

    else

        p_hInst=hInst;

    p_hWnd=NULL;

}





bool CApp::Initialize(WNDPROC WndProc)

{

    WNDCLASSEX wcex;



    ZeroMemory(&wcex,sizeof(WNDCLASSEX));

    wcex.cbSize = sizeof(WNDCLASSEX); 



    wcex.style            = CS_HREDRAW | CS_VREDRAW;

    wcex.lpfnWndProc    = WndProc;

    wcex.cbClsExtra        = 0;

    wcex.cbWndExtra        = 0;

    wcex.hInstance        = p_hInst;

    wcex.hIcon            = LoadIcon(p_hInst,(LPCTSTR)IDI_MAINICON);

    wcex.hCursor        = LoadCursor(NULL,IDC_ARROW);

    wcex.hbrBackground    = (HBRUSH)(COLOR_BTNFACE+1);

    wcex.lpszMenuName    = NULL;

    wcex.lpszClassName    = szWindowClass;

    wcex.hIconSm        = LoadIcon(p_hInst,(LPCTSTR)IDI_SMALL);



    RegisterClassEx(&wcex);



    p_hWnd = CreateWindow(szWindowClass, szTitle, WS_OVERLAPPEDWINDOW,

      CW_USEDEFAULT, 0, CW_USEDEFAULT, 0, NULL, NULL, p_hInst, NULL);

    if (!p_hWnd) return false;



    ShowWindow(p_hWnd,SW_SHOW);

    UpdateWindow(p_hWnd);



    return true;

}



J'ai ensuite (dans une autre fichier cpp), une fonction WNDPROC toute simple.

et je fais ceci dans la WinMain:

int APIENTRY _tWinMain(HINSTANCE hInstance, HINSTANCE hPrevInst,LPTSTR lpCmdLine,int nCmdShow)

{

    MSG msg;

    HACCEL hAccelTable;



    // Initialize global strings

    LoadString(hInstance, IDS_APP_TITLE, szTitle, MAX_LOADSTRING);

    LoadString(hInstance, IDS_WNDCLASS, szWindowClass, MAX_LOADSTRING);



    // Cr&#233;ation de l'application

    CApp TheApp(hInstance);

    if (TheApp.Initialize(MainWindowProc)==false)

        return FALSE;



    hAccelTable = LoadAccelerators(hInstance, (LPCTSTR)IDC_ACCELL);



    // Main message loop:

    while (GetMessage(&msg, NULL, 0, 0)) 

    {

        if (!TranslateAccelerator(msg.hwnd, hAccelTable, &msg)) 

        {

            TranslateMessage(&msg);

            DispatchMessage(&msg);

        }

    }



    return (int) msg.wParam;

}



Le probl&#232;me, c'est que j'obtiens un Acces
Violation lors de l'appel au RegisterClassEx dans CApp::Initialize, et
je n'arrive pas &#224; savoir pourquoi.



Est-ce d&#251; au fait que je passe la WNDPROC en param&#232;tre ?



Merci d'avance.

BruNews · Accepted Answer

essaie en les d&#233;clarant extern de la m&#234;me mani&#232;re que l'original, devrait aller mieux.

extern szWindowClass[MAX_LOADSTRING];

ciao...
BruNews, MVP VC++

cs_Xaviou · Answer

J'ai trouv&#233; : &#231;a viens des variables szWindowClass et szTitle que VC m'a cr&#233;&#233; en d&#233;but de projet de la fa&#231;on suivante :

#define MAX_LOADSTRING 100

TCHAR szWindowClass[MAX_LOADSTRING];

TCHAR szTitle[MAX_LOADSTRING];



Pour les utiliser avec ma classe CApp, j'avais ajout&#233;, en d&#233;but du fichier cpp :

extern TCHAR *szWindowClass,*szTitle;



J'ai remplac&#233; ces deux variables par un texte entre guillemets, et plus de plantage.

Par contre, si quelqu'un pouvait m'expliquer quel &#233;tait le probl&#232;me, &#231;a serait bien cool.

Merci d'avance.

BruNews · Answer

OUPS, oubli du type dans la copie:
extern TCHAR szWindowClass[MAX_LOADSTRING];

ciao...
BruNews, MVP VC++

cs_Xaviou · Answer

Je pensais pourtant qu'en d&#233;clarant un tableau, on pouvait utiliser le nom de ce tableau comme pointeur vers la 1&#232;re entit&#233;.



En tout cas, &#231;a marche en respectant la d&#233;claration d'origine.

Merci BruNews

BruNews · Answer

la raison est simple &#224; comprendre (si on a fait un peu d'asm...):

TCHAR szWindowClass[MAX_LOADSTRING];
est un tableau &#224; une adresse, le compilo &#233;crira:
mov dword ptr[esp+x], offset szWindowClass
qui est adresse du tableau.

TCHAR *szWindowClass,*szTitle;
mov dword ptr[esp+x], szWindowClass
et tu te retrouves avec les 4 premiers octets qui vont faire pointer sur une adresse invalide.

ciao...
BruNews, MVP VC++

Y'a quelque chose qui m'échappe

5 réponses

Votre réponse

Discussions similaires