Кнопки в WinApi

[Грендайзер]

Доброго времени суток, господа программисты Есть такая вот проблемка, а точнее недопонимание. Сам я инженер, и от программирования под Windows далёк, но вот тут пришлось сделать програмку для управления устройством по COM порту (точнее по USB но через адаптер). И как оказалось, можно конечно было графическим редактором воспользоваться, но когда я взялся выяснять, как всё это делается, я об этом не знал, так что навалился на WinApi... нууу и затянуло Так вот, программа состоит из родит. окна и из двух кнопок. Внимание вопрос номер адын: хочу, что б при нажатии на кнопку надпись на ней менялась. В принципе при нажатии на одну из кнопок (bt1) надпись на ней меняется, но по почему, она становится прежней если я нажимаю на другую кнопку? Есть ещё вопросы но я их тогда следом задам. Заранее спасибо. Исходничек прилагается. Почти всё что связано с работой с портом я выкинул, что бы не отвлекало.

[satmasterz]

Отличное решение!
---------------
http://fitoterapevt.pp.ua/likarski-r...ivosti-recepti

[Vladiger]

Сообщение от Грендайзер

хочу, что б при нажатии на кнопку надпись на ней менялась.

SetWindowText(hBtn1, "Закрыть порт");

Сообщение от Грендайзер

В принципе при нажатии на одну из кнопок (bt1) надпись на ней меняется, но по почему, она становится прежней если я нажимаю на другую кнопку?

У Вас ошибка в обработчике собития по нажатию кнопки bt1. Вот в этом месте:

case ID_BUTTON:
	btn1 = CreateWindow("BUTTON", "закрыть порт", BS_PUSHBUTTON | WS_VISIBLE | WS_CHILD | WS_TABSTOP, 70, 45, 240, 20, hMainWnd, (HMENU)ID_BUTTON, NULL, NULL);
	return 0;

Вы не меняете текст на кнопке, вы создаете новую кнопку, но при этом экземпляр старой кнопки btn1 остается где то в воздухе, потому что он не был удален. Получается не очень хорошая штука.
Перед созданием новой кнопки, старую нужно удалять, например так:

case ID_BUTTON:
	DestroyWindow(btn1);
	btn1 = CreateWindow("BUTTON", "закрыть порт", BS_PUSHBUTTON | WS_VISIBLE | WS_CHILD | WS_TABSTOP, 70, 45, 240, 20, hMainWnd, (HMENU)ID_BUTTON, NULL, NULL);
	return 0;

Но в Вашем же случае, я вообще не вижу смысла создавать новую кнопку, если Вам требуется всего навсего просто изменить её текст:

case ID_BUTTON:
	SetWindowText(btn1, "Закрыть порт");
	return 0;

Исправленный вариант:

#include <windows.h>
//#include "stdafx.h"

#define ID_BUTTON_1 3000
#define ID_BUTTON_2 3001

HWND hBtn1;
HWND hBtn2;
HANDLE hCOM; // хендл сом порта
LRESULT CALLBACK WndProc(HWND, UINT, WPARAM, LPARAM);

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow) {

	HWND hMainWnd; // хендл будущего окна
	char szClassName[] = "MyClass";
	MSG msg;
	WNDCLASSEX wc;

	// Заполняем структуру класса окна
	wc.cbSize = sizeof(wc);
	wc.style = CS_HREDRAW | CS_VREDRAW;
	wc.lpfnWndProc = WndProc; //WndProc;
	wc.cbClsExtra = 0;
	wc.cbWndExtra = 0;
	wc.hInstance = hInstance;
	wc.hIcon = LoadIcon(NULL, IDI_WINLOGO);
	wc.hCursor = LoadCursor(NULL, IDC_ARROW);
	wc.hbrBackground = (HBRUSH)GetStockObject(DKGRAY_BRUSH);
	wc.lpszMenuName = NULL;
	wc.lpszClassName = "MyClass";
	wc.hIconSm = LoadIcon(NULL, IDI_APPLICATION);

	// Регистрируем класс окна
	if (!RegisterClassEx(&wc)) {
		MessageBox(NULL, "Cannot register class", "Error", MB_OK);
		return 0;
	}

	// Создаем основное окно приложения
	hMainWnd = CreateWindow(szClassName, "WndProc", WS_OVERLAPPEDWINDOW, CW_USEDEFAULT, 0, CW_USEDEFAULT, 0, NULL, NULL, hInstance, NULL);
	if (!hMainWnd) {
		MessageBox(NULL, "Cannot create main window", "Error", MB_OK);
		return 0;
	}

	//Создаем кнопки hBtn1 и hBtn2;
	hBtn1 = CreateWindow("BUTTON", "Открыть порт", BS_PUSHBUTTON | WS_VISIBLE | WS_CHILD | WS_TABSTOP, 70, 45, 240, 20, hMainWnd, (HMENU)ID_BUTTON_1, hInstance, NULL);
	hBtn2 = CreateWindow("BUTTON", "Generate key2", BS_PUSHBUTTON | WS_VISIBLE | WS_CHILD | WS_TABSTOP, 400, 200, 250, 250, hMainWnd, (HMENU)ID_BUTTON_2, hInstance, NULL);


	// Показываем окно
	ShowWindow(hMainWnd, nCmdShow);
	UpdateWindow(hMainWnd);

	// Выполняем цикл обработки сообщений до закрытия приложения
	while(GetMessage(&msg, NULL, 0, 0)) {
		TranslateMessage(&msg);
		DispatchMessage(&msg);
	}

	return msg.wParam;
}

