c++ - GDI lagger efter x sekunder - Udvikleren.dk - Programmering, webdesign og grafik

Tags: c++

<< < 1 2 3 > >>

Bruger #8985 @ 14.04.09 00:35

Hej.

Jeg har lavet et skrivekomponent magen til det i Visual Studio, hvor der er som bekendt en hel del grafiske elementer (linjeboksen, markeringsfunktionen, highlight af visse dele af teksten, etc.), men efter lidt tid eller et bestemt antal operationer (som at markere en linje frem og tilbage en masse gange hurtigt) giver GDI op.

Mit komponent ser således ud:
http://123login.dk/dexo/vse.png

Jeg kan desværre ikke vise, hvordan det ser ud, når GDI kokser, for da vil Windows ikke lade mig tage screenshots. Men jeg kan sige, at markeringen forsvinder og at der kommer en masse streger, der ligner komponentets caret (tekstmarkøren "IBeam"). Hvis jeg minimerer og gendanner vinduet, er komponentet helt sort (hvilket det er indstillet til at være ved programstart-up, inden WM_PAINT bearbejdes).

Jeg har førhen løst problemet med en timer, men det giver programmet en lidt "tung" fornemmelse. Jeg har også udtænkt en måde at "skåne" WM_PAINT på, så den ikke maler HELE komponentet hver gang, men den skal lige finpudses og omdannes til kode, hvilket jeg endnu ikke har gjort, i håb om end nemmere måde.

Thomas

Indlæg senest redigeret d. 14.04.2009 00:39 af Bruger #8985

22 svar postet i denne tråd vises herunder
1 indlæg har modtaget i alt 3 karma

Sorter efter stemmer Sorter efter dato

Bruger #4996 @ 14.04.09 23:58

196

Hvis du f.eks. ser på CreatePen:
http://msdn.microsoft.com/en-us/library/ms908180.aspx

Kan du se at du skal kalde DeleteObject når du er færdig med at bruge pennen. Det samme gælder for de andre GDI-objekter.

Bruger #8985 @ 15.04.09 00:02

160

Okay. Problemet er bare, at alle mine GDI-objekter er statiske... Det kan selvfølgelig ændres (hvilket jeg lige gjorde), men GDI giver stadig op. Nu vil jeg prøve at kalde DeleteObject så snart GDI-objektet er færdigt og ikke i bunden sammen med alle de andre.

Bruger #4996 @ 15.04.09 00:10

196

Du må stadig oprette ufattelig mange GDI-objekter. Man kan ved et uheld komme til at allokere et par stykker som man ikke for nedlagt rigtigt igen, men 1000 pr. sekund er langt fra normalt.

Uden at se koden er det svært præcist at fortælle hvad der går galt.

Bruger #8985 @ 15.04.09 00:24

160

Well jeg har intet imod at sende dig koden, Bertel... Jeg er bare ikke glad for at offentliggøre den.

Forresten, skal jeg også kalde DeleteObject på strukturer som RECT?

Her er koden til GDI-eksemplet:

Kode

#include <windows.h>
#include <vector>

std::vector<RECT> Squares;

void _Rectangle(HDC hdc, RECT sq)
{
	SelectObject(hdc, CreatePen(PS_SOLID, 2, RGB(142, 198, 254)));
	
	// Draw from left, top to bottom, top
	MoveToEx(hdc, sq.left, sq.top, NULL);
	LineTo  (hdc, sq.left, sq.bottom);
	
	// Draw from left, top to right, top
	MoveToEx(hdc, sq.left, sq.top, NULL);
	LineTo  (hdc, sq.right, sq.top);
	
	// Draw from right, top to right, bottom
	MoveToEx(hdc, sq.right, sq.top, NULL);
	LineTo  (hdc, sq.right, sq.bottom);
	
	// Draw from left, bottom to right, bottom
	MoveToEx(hdc, sq.left, sq.bottom, NULL);
	LineTo  (hdc, sq.right, sq.bottom);
	
	HDC     mem  = CreateCompatibleDC(hdc);
	HBITMAP sqbg = LoadBitmap(GetModuleHandle(NULL), TEXT("IDB_SQBG"));
	
	SelectObject(mem, sqbg);
	StretchBlt  (hdc, sq.left + 1, sq.top + 1, sq.right - sq.left - 2, sq.bottom - sq.top - 2, mem, 0, 0, 6, 601, SRCCOPY);
	
	DeleteObject(sqbg);
	DeleteDC(mem);
}

