Обработка сообщений в win32 api

@hromjo · Регистрация: 04.04.2010

Написал программу которая после ввода в поле edit текста exit и нажатия на кнопку выходит из программы, но почему то не работает.
Вот код:

C++

#include "windows.h"
#define IDB_BUTTON 200
#define IDB_EDIT 201
int RegClass(WNDPROC Proc, LPCTSTR szName);
 
LRESULT CALLBACK _export WndProc(HWND, UINT, WPARAM, LPARAM);
 
HINSTANCE hInstance;
 
int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
{
    MSG msg;
    HWND hwnd;
    if(!RegClass(WndProc, "AppClass")) return FALSE;
    hwnd = CreateWindow("AppClass", "Programm", WS_OVERLAPPEDWINDOW|WS_VISIBLE, 
         CW_USEDEFAULT, CW_USEDEFAULT, 1280, 1024, NULL, NULL, hInstance, NULL);
    
    while (GetMessage(&msg, NULL, 0, 0))
    {
        TranslateMessage(&msg);
        DispatchMessage(&msg);
    }
 
    return (int) msg.wParam;
    
}
 
int RegClass(WNDPROC Proc, LPCTSTR szName)
{
    WNDCLASS wc;
    wc.style = CS_HREDRAW | CS_VREDRAW;
    wc.cbClsExtra = wc.cbWndExtra = 0;
    wc.lpfnWndProc = Proc;
    wc.hInstance = hInstance;
    wc.hIcon = LoadIcon(NULL, IDI_APPLICATION);
    wc.hCursor= LoadCursor(NULL, IDC_ARROW);
    wc.hbrBackground = (HBRUSH)(COLOR_WINDOW + 1);
    wc.lpszMenuName = (LPCTSTR)NULL;
    wc.lpszClassName = szName;
    return RegisterClass(&wc);
}
 
LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
{
        HWND hButton;
        HWND hEdit;
        CHAR *word;
        switch(message)
        {
        case WM_COMMAND:
             switch(LOWORD(wParam))
             {
             case IDB_BUTTON:
               GetWindowText(hEdit, word, 100);
                  if(word=="exit") PostQuitMessage(0);
             }
             
        case WM_CREATE:
             {
        hButton = CreateWindow("BUTTON","OK", WS_CHILD|WS_VISIBLE,
        790,500,70,25,hWnd,(HMENU)IDB_BUTTON,hInstance,NULL);
        hEdit = CreateWindow("EDIT", NULL, WS_CHILD|WS_VISIBLE|WS_BORDER,
        485,503,300,20,hWnd,(HMENU)IDB_EDIT,hInstance,NULL);
                 return 0;
            
            }
        case WM_DESTROY:{PostQuitMessage(0); return 0;}}
        return DefWindowProc(hWnd, message, wParam, lParam);               
}

@Somebody · 27.06.2010, 14:56

1) GetWindowText пишет в то, куда указывает word, а указывает он неизвестно куда.
2) hEdit - локальная переменная, при нажатии на кнопку он уже равен неизвестно чему. (Можно сделать его static, но это не слишком хорошо, потому что если создать несколько окон с этим классом и общей WndProc, то будут проблемы.)
3) В word == "exit" сравниваются указатели, для сравнения самих строк - strcmp.
4) В конце case WM_COMMAND надо бы break, а то дальше пойдёт выполняться.

@Digit@ll · 12.02.2011, 00:48

C++

#include "windows.h"
 
#define IDB_BUTTON 200
#define IDB_EDIT 201
 
int RegClass(WNDPROC Proc, LPCTSTR szName);
 
LRESULT CALLBACK _export WndProc(HWND, UINT, WPARAM, LPARAM);// непонятки какие-то с _export ((( так и не вьехал 
 
HINSTANCE hInstance;
 
int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
{
    MSG msg;
    HWND hwnd;
    
    if(!RegClass(WndProc, "AppClass"))
        
        return FALSE;
    
    hwnd = CreateWindow ("AppClass", "Programm", WS_OVERLAPPEDWINDOW | WS_VISIBLE,
                         0, 0, CW_USEDEFAULT, CW_USEDEFAULT, NULL, NULL, hInstance, NULL);
 
    
    while (GetMessage(&msg, NULL, 0, 0))
    {
        TranslateMessage(&msg);
        DispatchMessage(&msg);
    }
 
    return (int) msg.wParam;
    
}
 
int RegClass(WNDPROC Proc, LPCTSTR szName)
{
    WNDCLASS wc;
    wc.style         = CS_HREDRAW | CS_VREDRAW;
    wc.cbClsExtra    = wc.cbWndExtra = 0;
    wc.lpfnWndProc   = Proc;
    wc.hInstance     = hInstance;
    wc.hIcon         = LoadIcon(NULL, IDI_APPLICATION);
    wc.hCursor       = LoadCursor(NULL, IDC_ARROW);
    wc.hbrBackground = (HBRUSH)(COLOR_WINDOW + 1);
    wc.lpszMenuName  = (LPCTSTR)NULL;
    wc.lpszClassName = szName;
    return RegisterClass(&wc);
}
 
LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
{
    static  HWND hButton;//ВАЖНО!!! чтоб окна не терялись
    static  HWND hEdit;
 
    TCHAR word[MAX_PATH];
    
    switch(message)// приоритет соблюдаем
    {
        case WM_CREATE:// сначала "рисуем"
            
            hButton = CreateWindow ("BUTTON","OK", WS_CHILD|WS_VISIBLE,
                                    790,500,70,25,hWnd,(HMENU)IDB_BUTTON,hInstance,NULL);
 
              hEdit = CreateWindow ("EDIT", NULL, WS_CHILD|WS_VISIBLE|WS_BORDER,
                                    485,503,300,20,hWnd,(HMENU)IDB_EDIT,hInstance,NULL);
            break;
        
        case WM_COMMAND:// потом управляем, а не на оборот )))
            
            switch(LOWORD(wParam))
            {
            case IDB_BUTTON:
 
                GetWindowText (hEdit, word, MAX_PATH);
                if (lstrcmpi (word, "exit") ==0)// и все будет работать
                    PostQuitMessage(0);
            }
            break;
        
        case WM_DESTROY:
            {
                PostQuitMessage(0);
            }
            return 0;
    }
    return DefWindowProc(hWnd, message, wParam, lParam);
}

@Somebody 2830 / 1639 / 253 Регистрация: 03.12.2007 Сообщений: 4,222
	27.06.2010, 14:56	2
	1) GetWindowText пишет в то, куда указывает word, а указывает он неизвестно куда. 2) hEdit - локальная переменная, при нажатии на кнопку он уже равен неизвестно чему. (Можно сделать его static, но это не слишком хорошо, потому что если создать несколько окон с этим классом и общей WndProc, то будут проблемы.) 3) В word == "exit" сравниваются указатели, для сравнения самих строк - strcmp. 4) В конце case WM_COMMAND надо бы break, а то дальше пойдёт выполняться. 1

Опции темы