Windows API SendMessage + WM_SETTEXT приводит к сбою - PullRequest
Новая
       8

Windows API SendMessage + WM_SETTEXT приводит к сбою

3 голосов
/ 12 сентября 2011

У меня проблемы с SendMessageA (дескриптор, текст WM_SETTEXT, 0, (LPARAM)); Это приводит к сбою, если используется где-либо еще, кроме конструктора.

MWC.h 

#include <Windows.h>
#include <iostream>
#include <string>

class MWC{
private:
    MSG msg;
public:
    MWC();
    ~MWC();
    int mLoop();
    int mClose(UINT);
    class System{
    public:
        System(){ }
        ~System(){ }
        class Form{
        public:
            HWND handle; // Need to access it in order to create other controls
            Form(char*,int,int,int,int);
            Form(char*,int,int);
            ~Form(){ }
            bool Show();
            HWND ReturnHandle();
        };
        class TextBox{
        protected:
            HWND handle;
        private:
            int ID;
        public:
            TextBox(char* text,int width,int height,int x,int y,int id,Form* p);
            TextBox(int width,int height,int x,int y,int id,Form* p);
            ~TextBox(){ }
            bool IsChanged(UINT,WPARAM);
            bool Text(char *);
            int GetId(){ return ID; }
        };

        class messageBox{
        public:
            messageBox(){ };
            ~messageBox(){ };
            bool Show(LPCWSTR,LPCWSTR,UINT);
            bool ShowA(char*,char*,UINT);
            bool ShowA(std::string,std::string,UINT);
        };
    };
};

MWC.cpp 

#include "MWC.h"

MWC::MWC(){ }
MWC::~MWC(){ }
int MWC::mLoop(){
    while(GetMessage(&msg,NULL,0,0)) { 
        TranslateMessage(&msg); 
        DispatchMessage(&msg);
    }
    return 1;
}
int MWC::mClose(UINT _msg){
    if(_msg == WM_CLOSE){PostQuitMessage(0);return 1;}
    return 0;
}
MWC::System::Form::Form(char* Title,int width,int height,int x,int y){
    handle =  CreateWindowEx(0,WC_DIALOG,Title,WS_OVERLAPPEDWINDOW | WS_VISIBLE,x,y,width,height,NULL,NULL,NULL,NULL);
}
MWC::System::Form::Form(char* Title,int width,int height){
    handle =  CreateWindowEx(0,WC_DIALOG,Title,WS_OVERLAPPEDWINDOW | WS_VISIBLE,400,100,width,height,NULL,NULL,NULL,NULL);
}

bool MWC::System::Form::Show(){
    return false;
}

HWND MWC::System::Form::ReturnHandle(){
    return handle;
}

MWC::System::TextBox::TextBox(char* text,int width,int height,int x,int y,int id,Form* p){
    ID = id;
    handle = CreateWindowEx(WS_EX_CLIENTEDGE,"EDIT",text,WS_VISIBLE | WS_CHILD,x,y,width,height,p->handle,(HMENU)id,NULL,NULL);
}
MWC::System::TextBox::TextBox(int width,int height,int x,int y,int id,Form* p){
    ID = id;
    handle = CreateWindowEx(WS_EX_CLIENTEDGE,"EDIT","",WS_VISIBLE | WS_CHILD,x,y,width,height,p->handle,(HMENU)id,NULL,NULL);
    SendMessageA(handle,WM_SETTEXT,0,(LPARAM)"It works here!");
}
bool MWC::System::TextBox::IsChanged(UINT message, WPARAM wParam){
    if(message==WM_COMMAND && HIWORD(wParam)==EN_CHANGE && LOWORD(wParam)==ID){
        return true;
    }
    return false;
}

bool MWC::System::TextBox::Text(char* text){
    if(handle == NULL){
        return 0;
    }
    else{
        //SendMessageA(handle,WM_SETTEXT,0,(LPARAM)text); // It doesn't work here!
        return 1;
    }
    return 1;
}


bool MWC::System::messageBox::Show(LPCWSTR text,LPCWSTR caption,UINT id){
    ::MessageBoxW(NULL,text,caption,id);
    return true;
}

bool MWC::System::messageBox::ShowA(char* text,char* caption,UINT id){
    ::MessageBoxA(NULL,(LPCSTR)text,(LPCSTR)caption,id);
    return true;
}

bool MWC::System::messageBox::ShowA(std::string text,std::string caption,UINT id){
    ::MessageBoxA(NULL,(LPCSTR)text.c_str(),(LPCSTR)caption.c_str(),id);
    return true;
}

main.cpp 

#include "MWC.h"

MWC mwc = MWC();
MWC::System::messageBox mbox = MWC::System::messageBox();
MWC::System::Form form1 = MWC::System::Form("Window",600,600);
MWC::System::TextBox textbox1 = MWC::System::TextBox(600,500,0,0,156,&form1);

BOOL WINAPI myProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam) 
{
    mwc.mClose(message);    

    //check if text in textbox has been changed by user
    textbox1.Text("Test");
    if(textbox1.IsChanged(message,wParam) == true){

    }
    return false; 
}





int APIENTRY WinMain(HINSTANCE, HINSTANCE, LPSTR, int)
{
    SetWindowLong(form1.ReturnHandle(), DWL_DLGPROC, (long)myProc); // to be added in the class
    mwc.mLoop();
    return 0;   
}

Дело в том, что sendmessage прекрасно работает (MWC.cpp): 

MWC::System::TextBox::TextBox(int width,int height,int x,int y,int id,Form* p){
    ID = id;
    handle = CreateWindowEx(WS_EX_CLIENTEDGE,"EDIT","",WS_VISIBLE | WS_CHILD,x,y,width,height,p->handle,(HMENU)id,NULL,NULL);
    SendMessageA(handle,WM_SETTEXT,0,(LPARAM)"It works here!"); // TESTING
}

Но это приводит к падению, если используется здесь: 

bool MWC::System::TextBox::Text(char* text){
    if(handle == NULL){
        return 0;
    }
    else{
        //SendMessageA(handle,WM_SETTEXT,0,(LPARAM)text); // It doesn't work here!
        return 1;
    }
    return 0;
}

Кстати, я просто любитель, который возится с winapi :). Не стесняйтесь комментировать любые найденные ошибки!

1 Ответ

4 голосов
/ 12 сентября 2011

Ваша диалоговая процедура выглядит неправильно. Вы пытаетесь отправить WM_SETTEXT каждый раз, когда запускается диалоговая процедура! Это, вероятно, создает бесконечную рекурсию и, возможно, множество других проблем.

Ваш SendMessage вызов в порядке, вы просто не можете сделать это каждый раз, когда запускается диалоговая процедура.

Я не понимаю, что такое функция IsChanged(). Дизайн вашего каркаса очень странный. Я предлагаю вам сначала взглянуть на некоторые другие фреймворки (например, ATL), чтобы получить некоторые идеи о том, как написать фреймворк для Win32. Или, что еще лучше, просто используйте существующий фреймворк.

...