Nicht in WM_PAINT zeichnen

[mantiz]

Versuch mal in WM_CREATE:

HDC hdc = GetDC(hwnd);
// und dann:
hdc2 = CreateCompatibleDC(hdc);
ReleaseDC(hwnd, hdc);

[Sir_Steve]

Hm, geht immer noch nicht
Immer bleibt nur das letzte wenn ich zb Fenster kleiner/größer mache ...

[mantiz]

OK, hier:

LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam) {
	static HDC hdc2;
	static HBITMAP hBM;

	switch (message) {
		case WM_CREATE: {
			HDC hdc = ::GetWindowDC(hWnd);
			hdc2 = ::CreateCompatibleDC(hdc);
			RECT r;
			::GetClientRect(hWnd, &r);
			hBM = CreateCompatibleBitmap(hdc, r.right - r.left, r.bottom - r.top);
			::SelectObject(hdc2, hBM);
			HBRUSH brush = (HBRUSH)GetStockObject(WHITE_BRUSH);
			::FillRect(hdc2, &r, brush);
			::DeleteObject(brush);
			::ReleaseDC(hWnd, hdc);
			break;
		}
		case WM_LBUTTONDOWN: {
			::SelectObject(hdc2, ::GetStockObject(BLACK_PEN));
			::LineTo(hdc2, LOWORD(lParam), HIWORD(lParam));
			::InvalidateRect(hWnd, NULL, true);
			break;
		}
		case WM_PAINT: {
			PAINTSTRUCT ps;
			::BeginPaint(hWnd, &ps);
			RECT r;
			::GetClientRect(hWnd, &r);
			::BitBlt(ps.hdc, 0, 0, r.right - r.left, r.bottom - r.top, hdc2, 0, 0, SRCCOPY);
			::EndPaint(hWnd, &ps);
			break;
		}
		case WM_DESTROY: {
			::DeleteDC(hdc2);
			PostQuitMessage(0);
			return 0;
		}
	}
}

Ist ein wenig durcheinander, aber ich musste auch erstmal schauen, wie das nochmal war.
Wenn Du das Bitmap erstellst, dann solltest Du es aber nicht nur so groß erstellen, wie der Clientbereich ist, ansonsten wird das Fenster nicht ganz ausgefüllt, wenn Du es maximierst, am Besten nimmst Du hier ein Bitmap, welches so groß ist, wie die eingestellte Auflösung oder so.

Hoffe, das hilft Dir ein wenig.

[SelectObject]

mantiz, du hast vergessen das Original-Objekt wieder reinzuselektieren.

[geeky]

Das Rect das GetClientRect() liefert, hat left und top übrigens immer auf 0 gesetzt

[mantiz]

@SelectObject: Es ging hier nur um die Funktionsweise, ich habe das nicht aus einem bestehenden Programm kopiert, sondern mal eben schnell zusammengeschrieben, so dass das gewollte funktioniert. Auf Feinheiten hab' ich keine Rücksicht genommen, dafür gibt es die MSDN und den Petzold.

@geeky:

MSDN schrieb:

Because client coordinates are relative to the upper-left corner of a window's client area, the coordinates of the upper-left corner are (0,0).

Hast Recht, aber imo würde es mehr Sinn machen, wenn die ClientArea relativ zur Fensterposition angegeben würde, egal.

Hab' jetzt sowieso keinen Bock mehr Sourcecode zu posten, jedes Mal, wenn ich Code poste wird dran rumgemeckert. Meistens handelt es sich um Codes, die aus dem Kontext gerissen sind, oder mal eben schnell zusammengeschrieben wurden, um die Vorgehensweise zu zeigen, nicht, um jmd. die ganze Arbeit und das Denken zu ersparen. Ich hab' zwar noch lange nicht so viel gepostet, wie manch anderer hier, aber so macht es einfach keinen Spass.

[SelectObject]

Ja dann schreib am besten einen kleinen Hinweis dabei, das der Code noch nicht richtig fertig ist. Dann wird auch bestimmt nicht soviel gemerkt.
Das Problem ist das die meisten das einfach abkopieren und dann haben die auch den Fehler drin.

[mantiz]

Naja, wenn hier Code gepostet wird, dann ist dieser ja eigentlich dazu gedacht, dass man diesen ohne zu überlegen kopiert und verwendet, sondern eher, gewisse Dinge besser zu verstehen, die man nicht so gut mit Worten beschreiben kann bzw. der besseren Veranschaulichung. Wenn jmd. den Code einfach nur kopiert, ohne zu verstehen, was da abgeht bzw. ohne sich selbst Gedanken dazu zu machen und sich näher damit zu beschäftigen, dann ist dieser jmd. auch selbst Schuld, wenn nicht alles einwandfrei funktioniert, oder evtl. Speicher-Leaks entstehen, oder ... oder.

Mir hat es auch schon des öfteren geholfen, wenn jmd. etwas Code gepostet hat, damit ich etwas besser verstehe, aber ich würde mich hüten den Code einfach so zu übernehmen. Sorry, aber dann hat man es auch nicht besser verdient. Nur meine Meinung.

[Sir_Steve]

Hm, also ich muss mantiz mal ein bisschen verteidigen. Am Quellcode von ihm rumzumeckern bringt echt nichts, ich bat nämlich echt nur um den Code damit ich endlich weiß wie es gehen könnte, ihn dumm zu kopieren, das mache ich nicht, so schlau bin SOGAR ICH!
Hoho!
Also bitte hör nicht auf Quellcode zu posten, ich tu mir damit viel leichter als wenn mir wer sagt wie es geht ... weil Code verstehe ich recht leicht, ein bisschen noch Googeln und MSDN und fertig!
Na jedenfalls danke schön! Ich schaue mir das gleich mal an ...

Grüße, Steve!

[Sir_Steve]

Also, einmal mehr ein gute und eine schlechte Nachricht:
Ich kann jetzt zwar endlich unendlich viele Sachen malen, die bleiben auch erhalten!
ABER: Jetzt funktionieren die Farben nicht mehr
Es ist alles schwarz/weiß.
Ist da noch irgendwo ein kleiner Parameter falsch oder so?

[flenders]

Normalerweise ist das Der Fall, wenn du bei CreateCompatibleBitmap den neu erstellten DC angibst (da dieser zu Beginn Schwarz-Weiß ist) - richtig ist es so wie im oben geposteten Beispielcode, wenn du hdc angibst
Hast du da evtl. was abgeändert?!

[mantiz]

Ja, genau, im Petzold steht z.B. drin, dass neu erstellte DCs zu Beginn 1x1 Pixel gross und monochrom sind. Deswegen wird direkt nach dem erstellen des DCs ein Bitmap erstellt, welches die gewünschte Größe und Farbtiefe hat und dann in den DC geladen, wodurch sich der DC anpasst, soweit, wie ich das verstanden hab'. Wenn Du jetzt kein vorhandenen Bitmap lädst, sondern ein kompatibles erstellst und zwar zu dem monochromen 1x1-Pixel großen, monochromen DC, dann ist das Bitmap natürlich auch monochrom. Da Du aber den DC des Fenster haben möchtest erstellst Du hier ein Bitmap, welches kompatibel zu diesem ist, wodurch sich der andere DC anpasst.

[Sir_Steve]

Ups, habe mich vertippt.
Habe statt hdc hdc2 geschrieben (und eine Zeile sowieso verschissen)

Aber es geht jetzt!
Vielen dank, ich spiele mich noch damit herum damit ich alles inetwa verstehe!

Grüße, Steve!