WinAPI. рисовалка

[aysium]

Должна меняться толщина и цвет линии по нажатию правой кнопки мыши, но не меняется

[aysium]

LRESULT WINAPI WindowProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam);

unsigned int lineWidth = 1;
COLORREF lineColor = RGB(0, 0, 0);
HDC hDC;
RECT rcClient;                 
POINT ptClientUL;             
POINT ptClientLR;              
static POINTS ptsBegin;       
static POINTS ptsEnd;         
static POINTS ptsPrevEnd;      
static BOOL fPrevLine = FALSE; 

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
{
	MessageBox(NULL, TEXT("Вход"), TEXT("Начать"), 0);


	WNDCLASSEX WindowClass;

	static LPCTSTR szAppName = L"WinApi";
	HWND hWnd;
	MSG msg;
	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(0, IDI_HAND);
	wc.hCursor = LoadCursor(0, IDC_ARROW);
	wc.hbrBackground = static_cast<HBRUSH>(GetStockObject(WHITENESS));
	wc.lpszMenuName = 0;
	wc.lpszClassName = szAppName;
	wc.hIconSm = 0;

	RegisterClassEx(&wc);
	hWnd = CreateWindow(szAppName, NULL, WS_OVERLAPPEDWINDOW, CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, 0, 0, hInstance, 0);
	ShowWindow(hWnd, nCmdShow);
	UpdateWindow(hWnd);
	hDC = GetDC(hWnd);
	while (GetMessage(&msg, 0, 0, 0))
	{
		TranslateMessage(&msg);
		DispatchMessage(&msg);
	}

	return static_cast<int>(msg.wParam);
}

