Почему дочерние окна MDI не существуют после WM_NCCREATE? - PullRequest
Новая
       2

Почему дочерние окна MDI не существуют после WM_NCCREATE?

0 голосов
/ 23 октября 2019

Я пишу графическое приложение MDI с использованием OpenGL. У меня есть класс ControllerGL, основанный на классе Controller, который будет иметь методы (для некоторых из которых потребуются свои собственные потоки) для рисования для потомков MDI. При создании дочерних окон я устанавливаю cbWndExtra из WNDCLASSEX в sizeof(Win::Controller*) и извлекаю указатель на класс ControllerGL в WM_NCCREACTE процедуры дочернего окна, используя Set/GetWindowLongPtr().

У меня естьэто работает для SDI, поэтому я совершенно уверен, что весь код для установки / извлечения указателя на класс ControllerGL в порядке, и в приложении MDI отображается главное окно, и я получаю работающий экземпляр ControllerGL класс из функции WM_NCCREATE процедуры дочернего окна MDI.

Таблица меню и строк в файле resource.rc: 

IDM_MDI MENU
BEGIN
    POPUP "&File"
    BEGIN
        MENUITEM "&New",                        ID_FILE_NEW
    END
    POPUP "&Window",
    BEGIN
        MENUITEM "&Cascade",                    ID_WINDOW_CASCADE
        MENUITEM "Tile &Horizontal",            ID_WINDOW_TILEHORIZONTAL
        MENUITEM "Tile &Vertical",              ID_WINDOW_TILEVERTICAL
        MENUITEM "Arrange &Icons",              ID_WINDOW_ARRANGEICONS
    END
END

STRINGTABLE
BEGIN
    IDS_MDI_TITLE           "Win32 MDI"
    IDS_MDI_CLASSNAME       "MDIMAIN"
    IDS_MDICHILD_TITLE      "MDIChild"
    IDS_MDICHILD_CLASSNAME  "MDICHILD"
END

Это для файла resource.h 

#define IDI_MDI                         101
#define IDI_MDI_SMALL                   102
#define IDI_MDICHILD                    103
#define IDI_MDICHILD_SMALL              104
#define IDS_MDI_TITLE                   105
#define IDS_MDI_CLASSNAME               106
#define IDS_MDICHILD_TITLE              107
#define IDS_MDICHILD_CLASSNAME          108
#define IDM_MDI                         109
#define ID_FILE_NEW                     110
#define ID_WINDOW_CASCADE               111
#define ID_WINDOW_TILEHORIZONTAL        112
#define ID_WINDOW_TILEVERTICAL          113
#define ID_WINDOW_ARRANGEICONS          114
#define IDC_MDICHILD_FIRST              50000

Классы ViewGL и ModelGL для класса ControllerGL: 

namespace Win
{
    class ViewGL
    {
    public:
        ViewGL();
        ~ViewGL();
    };
}

class ModelGL
{
public:
    ModelGL();
    ~ModelGL();
};

Класс Controller: 

#include <windows.h>
namespace Win
{
    class Controller
    {
    public:
        Controller();
        virtual ~Controller ();

        void setHandle(HWND handle);

        virtual int close();
        virtual int create();
        virtual int destroy();
    protected:
        HWND handle;
    };
    inline void Controller::setHandle(HWND hWnd) { handle = hWnd; }
    inline int Controller::close() { ::DestroyWindow(handle); return 0; }
    inline int Controller::create() { return 0; }
    inline int Controller::destroy() { return 0; }
}

Класс ControllerGL 

#include "Controller.h"
#include "ViewGL.h"
#include "ModelGL.h"
namespace Win
{
    class ControllerGL : public Controller
    {
    public:
        ControllerGL(ModelGL* model, ViewGL* view);
        ~ControllerGL() {};
    private:
        ModelGL* modelGL;
        ViewGL* viewGL;
    };
}

процедура.ч 

#include <windows.h>

namespace Win
{
    LRESULT CALLBACK MDIChildWndProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam);
}

процедура.cpp 

#include "procedure.h"
#include "Controller.h"