LRESULT __stdcall WinProc(HWND handle, unsigned int message, WPARAM wParam, LPARAM lParam)
{
	static PAINTSTRUCT ps;
	static HDC         mem;
	static HBITMAP     canvas;
	static RECT        rcSquare = { -1, -1, -1, -1 };
	static RECT        rcClient;
	
	switch (message)
	{
	case WM_SIZE:
	case WM_CREATE:
		GetClientRect(handle, &rcClient);
		return 0;
		
	case WM_PAINT:
		BeginPaint(handle, &ps);
		mem    = CreateCompatibleDC(ps.hdc);
		canvas = CreateCompatibleBitmap(ps.hdc, rcClient.right, rcClient.bottom);
		
		SelectObject(mem, canvas);
		FillRect(mem, &rcClient, (HBRUSH)GetStockObject(WHITE_BRUSH));
		
		// Draw squares
		for (int i = 0; i < Squares.size(); i++)
		{
			_Rectangle(mem, Squares[i]);
		}
		
		BitBlt(ps.hdc, 0, 0, rcClient.right, rcClient.bottom, mem, 0, 0, SRCCOPY);
		
		DeleteObject(canvas);
		DeleteDC(mem);
		EndPaint(handle, &ps);
		return 0;
		
	case WM_MOUSEMOVE:
		if (!(wParam & MK_LBUTTON))
			return 0;
		
		if (rcSquare.left < 0)
		{
			rcSquare.left = LOWORD(lParam);
			rcSquare.top  = HIWORD(lParam);	
			
			return 0;
		}
		else
		{
			rcSquare.right  = LOWORD(lParam);
			rcSquare.bottom = HIWORD(lParam);
		}
		
		if (!Squares.size())
			Squares.push_back(rcSquare);
		else
		{
			Squares.erase(Squares.begin() + (Squares.size() - 1));
			Squares.push_back(rcSquare);
		}
		
		InvalidateRect(handle, &rcClient, false);
		return 0;
		
	case WM_LBUTTONUP:
		if (rcSquare.left >= 0)
		{
			Squares.push_back(rcSquare);
			rcSquare.left = -1;
			
			InvalidateRect(handle, &rcClient, false);
		}
		return 0;
		
	case WM_DESTROY:
		PostQuitMessage(0);
		return 0;
	}
	
	return DefWindowProc(handle, message, wParam, lParam);
}

int __stdcall WinMain(HINSTANCE, HINSTANCE, PSTR, int)
{
	WNDCLASS wc;
	
	wc.cbClsExtra    = 0;
	wc.cbWndExtra    = 0;
	wc.hbrBackground = (HBRUSH)GetStockObject(BLACK_BRUSH);
	wc.hCursor       = LoadCursor(NULL, IDC_ARROW);
	wc.hIcon         = LoadIcon(NULL, IDI_APPLICATION);
	wc.hInstance     = GetModuleHandle(NULL);
	wc.lpfnWndProc   = WinProc;
	wc.lpszClassName = TEXT("GdiLagEks");
	wc.lpszMenuName  = NULL;
	wc.style         = 0;
	
	RegisterClass(&wc);
	
	HWND handle = CreateWindowEx(WS_EX_CLIENTEDGE, TEXT("GdiLagEks"), TEXT("Eksempel på \"GDI lag\""),
		WS_CAPTION | WS_SYSMENU | WS_MINIMIZEBOX,
		GetSystemMetrics(SM_CXSCREEN) / 2 - 400,
		GetSystemMetrics(SM_CYSCREEN) / 2 - 300,
		800, 600,
		NULL, NULL, NULL, NULL);
		
	ShowWindow(handle, SW_SHOW);
	UpdateWindow(handle);
	
	MSG msg;
	while (GetMessage(&msg, NULL, 0, 0))
	{
		TranslateMessage(&msg);
		DispatchMessage(&msg);
	}
	
	return msg.wParam;
}