LRESULT WINAPI WindowProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
{

	PAINTSTRUCT PaintSt;
	RECT aRect;
	switch (message)

	{

	case WM_PAINT:
	{
		hDC = BeginPaint(hWnd, &PaintSt);
		GetClientRect(hWnd, &aRect);
		SetBkMode(hDC, TRANSPARENT);
		DrawText(hDC, L"Рис", -1, &aRect, DT_SINGLELINE | DT_LEFT | DT_WORDBREAK);
		RECT widthRect = { 300, 0, 480, 50 }, widthValueRect = { 480, 0, 520, 50 },
		colorRect = { 520, 0, 630, 50 }, colorValueRect = { 630, 0, 900, 50 };

		RECT infoRect = { 300, 0, 900, 50 };
		FillRect(hDC, &infoRect, (HBRUSH)(COLOR_WINDOW + 1));
		DrawText(hDC, L"Текущая толщина линии:", -1, &widthRect, DT_SINGLELINE | DT_LEFT | DT_WORDBREAK);
		DrawText(hDC, std::to_wstring(lineWidth).c_str(), -1, &widthValueRect, DT_SINGLELINE | DT_LEFT | DT_WORDBREAK);
		DrawText(hDC, L"Текущий цвет:", -1, &colorRect, DT_SINGLELINE | DT_LEFT | DT_WORDBREAK);
		std::wstringstream wss;
		wss << L"R: " << GetRValue(lineColor) << L" | " << L"G: " << GetGValue(lineColor) << L" | " << L"B: " << GetBValue(lineColor);
		DrawText(hDC, wss.str().c_str(), -1, &colorValueRect, DT_SINGLELINE | DT_LEFT | DT_WORDBREAK);


		HBRUSH hBrush;
		CreateSolidBrush(RGB(255, 0, 67));
		hBrush = CreateHatchBrush(HS_FDIAGONAL, RGB(255, 10, 20));
		SelectObject(hDC, hBrush);
		Ellipse(hDC, 100, 100, 200, 300);


		HPEN hPen1, hPen2, hPen3;
		hPen1 = CreatePen(PS_DASHDOT, 1, RGB(0, 0, 255));
		hPen2 = CreatePen(PS_DASH, 1, RGB(255, 0, 255));
		hPen3 = CreatePen(PS_DOT, 1, RGB(0, 128, 256));

		SelectObject(hDC, hPen1);
		Rectangle(hDC, 10, 10, 100, 100);

		SelectObject(hDC, hPen2);
		Arc(hDC, 10, 20, 30, 40, 50, 60, 300, 400);

		SelectObject(hDC, hPen3);
		RoundRect(hDC, 200, 100, 140, 150, 160, 170);
		EndPaint(hWnd, &PaintSt);
	}
		break;
	case WM_LBUTTONDOWN:
	
		SetCapture(hWnd);
		
		GetClientRect(hWnd, &rcClient);
		ptClientUL.x = rcClient.left;
		ptClientUL.y = rcClient.top;

		ptClientLR.x = rcClient.right + 1;
		ptClientLR.y = rcClient.bottom + 1;
		ClientToScreen(hWnd, &ptClientUL);
		ClientToScreen(hWnd, &ptClientLR);

		SetRect(&rcClient, ptClientUL.x, ptClientUL.y, ptClientLR.x, ptClientLR.y);
		ClipCursor(&rcClient);

		ptsBegin = MAKEPOINTS(lParam);
		return 0;

	case WM_RBUTTONDOWN:
	{
			lineWidth = rand() % 19 + 1;
			lineColor = RGB(rand() % 255, rand() % 255, rand() % 255);
			InvalidateRect(hWnd, NULL, TRUE);
	}
		break;
	case WM_MOUSEMOVE:
		
		if (wParam & MK_LBUTTON)
		{

			hDC = GetDC(hWnd);

			SetROP2(hDC, R2_NOTXORPEN);

			if (fPrevLine)
				{
					MoveToEx(hDC, ptsBegin.x, ptsBegin.y, (LPPOINT)NULL);
					LineTo(hDC, ptsPrevEnd.x, ptsPrevEnd.y);
				}

			ptsEnd = MAKEPOINTS(lParam);
			MoveToEx(hDC, ptsBegin.x, ptsBegin.y, (LPPOINT)NULL);
			LineTo(hDC, ptsEnd.x, ptsEnd.y);

			fPrevLine = TRUE;
			ptsPrevEnd = ptsEnd;
			ReleaseDC(hWnd, hDC);
		}
		break;
	case WM_LBUTTONUP:
	{
					//
						 ptsEnd.x = LOWORD(lParam);
						 ptsEnd.y = HIWORD(lParam);
						 //
						 
						 HPEN linePen;
						 linePen = CreatePen(PS_SOLID, lineWidth, lineColor);
						 HGDIOBJ prevObj = SelectObject(hDC, linePen);
						 LineTo(hDC, ptsEnd.x, ptsEnd.y);
						 SelectObject(hDC, prevObj);
						 DeleteObject(linePen);
						 InvalidateRect(hWnd, NULL, TRUE);
						
						 
	}
		
		return 0;

	case WM_DESTROY:
		
		PostQuitMessage(0);
		return 0;

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



}

[aysium]

никто не поможет ?

[Stilet]

Слей коды из WM_LBUTTONUP и WM_RBUTTONDOWN в один обработчик в WM_RBUTTONDOWN к примеру.

[aysium]

Stilet, не помогло

[Stilet]

Ну я так не скажу. У меня не на чем тестить код увы...
А на глаз не определю ошибку.

[waleri]

Да тут на глаз столько ошибок, что дальше ехать некуда.

Например, подсчитайте, сколько раз получается DC и посмотрите сколько переменных для хранения есть?

А кто будет удалять созданные объекты?

Если рисовать в коде WM_MOUSEMOVE то что будет если окно изменится (и выполнится WM_PAINT)? Весь код рисования должен быть в одном месте, сообщения должны только делать InvalidateRect().

Надо иметь очень хорошую причину делать return 0 из обработчика сообщений. Если не знаете что делаете, надо передавать все сообщения в DefWindowProc()