LRESULT CALLBACK Win::MDIChildWndProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
{
    LRESULT returnValue = 0;

    Controller *ctrl;
    if (uMsg == WM_NCCREATE)
    {
        CREATESTRUCT* pCreate = (CREATESTRUCT*)(lParam);
        MDICREATESTRUCT* pMdiCreate = (MDICREATESTRUCT*)pCreate->lpCreateParams;
        ctrl = (Controller*)pMdiCreate->lParam;
        SetWindowLongPtr(hWnd, 0, reinterpret_cast<LONG_PTR>(ctrl));

        ctrl->setHandle(hWnd);

        SetWindowPos(hWnd, 0, 0, 0, 0, 0, SWP_NOMOVE | SWP_NOSIZE | SWP_NOZORDER | SWP_FRAMECHANGED);
    }
    else
        ctrl = reinterpret_cast<Controller *>(GetWindowLongPtr(hWnd, 0));

    if (!ctrl)
        return DefMDIChildProc(hWnd, uMsg, wParam, lParam);

    switch (uMsg)
    {
        case WM_CREATE:
            //returnValue = ctrl->create(); For when ControllerGL works properly
            break;
        case WM_CLOSE:
            //returnValue = ctrl->close();
            break;
        case WM_DESTROY:
            //returnValue = ctrl->destroy();
            break;
    }
    //return returnvalue;
    return DefMDIChildProc(hWnd, uMsg, wParam, lParam);
}

main.cpp 

#include <windows.h>
#include "ControllerGL.h"
#include "ModelGL.h"
#include "ViewGL.h"
#include "procedure.h"

#include "resource.h"

#define MAX_LOADSTRING 100

char g_szMDIChild_Title[MAX_LOADSTRING];
char g_szMDIChild_ClassName[MAX_LOADSTRING];

static HINSTANCE g_hInst;
static HWND g_hMDI;
static HWND g_hMDIClient;

static LRESULT CALLBACK MDIWndProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam) {
    switch (uMsg)
    {
        case WM_CREATE:
        {
            DragAcceptFiles(hWnd, TRUE);

            CLIENTCREATESTRUCT ccs;

            ccs.hWindowMenu = GetSubMenu(GetMenu(hWnd), 1);
            ccs.idFirstChild = IDC_MDICHILD_FIRST;

            g_hMDIClient = CreateWindowEx(0,
                "mdiclient",
                NULL,
                WS_VISIBLE | WS_CHILD | WS_CLIPCHILDREN | WS_VSCROLL | WS_HSCROLL,
                CW_USEDEFAULT,
                CW_USEDEFAULT,
                CW_USEDEFAULT,
                CW_USEDEFAULT,
                hWnd,
                0,
                g_hInst,
                (void*)(&ccs));

            if (!g_hMDIClient)
                MessageBox(hWnd, "Could not create MDI client!", "Error!", MB_OK | MB_ICONERROR);

            return 0;
        }
        break;
        case WM_CLOSE:
            DestroyWindow(hWnd);
            break;
        case WM_DESTROY:
            PostQuitMessage(0);
            break;
        case WM_COMMAND:
        {
            switch LOWORD(wParam)
            {
                case ID_FILE_NEW:
                {
                    ModelGL model;
                    Win::ViewGL view;

                    Win::ControllerGL glCtrl(&model, &view);

                    HWND hChild = CreateWindowEx(WS_EX_MDICHILD,
                        g_szMDIChild_ClassName,
                        g_szMDIChild_Title,
                        WS_CHILD | WS_VISIBLE | WS_OVERLAPPEDWINDOW,
                        CW_USEDEFAULT,
                        CW_USEDEFAULT,
                        CW_USEDEFAULT,
                        CW_USEDEFAULT,
                        g_hMDIClient,
                        NULL,
                        g_hInst,
                        (LPVOID)&glCtrl);
                }
                break;
                case ID_WINDOW_CASCADE:
                    PostMessage(g_hMDIClient, WM_MDICASCADE, 0, 0);
                    break;
                case ID_WINDOW_TILEHORIZONTAL:
                    PostMessage(g_hMDIClient, WM_MDITILE, MDITILE_HORIZONTAL, 0);
                    break;
                case ID_WINDOW_TILEVERTICAL:
                    PostMessage(g_hMDIClient, WM_MDITILE, MDITILE_VERTICAL, 0);
                    break;
                case ID_WINDOW_ARRANGEICONS:
                    PostMessage(g_hMDIClient, WM_MDIICONARRANGE, 0, 0);
                    break;
                default:
                {
                    if (LOWORD(wParam) >= IDC_MDICHILD_FIRST)
                    {
                        DefFrameProc(hWnd, g_hMDIClient, uMsg, wParam, lParam);
                    }
                    else
                    {
                        HWND hChild;
                        hChild = (HWND)SendMessage(g_hMDIClient, WM_MDIGETACTIVE, 0, 0);
                        if (hChild)
                        {
                            SendMessage(hChild, WM_COMMAND, wParam, lParam);
                        }
                    }
                }
            };
        }
        break;
        default:
            return DefFrameProc(hWnd, g_hMDIClient, uMsg, wParam, lParam);
    }
    return 0;
}

