ボタンを押すともう一つウィンドウが開くプログラムを作りたい

Question

そのプログラムを書いたのですが、なぜかボタンを押すとウィンドウが開く時と開かずに終了してしまう時があります。どこを修正すればいいでしょうか。win apiなのでコードが長くなりますが、(ファイルをアップロードする方法も分からないので)一応全部ここに載せます。あと面倒でなければこのコードの中で「こういう風には書いてしまうのは絶対避けるべき」などの点があったらそれも教えてください。

---

header.hpp

#include<windows.h>

//window placement
#define COORD_X 700
#define COORD_Y 0

//window size
#define W_LENTH 800
#define W_WIDTH 700

//Window Style
#define MAIN_WND_STYLE WS_OVERLAPPEDWINDOW
#define BSTYLE BS_PUSHBUTTON | WS_VISIBLE | WS_CHILD

class WINDOWOBJECT{
	HWND Button;
	HWND Wnd_s1;
public:
	void WM_CREATEfunc(HWND, LPCREATESTRUCT);
	void Button_func(HWND, LPCREATESTRUCT);
};

LRESULT CALLBACK WindowProc(HWND, UINT, WPARAM, LPARAM);
LRESULT CALLBACK WindowProc_s1(HWND, UINT, WPARAM, LPARAM);
void WM_COMMANDfunc(HWND, WPARAM, LPARAM, WINDOWOBJECT&);

---

---

WinMain.cpp

#include "header.hpp"

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInst, LPSTR lpCmdLine, int nCmdShow){
	WNDCLASSEX wc;
	wc.cbSize = sizeof(WNDCLASSEX);
	wc.style = CS_HREDRAW | CS_VREDRAW;
	wc.lpfnWndProc = WindowProc;
	wc.cbClsExtra = 0;
	wc.cbWndExtra = 0;
	wc.hInstance = hInstance;
	wc.hIcon = LoadIcon(NULL, IDI_APPLICATION);
	wc.hCursor = LoadCursor(NULL, IDC_ARROW);
	wc.hbrBackground = (HBRUSH)GetStockObject(BLACK_BRUSH);
	wc.lpszMenuName = NULL;
	wc.lpszClassName = "MainWndClass1";
	wc.hIconSm = NULL;
	
	if(!RegisterClassEx(&wc))	return FALSE;

	HWND hWnd;
	hWnd = CreateWindowEx(WS_EX_CLIENTEDGE, wc.lpszClassName, "Title",
		MAIN_WND_STYLE,　COORD_X, COORD_Y, W_WIDTH, W_LENTH,
		NULL, NULL, hInstance, NULL);
	
	if(hWnd == NULL)	return FALSE;
	
	ShowWindow(hWnd, nCmdShow);
	UpdateWindow(hWnd);
	
	MSG msg;
	BOOL bRet;
	
	while(bRet = GetMessage(&msg, NULL, 0, 0)){ 
		if(bRet == -1){
			MessageBox(NULL, "Message loop error", "alert", MB_OK);
			return 0;
		}
		
		TranslateMessage(&msg);
		DispatchMessage(&msg); 
	}

	return msg.wParam;	
}

LRESULT CALLBACK WindowProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam){
	static WINDOWOBJECT WndOb;
	switch(uMsg){
		case WM_CREATE:	WndOb.WM_CREATEfunc(hWnd, (LPCREATESTRUCT)lParam);	break;
		case WM_COMMAND:	WM_COMMANDfunc(hWnd, wParam, lParam, WndOb);	break;
		case WM_DESTROY:	PostQuitMessage(0);	break;
		default:	return DefWindowProc(hWnd, uMsg, wParam, lParam);
	}

	return 0;
}

---

---

WM_COMMANDfunc.cpp

#include "header.hpp"

void WM_COMMANDfunc(HWND hWnd, WPARAM wParam, LPARAM lParam, WINDOWOBJECT &WndOb){
	switch(LOWORD(wParam)){
		case 10: WndOb.Button_func(hWnd, (LPCREATESTRUCT)lParam);
	}
}

---

---

WM_CREATEfunc.cpp

#include "header.hpp"

void WINDOWOBJECT::WM_CREATEfunc(HWND hWnd, LPCREATESTRUCT lpCreateStruct){
	Button = CreateWindow("button", NULL, BSTYLE, 250, 380, 60, 30, hWnd, (HMENU)10,
			lpCreateStruct->hInstance, NULL);
	UpdateWindow(Button);
}

---

---

Button_func.cpp

#include "header.hpp"

void WINDOWOBJECT::Button_func(HWND hWnd, LPCREATESTRUCT lpCreateStruct){
	WNDCLASSEX wc;
	wc.cbSize = sizeof(WNDCLASSEX);
	wc.style = CS_HREDRAW | CS_VREDRAW;
	wc.lpfnWndProc = WindowProc_s1;
	wc.cbClsExtra = 0;
	wc.cbWndExtra = 0;
	wc.hInstance = lpCreateStruct->hInstance;
	wc.hIcon = LoadIcon(NULL, IDI_APPLICATION);
	wc.hCursor = LoadCursor(NULL, IDC_ARROW);
	wc.hbrBackground = (HBRUSH)GetStockObject(WHITE_BRUSH);
	wc.lpszMenuName = NULL;
	wc.lpszClassName = "SubWndClass1";
	wc.hIconSm = NULL;

	RegisterClassEx(&wc);
	Wnd_s1 = CreateWindowEx(WS_EX_CLIENTEDGE, wc.lpszClassName, NULL,
		WS_VISIBLE | WS_CAPTION | WS_SYSMENU,
		COORD_X + 10, COORD_Y, W_WIDTH - 50, W_LENTH - 50,
		hWnd, NULL, lpCreateStruct->hInstance, NULL);

	ShowWindow(Wnd_s1, SW_SHOW);
	UpdateWindow(Wnd_s1);
	
	MSG msg;
	BOOL bRet;
	
	while(bRet = GetMessage(&msg, NULL, 0, 0)){ 
		if(bRet == -1){
			MessageBox(NULL, "Message loop error", "alert", MB_OK);
			break;
		}
		
		TranslateMessage(&msg);
		DispatchMessage(&msg); 
	}

	DestroyWindow(Wnd_s1);
}

LRESULT CALLBACK WindowProc_s1(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam){
	switch(uMsg){
		case WM_DESTROY:	PostQuitMessage(0);	break;
		default:	return DefWindowProc(hWnd, uMsg, wParam, lParam);
	}

	return 0;
}

---

Answer 1

Windowシステムに対するいくつかの誤解があるようです。

1. lParamは常にLPCREATESTRUCTというわけではない
2. umsg == WM_COMMAND && LOWORD(wParam) == ボタンのID だからといって常にボタンが押下された通知とは限らない

以上2点は、Notification Messages from Buttonsやこちらを参照してください
3. ウィンドウを開くたびにWindowClassの登録が必要ではない
同一のウィンドウプロシージャであるならば使いまわしてもよい

Comments

[tometome]

私はインターネットのサイトを見て、winapiの基本的な部分を勉強して、そのサイトにlParamをLPCREATESTRUCTにキャストしてhInstanceを使っていた部分があったので、何も考えずそれを続けてきて今まで何も起こらなかったので間違いに気が付かなかったです。hInstanceは元のをそのまま使うことにしました。指摘された3つのことを一気に直したので動作がおかしかった原因がどれなのかは分かりませんでしたが、修正後はウインドウが開かないということはなくなりました。回答ありがとうございます。おかげで解決しました。