Thread: Something wrong with PUSHBUTTON

I just programmed the Child Window BS_PUSHBUTTON. Well, my problem is that when I move around the main window, it "disappears". And then when I wanted to test out the WM_COMMAND message I processed it like this:

Code:

case WM_COMMAND:
    MessageBox(NULL, TEXT("Test"), TEXT("Test"), MB_OK);
    return 0;

And nothing happened. Can you answer my questions and solve my problems? Thanks!

--Garfield

well some code might be nice...

Be sure you ddidn't do soemthing stupid I once did, and put the createwindow() in WM_PAINT....I did that once...baffled me for a while until I realized I had typed "PAINT" instead of "CREATE"...oh well

But CODE, garfield. Code. How can we solve your problems with a vague description and nothing more?

Umm...this is the code. I had deleted some parts of it and then I just quickly put some of the code back in (I think I might have some parameters of CreateWindow wrong -- but that's not the initial problem).

Code:

/****************************************************
****	FirstButton -- an example to create child****
****	window controls starting with the button ****
****************************************************/

#include <windows.h>

/***************************************************
****	Function Prototypes:					****
****		- WndProc : main window procedure	****
***************************************************/

LRESULT CALLBACK WndProc(HWND,UINT,WPARAM,LPARAM);

/***************************************************
****	Globals/Constants:						****
****	hInst : Global type HINSTANCE			****
****	NUM : Constant number of buttons		****
***************************************************/

#define NUM 1
HINSTANCE hInst;

/***************************************************
****	Main Code Section/Functions/Definitions:****
****		- WinMain							****
****		- WndProc : main window procedure	****
***************************************************/

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, PSTR szCmdLine, int nCmdShow)
{
	HWND			hwnd;
	WNDCLASS		wc;
	MSG				msg;
	TCHAR			szClassName[] = TEXT("MainWindow");

	wc.cbClsExtra = 0;
	wc.cbWndExtra = 0;
	wc.hbrBackground = (HBRUSH) GetStockObject(WHITE_BRUSH);
	wc.hCursor = LoadCursor(NULL, IDC_ARROW);
	wc.hIcon = LoadIcon(NULL, IDI_APPLICATION);
	wc.hInstance = hInstance;
	wc.lpfnWndProc = WndProc;
	wc.lpszClassName = szClassName;
	wc.lpszMenuName = NULL;
	wc.style = CS_HREDRAW | CS_VREDRAW;

	if(!RegisterClass(&wc))
	{
		MessageBox(NULL, TEXT("Error registering window!"), szClassName, MB_ICONEXCLAMATION | MB_OK);
		return 0;
	}

	hInst = hInstance;

	hwnd = CreateWindow(szClassName, TEXT("Main Window"),
						WS_OVERLAPPEDWINDOW,
						CW_USEDEFAULT, CW_USEDEFAULT,
						CW_USEDEFAULT, CW_USEDEFAULT,
						NULL, NULL, hInstance, NULL);

	ShowWindow(hwnd, nCmdShow);
	UpdateWindow(hwnd);

	while(GetMessage(&msg, NULL, 0, 0))
	{
		TranslateMessage(&msg);
		DispatchMessage(&msg);
	}

	return msg.wParam;
}

/***************************************************
****	WndProc : main window procedure			****
***************************************************/

LRESULT CALLBACK WndProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
{
	static HWND hwndButton;
	HDC			hdc;
	static int	cyChar, cxChar;

	switch(message)
	{
	case WM_CREATE:
		cxChar = LOWORD(GetDialogBaseUnits());
		cyChar = HIWORD(GetDialogBaseUnits());

	hwndButton = CreateWindow("button", TEXT("firstbutton"),
		WS_CHILD | WS_VISIBLE | BS_PUSHBUTTON,
		cxChar, cyChar, 20 * cxChar, 7 * cyChar / 4,
		hwnd, (HMENU) 1, hInst, NULL);

	return 0;

	case WM_SIZE:
		return 0;

	case WM_PAINT:
		return 0;

	case WM_COMMAND:
	MessageBox(NULL, TEXT("Test"), TEXT("Test"), MB_OK);
	return 0;

	case WM_DESTROY:
	PostQuitMessage(0);
	return 0;

	default:
		break;
	}
	return DefWindowProc(hwnd, message, wParam, lParam);
}

Garfield

here's your mistake:

case WM_PAINT:
break;

not

case WM_PAINT:
return 0;

good luck!
U.

::slaps face:: I should have known that! Of course, I'm processing the WM_PAINT method with just returning without processing. I should pass it on to DefWindowProc. Thanks, dude!

Also watch that you have given the button an ID# of one.

I would use more descriptive ID's, especially when you need more controls.

ie
#define IDB_BUTTON1 10001
and use this as the HMENU cast.

Okay. Thanks, guys!

no problem.

By the way, are you the dude that's helping Oskilian out on Aurora?

Yeah, Uraldor, I am. Go to osky's board so we can talk. He might have started a thread, but I'm not sure. Well anyway, we have to sort this out and assign jobs to each other.

If you get a minute, can you PM me your e-mail address, Uraldor?

--Garfield