Nu har jeg ændret lidt i koden, og wow... Den tilføjer næsten ikke nogle GDI-objekter (kun en gang i mellem og ikke 1 per firkant, som det er tilfældet med Paint).

Koden ser nu således ud:

Kode

#include <windows.h>
#include <vector>

std::vector<RECT> Squares;

void _Rectangle(HDC hdc, RECT sq)
{
	HBITMAP sqbg   = LoadBitmap(GetModuleHandle(NULL), TEXT("IDB_SQBG"));
	HPEN    border = CreatePen(PS_SOLID, 2, RGB(142, 198, 254));
	
	SelectObject(hdc, border);
	
	// Draw from left, top to bottom, top
	MoveToEx(hdc, sq.left, sq.top, NULL);
	LineTo  (hdc, sq.left, sq.bottom);
	
	// Draw from left, top to right, top
	MoveToEx(hdc, sq.left, sq.top, NULL);
	LineTo  (hdc, sq.right, sq.top);
	
	// Draw from right, top to right, bottom
	MoveToEx(hdc, sq.right, sq.top, NULL);
	LineTo  (hdc, sq.right, sq.bottom);
	
	// Draw from left, bottom to right, bottom
	MoveToEx(hdc, sq.left, sq.bottom, NULL);
	LineTo  (hdc, sq.right, sq.bottom);
	
	HDC     mem  = CreateCompatibleDC(hdc);
	
	SelectObject(mem, sqbg);
	StretchBlt  (hdc, sq.left + 1, sq.top + 1, sq.right - sq.left - 2, sq.bottom - sq.top - 2, mem, 0, 0, 6, 601, SRCCOPY);
	
	DeleteObject(border);
	DeleteObject(sqbg);
	DeleteDC(mem);
}

LRESULT __stdcall WinProc(HWND handle, unsigned int message, WPARAM wParam, LPARAM lParam)
{
	PAINTSTRUCT ps;
	HDC         mem;
	HBITMAP     canvas;
	HBRUSH      hbrBack = (HBRUSH)GetStockObject(WHITE_BRUSH);
	
	static RECT rcSquare = { -1, -1, -1, -1 };
	static RECT rcClient;
	
	switch (message)
	{
	case WM_SIZE:
	case WM_CREATE:
		GetClientRect(handle, &rcClient);
		return 0;
		
	case WM_PAINT:
		BeginPaint(handle, &ps);
		mem    = CreateCompatibleDC(ps.hdc);
		canvas = CreateCompatibleBitmap(ps.hdc, rcClient.right, rcClient.bottom);
		
		SelectObject(mem, canvas);
		FillRect(mem, &rcClient, hbrBack);
		
		// Draw squares
		for (int i = 0; i < Squares.size(); i++)
		{
			_Rectangle(mem, Squares[i]);
		}
		
		BitBlt(ps.hdc, 0, 0, rcClient.right, rcClient.bottom, mem, 0, 0, SRCCOPY);
		
		DeleteObject(canvas);
		DeleteObject(hbrBack);
		DeleteDC(mem);
		EndPaint(handle, &ps);
		
		DeleteObject(mem);
		return 0;
		
	case WM_MOUSEMOVE:
		if (!(wParam & MK_LBUTTON))
			return 0;
		
		if (rcSquare.left < 0)
		{
			rcSquare.left = LOWORD(lParam);
			rcSquare.top  = HIWORD(lParam);	
			
			return 0;
		}
		else
		{
			rcSquare.right  = LOWORD(lParam);
			rcSquare.bottom = HIWORD(lParam);
		}
		
		if (!Squares.size())
			Squares.push_back(rcSquare);
		else
		{
			Squares.erase(Squares.begin() + (Squares.size() - 1));
			Squares.push_back(rcSquare);
		}
		
		InvalidateRect(handle, &rcClient, false);
		return 0;
		
	case WM_LBUTTONUP:
		if (rcSquare.left >= 0)
		{
			Squares.push_back(rcSquare);
			rcSquare.left = -1;
			
			InvalidateRect(handle, &rcClient, false);
		}
		return 0;
		
	case WM_DESTROY:
		PostQuitMessage(0);
		return 0;
	}
	
	return DefWindowProc(handle, message, wParam, lParam);
}