int WINAPI WinMain(HINSTANCE hInst, HINSTANCE hPrevInst, LPSTR lpCmdLine, int nCmdShow) {
    UNREFERENCED_PARAMETER(hPrevInst);
    UNREFERENCED_PARAMETER(lpCmdLine);

    char g_szMDI_Title[MAX_LOADSTRING];
    char g_szMDI_ClassName[MAX_LOADSTRING];

    LoadString(hInst, IDS_MDI_TITLE, g_szMDI_Title, MAX_LOADSTRING);
    LoadString(hInst, IDS_MDI_CLASSNAME, g_szMDI_ClassName, MAX_LOADSTRING);
    LoadString(hInst, IDS_MDICHILD_TITLE, g_szMDIChild_Title, MAX_LOADSTRING);
    LoadString(hInst, IDS_MDICHILD_CLASSNAME, g_szMDIChild_ClassName, MAX_LOADSTRING);

    HACCEL hAccelTable = LoadAccelerators(hInst, MAKEINTRESOURCE(IDI_MDI));

    WNDCLASSEX wcex;
    ZeroMemory(&wcex, sizeof(WNDCLASSEX));
    wcex.cbSize = sizeof(WNDCLASSEX);
    wcex.style = CS_HREDRAW | CS_VREDRAW;
    wcex.lpfnWndProc = Win::MDIChildWndProc;
    wcex.cbClsExtra = 0;
    wcex.cbWndExtra = sizeof(Win::Controller*);
    wcex.hInstance = hInst;
    wcex.hIcon = LoadIcon(hInst, MAKEINTRESOURCE(IDI_MDICHILD));
    wcex.hCursor = LoadCursor(nullptr, IDC_ARROW);
    wcex.hbrBackground = (HBRUSH)(COLOR_WINDOW + 1);
    wcex.lpszMenuName = nullptr;
    wcex.lpszClassName = g_szMDIChild_ClassName;
    wcex.hIconSm = LoadIcon(hInst, MAKEINTRESOURCE(IDI_MDICHILD_SMALL));

    if (!RegisterClassEx(&wcex))
    {
        MessageBox(0, "Failed to create MDI child window class", "Error!", MB_ICONEXCLAMATION | MB_OK);
        return 0;
    }

    ZeroMemory(&wcex, sizeof(WNDCLASSEX));
    wcex.cbSize = sizeof(WNDCLASSEX);
    wcex.style = CS_OWNDC;
    wcex.lpfnWndProc = MDIWndProc;
    wcex.cbClsExtra = 0;
    wcex.cbWndExtra = 0;
    wcex.hInstance = hInst;
    wcex.hIcon = LoadIcon(hInst, MAKEINTRESOURCE(IDI_MDI));
    wcex.hCursor = LoadCursor(0, IDC_ARROW);
    wcex.hbrBackground = (HBRUSH)GetStockObject(NULL_BRUSH);
    wcex.lpszMenuName = MAKEINTRESOURCE(IDM_MDI);
    wcex.lpszClassName = g_szMDI_ClassName;
    wcex.hIconSm = LoadIcon(hInst, MAKEINTRESOURCE(IDI_MDI_SMALL));

    if (!RegisterClassEx(&wcex))
    {
        MessageBox(0, "Failed to create MDI main window class", "Error!", MB_ICONEXCLAMATION | MB_OK);
        return 0;
    }

    g_hMDI = CreateWindowEx(0,
        g_szMDI_ClassName,
        g_szMDI_Title,
        WS_VISIBLE | WS_OVERLAPPEDWINDOW,
        CW_USEDEFAULT,
        CW_USEDEFAULT,
        CW_USEDEFAULT,
        CW_USEDEFAULT,
        0,
        0,
        hInst,
        NULL);

    if (g_hMDI == NULL)
    {
        MessageBox(NULL, "Failed to create MDI main window!", "Error!", MB_ICONEXCLAMATION | MB_OK);
        return 0;
    }

    g_hInst = hInst;

    MSG uMsg;

    while (GetMessage(&uMsg, 0, 0, 0)) {
        if (!TranslateMDISysAccel(g_hMDIClient, &uMsg) &&
            !TranslateAccelerator(uMsg.hwnd, hAccelTable, &uMsg))
        {
            TranslateMessage(&uMsg);
            DispatchMessage(&uMsg);
        }
    };

    return (int)uMsg.wParam;
}

При открытии нового документа с помощью ФАЙЛ> НОВЫЙ:

  1. Ни одно из дочерних окон не видно.

  2. Победит только первый ребенокDow получает запись в меню Windows главной формы, и создание других окон удаляет эту запись.

  3. Я не получаю ошибок, и присоединение отладчика ничего не показывает.

Кто-нибудь сталкивался с этой проблемой?

Или это не проблема, а яйцо Колумба, и я упускаю что-то очевидное?

Извините застопки кода. Я действительно пытался урезать это, честно.

Ответы [ 2 ]

1 голос
/ 23 октября 2019

Я вижу по крайней мере 2 основные ошибки в вашем коде, которые нужно исправить:

  1. WM_MDICREATE - это сообщение, которое приложение отправляет в окно клиента MDI для созданияMDI дочернее окно. Дочерний WndProc будет получать WM_(NC)CREATE сообщений, содержащих указатель MDICREATESTRUCT*, который был передан в lParam из WM_MDICREATE (если вы используете CreateMDIWindow() вместо WM_MDICREATE, он создаст для вас необходимый MDICREATESTRUCT). Однако ваш код не использует WM_MDICREATE (или CreateMDIWindow()) для создания своих дочерних окон MDI, вы используете CreateWindowEx(WS_EX_MDICHILD, ...) напрямую, и вы не передаете MDICREATESTRUCT* в параметре lpParam, вы передаетеуказатель ControllerGL* вместо. Таким образом, в обработчике WM_(NC)CREATE вашего дочернего WndProc поле CREATESTRUCT::lpCreateParam будет указателем ControllerGL*, а не MDICREATESTRUCT*.

  2. ваш обработчик ID_FILE_NEWпередача указателя на CreateWindowEx() локальному ControllerGL объекту, который выходит из области видимости и уничтожается после выхода CreateWindowEx(), оставляя вновь созданное дочернее окно MDI с висящим указателем Controller* на недопустимую память для всех последующих сообщений. Вам необходимо динамически выделить объект ControllerGL через new, а затем delete, когда дочернее окно MDI уничтожено, например, в его обработчике WM_(NC)DESTROY.

0 голосов
/ 23 октября 2019

Ну, это унизительно. У меня был класс ControllerGL, объявленный локально и уничтоженный при выходе из CreateWindowEx (), оставив дочернее окно MDI с указателем на несуществующий экземпляр класса ControllerGL. Я должен был сделать это;создайте указатель на классы ModelGL, ViewGL и ControllerGL в начале основной функции: 

#include <windows.h>
#include "ControllerGL.h"
#include "ModelGL.h"
#include "ViewGL.h"
#include "procedure.h"

#include "resource.h"

#define MAX_LOADSTRING 100

char g_szMDIChild_Title[MAX_LOADSTRING];
char g_szMDIChild_ClassName[MAX_LOADSTRING];

static HINSTANCE g_hInst;
static HWND g_hMDI;
static HWND g_hMDIClient;

ModelGL *model;
Win::ViewGL *view;
Win::ControllerGL *glCtrl;
//The rest of the main function remains unchanged

и объявите новый экземпляр каждого в обработчике ID_FILE_NEW, не забывая выбить амперсанд из переменнойglCtrl, уже объявленный в качестве указателя: 

model = new ModelGL();
view = new Win::ViewGL();
glCtrl = new Win::ControllerGL(model, view);

HWND hChild = CreateWindowEx(WS_EX_MDICHILD,
                        g_szMDIChild_ClassName,
                        g_szMDIChild_Title,
                        WS_CHILD | WS_VISIBLE | WS_OVERLAPPEDWINDOW,
                        CW_USEDEFAULT,
                        CW_USEDEFAULT,
                        rcClient.right,
                        rcClient.bottom,
                        g_hMDIClient,
                        NULL,
                        g_hInst,
                        (LPVOID)glCtrl);
If (!hWnd)
{
    delete model; model = NULL;
    delete view; view = NULL;
    delete glCtrl; glCtrl= NULL;
}

Большое спасибо Реми Лебо за его комментарии и ответ, и если у кого-то есть другой / лучший способ сделать это, пожалуйста, дайте мне знать.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.

Нет похожих вопросов

...