LRESULT CALLBACK WndProc(HWND hMainWnd, UINT msg, WPARAM wParam, LPARAM lParam) {

	char buffer = '1';
	//int fl;
	//DWORD signal;
	//DWORD dwByteWrite;

	//HWND hBtn1;
	HDC hDC;
	PAINTSTRUCT ps;
	RECT rect;

	switch (msg) {
		case WM_PAINT:
			hDC = BeginPaint(hMainWnd, &ps);

			GetClientRect(hMainWnd, &rect);
			DrawText(hDC, "Hello, World!", -1, &rect, DT_SINGLELINE | DT_CENTER | DT_VCENTER);

			EndPaint(hMainWnd, &ps);
			return 0;

		case WM_CLOSE:
			DestroyWindow(hMainWnd);
			return 0;

		case WM_DESTROY:
			PostQuitMessage(0);
			return 0;


		case WM_COMMAND:
			switch(wParam) {
				case ID_BUTTON_1:
					SetWindowText(hBtn1, "Закрыть порт");
					return 0;
				
				case ID_BUTTON_2:
					//WriteFile(
					// hCOM, Для работы с портом!!!
					// &buffer,
					// sizeof(buffer),
					// &dwByteWrite,
					// &overlapTX
					// );

					//signal = WaitForSingleObject(overlapTX.hEvent, INFINITE);
					//fl = ((signal == WAIT_OBJECT_0) && (GetOverlappedResult(hCOM, &overlapTX, &dwByteWrite, true))) ? true : false;
					//CloseHandle(hCOM);

					return 0;
			}

		case WM_KEYDOWN:
			switch(wParam) {
				case VK_ESCAPE:
					SendMessage(hMainWnd, WM_CLOSE, 0, 0);
					break;
			}
			break;
		
		default:
			return DefWindowProc(hMainWnd, msg, wParam, lParam);
	}
	return 0;
}

[Грендайзер]

Ух ты! Большое спасибо! У меня ещё была пара вопросиков, но сначала это дело обмозгую!

[Vladiger]

Кстати, забыл написать: - Раз уж упомянул про DestroyWindow(HWND hWnd), то должен заметить, что это касается всех окон/контролов созданных функцией CreateWindow(), в том числе и основного окна приложения...
Не знаю что страшного может произойти если перед выходом из программы основное окно не удалить, возможны утечки памяти, но точно не знаю, не задавался пока таким вопросом. И тем не менее, удалять его нужно.
Перед выходом из программы, нужно добавить строчку DestroyWindow(hMainWnd);

То есть примерно так:

while(GetMessage(&msg, NULL, 0, 0)) {
	TranslateMessage(&msg);
	DispatchMessage(&msg);
}
DestroyWindow(hMainWnd); // Уничтожаем основное окно
UnregisterClass(szClassName, NULL); // Освобождаем память, отменяя регистрацию класса основного окна.
return msg.wParam;

[Грендайзер]

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

[waleri]

При завершении процесса освобождаются все его ресурсы, включая и окна.
Удалять WS_CHILD окна необязательно - они автоматически закрываются при закрытии родителя.

[Vladiger]

А вообще то пардон, не заметил вот это:

case WM_CLOSE:
	DestroyWindow(hMainWnd);
	return 0;

То есть основное окно в Вашем варианте разрушается по сообщению WM_CLOSE - этого достаточно. Как выше было добавлено waleri, кнопки созданы дочерними окнами WS_CHILD, а значит удалятся автоматически вместе с основным окном.
Единственное что сюда можно было добавить, это отмену регистрации класса, например так:

case WM_CLOSE:
	DestroyWindow(hMainWnd);
	UnregisterClass(szClassName, NULL);
	return 0;

Но переменная szClassName является локальной в функции WinMain и в обработчике WndProc она недоступна. Вариантов 2, либо сделать её глобальной, либо отмену регистрации делать не в обработчике, а непосредственно перед выходом из программы (как и было замечено выше)...
Это конечно не суть важно где делать эти отмены и уничтожения окон, главное что бы вы суть уловили. Зарегистрировали класс, создали окно - значит перед выходом нужно все это дело освободить!

[Грендайзер]

Опаньки... т.е. ещё и класс надо разрегистрировать? А вот об этом я даже и не знал...

[waleri]

WM_CLOSE стандартно сам закроет окно в DefWindowProc.
Отрегистрировать окно в обработчике WM_CLOSE не получится, потому что окно в этот момент все еще существует, т.е. класс окна используется.

При выходе из программы *все* ресурсы освобождаются.