int __stdcall WinMain(HINSTANCE, HINSTANCE, PSTR, int)
{
	WNDCLASS wc;
	
	wc.cbClsExtra    = 0;
	wc.cbWndExtra    = 0;
	wc.hbrBackground = (HBRUSH)GetStockObject(BLACK_BRUSH);
	wc.hCursor       = LoadCursor(NULL, IDC_ARROW);
	wc.hIcon         = LoadIcon(NULL, IDI_APPLICATION);
	wc.hInstance     = GetModuleHandle(NULL);
	wc.lpfnWndProc   = WinProc;
	wc.lpszClassName = TEXT("GdiLagEks");
	wc.lpszMenuName  = NULL;
	wc.style         = 0;
	
	RegisterClass(&wc);
	
	HWND handle = CreateWindowEx(WS_EX_CLIENTEDGE, TEXT("GdiLagEks"), TEXT("Eksempel på \"GDI lag\""),
		WS_CAPTION | WS_SYSMENU | WS_MINIMIZEBOX,
		GetSystemMetrics(SM_CXSCREEN) / 2 - 400,
		GetSystemMetrics(SM_CYSCREEN) / 2 - 300,
		800, 600,
		NULL, NULL, NULL, NULL);
		
	ShowWindow(handle, SW_SHOW);
	UpdateWindow(handle);
	
	MSG msg;
	while (GetMessage(&msg, NULL, 0, 0))
	{
		TranslateMessage(&msg);
		DispatchMessage(&msg);
	}
	
	return msg.wParam;
}

Indlæg senest redigeret d. 15.04.2009 01:45 af Bruger #8985

Bruger #8985 @ 15.04.09 02:01

160

Hvad med funktioner? Tilføjer de også GDI-objekter? Det virker som et lidt dumt spørgsmål, men jeg synes bare, jeg har DeleteObject'ed alle mine GDI-strukturer, hvilket efterlader funktioner som SetTextColor, SetTextMode, TextOut etc. som det eneste GDI-mæssige tilbage.

Bruger #4996 @ 15.04.09 19:53

196

Så vidt jeg kan se er det ok nu.

Bruger #8985 @ 16.04.09 00:03

160

Ja, testprogrammet er fint, men hvad med editoren? Jeg har sendt projektet til din e-mail.

Bruger #4996 @ 16.04.09 00:17

196

Jeg har ikke modtaget "editoren" endnu.

Bruger #8985 @ 16.04.09 03:15

160

Okay, har du skiftet e-mail? Jeg sendte den i går nat.

Men jeg tror, jeg har løst problemet. Jeg gjorde alle GDI-objekterne statiske. Nu kan jeg så self. ikke kalde DeleteObject på dem, men pyt med det.

Bruger #8985 @ 16.04.09 03:22

160

HA! Nu virker det helt perfekt! Jeg ville ønske, du ville lade mig give dig pointene, Bertel, for du har sgu fortjent dem... Men skidt med det!

<< < 1 2 3 > >>

GDI lagger efter x sekunder

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler