<![CDATA[Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen]]>Hallo liebe Gemeinde.

Ich stehe gerade vor einem Problem.
Ich möchte mir eine Funktion schreiben,welches mir etwa jede Sekunde ein Screenshot eines Desktopbereichs macht.

Das Funktioniert auch. Nur das Programm beendet mit einem Fehler nach x Durchläufen (je nach Rechner(Speicher))

Hier der Code

#include <iostream>
#include <fstream>
#include <cstdint>
#include <cassert>

#include <Windows.h>

void capture();
char c;

void main (){

	int i =0;
	while(1){
	i++;

	std::cout<<i<<std::endl;
	Sleep(1000);
	capture();
	remove("file.bmp");
	}

}
void capture()
{
    HDC screen = GetDC(NULL);
    HDC target = CreateCompatibleDC(screen);

    /*uint32_t width = GetSystemMetrics(SM_CXSCREEN);
    uint32_t height = GetSystemMetrics(SM_CYSCREEN);*/

	uint32_t width = 640;
	uint32_t height= 480;

    HBITMAP bmp = CreateCompatibleBitmap(screen, width, height);

    SelectObject(target, bmp);

    int x = 0;
    x = BitBlt(target, 0, 0, width, height, screen, 10,10, SRCCOPY | CAPTUREBLT);
    assert(x);

	BITMAPINFO bminfo;
    bminfo.bmiHeader.biBitCount = 16;
    bminfo.bmiHeader.biSize = sizeof(BITMAPINFOHEADER);
	bminfo.bmiHeader.biCompression = BI_RGB;
    bminfo.bmiHeader.biPlanes = 1;
    bminfo.bmiHeader.biWidth = width;
    bminfo.bmiHeader.biHeight = height;
    bminfo.bmiHeader.biSizeImage = width * 4 * height; // must be DWORD aligned
    bminfo.bmiHeader.biXPelsPerMeter = 0;
    bminfo.bmiHeader.biYPelsPerMeter = 0;
    bminfo.bmiHeader.biClrUsed = 0;
    bminfo.bmiHeader.biClrImportant = 0;

    uint8_t *bmpBuffer = new uint8_t[bminfo.bmiHeader.biSizeImage];

	x = GetDIBits(target, bmp, 0, height, bmpBuffer, &bminfo, DIB_RGB_COLORS);
    assert(x);

    BITMAPFILEHEADER fheader;
    fheader.bfType = 0x4D42;
    fheader.bfSize = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER) + bminfo.bmiHeader.biSizeImage;
    fheader.bfReserved1 = 0;
    fheader.bfReserved2 = 0;
    fheader.bfOffBits = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER);

    std::ofstream bmp_file("file.bmp", std::ios_base::binary);

    bmp_file.write((const char *) &fheader, sizeof(fheader));
    bmp_file.write((const char *) &(bminfo.bmiHeader), sizeof(bminfo.bmiHeader));
    bmp_file.write((const char *) bmpBuffer, bminfo.bmiHeader.biSizeImage);
    bmp_file.close();

}

Ich vermute, dass ich irgendwie den Speicher wieder zurücksetzen muss.

Nur leider geht

free(&bminfo);

nicht

Evtl habt Ihr eine Idee.

Das Programm soll idealerweise 24h laufen.

VG

]]>https://www.c-plusplus.net/forum/topic/305682/screenshot-mit-gdi-in-endlosschleife-beendet-sich-nach-x-durchläufenRSS for NodeSat, 08 Aug 2026 19:59:23 GMTThu, 05 Jul 2012 11:19:30 GMT60<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 11:21:07 GMT]]>Hallo liebe Gemeinde.

Ich stehe gerade vor einem Problem.
Ich möchte mir eine Funktion schreiben,welches mir etwa jede Sekunde ein Screenshot eines Desktopbereichs macht.

Das Funktioniert auch. Nur das Programm beendet mit einem Fehler nach x Durchläufen (je nach Rechner(Speicher))

Hier der Code

#include <iostream>
#include <fstream>
#include <cstdint>
#include <cassert>

#include <Windows.h>

void capture();
char c;

void main (){

	int i =0;
	while(1){
	i++;

	std::cout<<i<<std::endl;
	Sleep(1000);
	capture();
	remove("file.bmp");
	}

}
void capture()
{
    HDC screen = GetDC(NULL);
    HDC target = CreateCompatibleDC(screen);

    /*uint32_t width = GetSystemMetrics(SM_CXSCREEN);
    uint32_t height = GetSystemMetrics(SM_CYSCREEN);*/

	uint32_t width = 640;
	uint32_t height= 480;

    HBITMAP bmp = CreateCompatibleBitmap(screen, width, height);

    SelectObject(target, bmp);

    int x = 0;
    x = BitBlt(target, 0, 0, width, height, screen, 10,10, SRCCOPY | CAPTUREBLT);
    assert(x);

	BITMAPINFO bminfo;
    bminfo.bmiHeader.biBitCount = 16;
    bminfo.bmiHeader.biSize = sizeof(BITMAPINFOHEADER);
	bminfo.bmiHeader.biCompression = BI_RGB;
    bminfo.bmiHeader.biPlanes = 1;
    bminfo.bmiHeader.biWidth = width;
    bminfo.bmiHeader.biHeight = height;
    bminfo.bmiHeader.biSizeImage = width * 4 * height; // must be DWORD aligned
    bminfo.bmiHeader.biXPelsPerMeter = 0;
    bminfo.bmiHeader.biYPelsPerMeter = 0;
    bminfo.bmiHeader.biClrUsed = 0;
    bminfo.bmiHeader.biClrImportant = 0;

    uint8_t *bmpBuffer = new uint8_t[bminfo.bmiHeader.biSizeImage];

	x = GetDIBits(target, bmp, 0, height, bmpBuffer, &bminfo, DIB_RGB_COLORS);
    assert(x);

    BITMAPFILEHEADER fheader;
    fheader.bfType = 0x4D42;
    fheader.bfSize = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER) + bminfo.bmiHeader.biSizeImage;
    fheader.bfReserved1 = 0;
    fheader.bfReserved2 = 0;
    fheader.bfOffBits = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER);

    std::ofstream bmp_file("file.bmp", std::ios_base::binary);

    bmp_file.write((const char *) &fheader, sizeof(fheader));
    bmp_file.write((const char *) &(bminfo.bmiHeader), sizeof(bminfo.bmiHeader));
    bmp_file.write((const char *) bmpBuffer, bminfo.bmiHeader.biSizeImage);
    bmp_file.close();

}

Ich vermute, dass ich irgendwie den Speicher wieder zurücksetzen muss.

Nur leider geht

free(&bminfo);

nicht

Evtl habt Ihr eine Idee.

Das Programm soll idealerweise 24h laufen.

VG

]]>https://www.c-plusplus.net/forum/post/2230276https://www.c-plusplus.net/forum/post/2230276Thu, 05 Jul 2012 11:21:07 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 11:32:25 GMT]]>Überleg doch mal an welcher Stelle Du Speicher allokierst. Den musst Du auch wieder freigeben. (Tipp: Sollte nicht so schwer sein, Du verwendest nur einmal eine Allokation)

]]>https://www.c-plusplus.net/forum/post/2230279https://www.c-plusplus.net/forum/post/2230279Thu, 05 Jul 2012 11:32:25 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 11:34:55 GMT]]>Mit vector wär das nicht passiert
[/heisetrollmodus]

]]>https://www.c-plusplus.net/forum/post/2230280https://www.c-plusplus.net/forum/post/2230280Thu, 05 Jul 2012 11:34:55 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 11:41:26 GMT]]>Wärs wohl doch. Auch die von Windows angeforderten Ressourcen (via GetDC, CreateCompatibleBitmap usw.) müssen wieder freigegeben werden.

]]>https://www.c-plusplus.net/forum/post/2230282https://www.c-plusplus.net/forum/post/2230282Thu, 05 Jul 2012 11:41:26 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 11:42:17 GMT]]>Ich glaube, dass die Anzahl der GDI-Objekte hier eher zum Absturz führt. Es fehlen ReleaseDC und DeleteDC.
Natürlich muss das Erwähnte auch behoben werden (std::vector!).

]]>https://www.c-plusplus.net/forum/post/2230283https://www.c-plusplus.net/forum/post/2230283Thu, 05 Jul 2012 11:42:17 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 11:45:08 GMT]]>Nachtrag: Wie Belli geschrieben hat, fehlt auch noch DeleteObject für die Bitmap.

Das Beste wäre wohl, wenn du dir kleine Wrapperklassen schreiben würdest, die das automatisch erledigen (RAII).

]]>https://www.c-plusplus.net/forum/post/2230284https://www.c-plusplus.net/forum/post/2230284Thu, 05 Jul 2012 11:45:08 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 12:28:19 GMT]]>Danke schonmal für die Antworten. So schnell hatte ich garnicht mit gerechnet 🙂

Also den Speicher löschen bringt nix.

Also ist es sicher das was mapper und Belli sagen.

Leider ist es das erste Programm was ich mit GDI versuche.
Demzufolge weiß ich jetzt nicht ganz wie die Syntax aussehen sollte ReleaseDC erwartet ganz andere Datenstrukturen als die meinen.

das ist mein aktueller code:

#include <iostream>
#include <fstream>
#include <cstdint>
#include <cassert>

#include <Windows.h>

void capture();
char c;

void main (){

	int i =0;
	while(1){
	i++;

	std::cout<<i<<std::endl;

	capture();
	remove("file.bmp");
	}

}
void capture()
{
	HWND hWnd = FindWindow(NULL, NULL);
    HDC screen = GetDC(NULL);
    HDC target = CreateCompatibleDC(screen);

    /*uint32_t width = GetSystemMetrics(SM_CXSCREEN);
    uint32_t height = GetSystemMetrics(SM_CYSCREEN);*/

	uint32_t width = 640;
	uint32_t height= 480;

    HBITMAP bmp = CreateCompatibleBitmap(screen, width, height);

    SelectObject(target, bmp);

    int x = 0;
    x = BitBlt(target, 0, 0, width, height, screen, 10,10, SRCCOPY | CAPTUREBLT);
    assert(x);

	BITMAPINFO bminfo;
    bminfo.bmiHeader.biBitCount = 16;
    bminfo.bmiHeader.biSize = sizeof(BITMAPINFOHEADER);
	bminfo.bmiHeader.biCompression = BI_RGB;
    bminfo.bmiHeader.biPlanes = 1;
    bminfo.bmiHeader.biWidth = width;
    bminfo.bmiHeader.biHeight = height;
    bminfo.bmiHeader.biSizeImage = width * 4 * height; // must be DWORD aligned
    bminfo.bmiHeader.biXPelsPerMeter = 0;
    bminfo.bmiHeader.biYPelsPerMeter = 0;
    bminfo.bmiHeader.biClrUsed = 0;
    bminfo.bmiHeader.biClrImportant = 0;

    uint8_t *bmpBuffer = new uint8_t[bminfo.bmiHeader.biSizeImage];

	x = GetDIBits(target, bmp, 0, height, bmpBuffer, &bminfo, DIB_RGB_COLORS);
    assert(x);

    BITMAPFILEHEADER fheader;
    fheader.bfType = 0x4D42;
    fheader.bfSize = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER) + bminfo.bmiHeader.biSizeImage;
    fheader.bfReserved1 = 0;
    fheader.bfReserved2 = 0;
    fheader.bfOffBits = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER);

    std::ofstream bmp_file("file.bmp", std::ios_base::binary);

    bmp_file.write((const char *) &fheader, sizeof(fheader));
    bmp_file.write((const char *) &(bminfo.bmiHeader), sizeof(bminfo.bmiHeader));
    bmp_file.write((const char *) bmpBuffer, bminfo.bmiHeader.biSizeImage);
    bmp_file.close();
	DeleteObject(bmp_file);

	DeleteDC(target);
	DeleteDC(screen);
	ReleaseDC(hWnd, target);
	ReleaseDC(hWnd, screen);
}

Evtl kann sich jemand die mühe mache und kurz ein Bsp posten.

mit dem hier

free(&fheader);

stürzt das Programm nach dem ersten Schleifendurchlauf ab

vielen Dank 😕

]]>https://www.c-plusplus.net/forum/post/2230287https://www.c-plusplus.net/forum/post/2230287Thu, 05 Jul 2012 12:28:19 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 12:36:54 GMT]]>uint8_t *bmpBuffer = new uint8_t
hast du wohl vergessen...

free brauchst du in C++ nicht, dafür gibts delete. Außerdem kannst du keine Stackobjekte free'n. Du solltest dir nochmal genau anschauen, wie die Speicherverwaltung bei C++ funktioniert.

greetz KN4CK3R

]]>https://www.c-plusplus.net/forum/post/2230293https://www.c-plusplus.net/forum/post/2230293Thu, 05 Jul 2012 12:36:54 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 12:42:32 GMT]]>Du brauchst auf jeden Fall mindestens (hab nur flüchtig drübergeschaut) an geeigneter Stelle:

ReleaseDC(0, screen);
DeleteObject(bmp);
DeleteDC(target);
delete []bmpBuffer;

Die ersten drei sind für Windows-Ressourcen, die Du anforderst (dazu empfiehlt sich immer ein Blick in die MSDN!), das letzte für dynamisch via new angeforderten Speicher.

]]>https://www.c-plusplus.net/forum/post/2230294https://www.c-plusplus.net/forum/post/2230294Thu, 05 Jul 2012 12:42:32 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 12:47:42 GMT]]>Also entweder ReleaseDC (auf das Handle, das du per GetDC erhalten hast) oder DeleteDC (CreateCompatibleDC).
Mit vector ist es auch nicht komplizierter:

// ab deiner Zeile, die zu ersetzen ist:
//uint8_t *bmpBuffer = new uint8_t[bminfo.bmiHeader.biSizeImage];
vector<uint8_t> bmpBuffer(bminfo.bmiHeader.biSizeImage); 

x = GetDIBits(target, bmp, 0, height, &bmpBuffer[0], &bminfo, DIB_RGB_COLORS); 
assert(x); 

BITMAPFILEHEADER fheader; 
fheader.bfType = 0x4D42; 
fheader.bfSize = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER) + bminfo.bmiHeader.biSizeImage; 
fheader.bfReserved1 = 0; 
fheader.bfReserved2 = 0; 
fheader.bfOffBits = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER); 

std::ofstream bmp_file("file.bmp", std::ios_base::binary); 

bmp_file.write((const char *) &fheader, sizeof(fheader)); 
bmp_file.write((const char *) &(bminfo.bmiHeader), sizeof(bminfo.bmiHeader)); 
bmp_file.write((const char *) &bmpBuffer[0], bminfo.bmiHeader.biSizeImage); 

DeleteObject(bmp);
DeleteDC(target);
ReleaseDC(NULL, screen);

Aber wie geschrieben, kleine Wrapperklassen sind dem vorzuziehen und schnell geschrieben:

struct HBitmapWrapper
{
	HBitmapWrapper(HBITMAP bm) : bm(bm) {}
	~HBitmapWrapper() {DeleteObject(bm);}
	operator HBITMAP() {return bm;} 
private:
	HBITMAP bm;
	HBitmapWrapper(const HBitmapWrapper&);
	HBitmapWrapper& operator=(const HBitmapWrapper& b);
};

Und im Code

HBitmapWrapper bmp=CreateCompatibleBitmap(screen, width, height);

Natürlich kann das noch weiter ausgebaut werden: Template um auch andere Handles zu verwalten (HFONT, HBRUSH, HPEn usf.), Funktion zum Freigeben mitliefern etc.

]]>https://www.c-plusplus.net/forum/post/2230298https://www.c-plusplus.net/forum/post/2230298Thu, 05 Jul 2012 12:47:42 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 12:53:31 GMT]]>Nachtrag:
Da Deine Funktion ja dann jede Sekunde aufgerufen wird, solltest Du im Taskmanager beobachten können, ob die Anzahl der benötigten GDI-Objekte immer größer wird. Falls nicht, gibst Du sie wohl alle säuberlich wieder frei, falls doch, hast Du entweder eins vergessen, oder beim Freigeben etwas falsch gemacht.

]]>https://www.c-plusplus.net/forum/post/2230299https://www.c-plusplus.net/forum/post/2230299Thu, 05 Jul 2012 12:53:31 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 12:53:43 GMT]]>Und, nicht vergessen: Bei voneinander abhängigen Windows-Ressourcen immer in umgekehrter Reihenfolge löschen: Wenn du eine Bitmap mit einem HDC verknüpfst, dann muss am Ende erst der HDC gelöscht werden, dann die Bitmap.

Und der Screen wird nicht gelöscht, sondern muss freigegeben werden:
CreateCompatibleDC <--> DeleteDC
GetDC <--> ReleaseDC

]]>https://www.c-plusplus.net/forum/post/2230301https://www.c-plusplus.net/forum/post/2230301Thu, 05 Jul 2012 12:53:43 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Thu, 05 Jul 2012 13:25:04 GMT]]>Auf gültige Rückgabewerte zu prüfen ist auch immer eine gute Idee...

]]>https://www.c-plusplus.net/forum/post/2230309https://www.c-plusplus.net/forum/post/2230309Thu, 05 Jul 2012 13:25:04 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Sat, 07 Jul 2012 22:09:31 GMT]]>Danke erstmal für die Hinweise.

Das mit dem Wrapper und dem vector hat nicht funktioniert.

Scheinbar bin ich zu unwissend was das ganze angeht.

In Java hat das alles funktioniert 😉 (wahrscheinlich darf man das Wort hier nicht nennen)

so sieht mein Code aus

#include <iostream>
#include <fstream>
#include <cstdint>
#include <cassert>

#include <Windows.h>

void capture();
char c;

void main (){

	int i =0;
	while(1){
	i++;

	std::cout<<i<<std::endl;

	capture();
	remove("file.bmp");
	}

}
void capture()
{
	HWND hWnd = FindWindow(NULL, NULL);
    HDC screen = GetDC(NULL);
    HDC target = CreateCompatibleDC(screen);

    /*uint32_t width = GetSystemMetrics(SM_CXSCREEN);
    uint32_t height = GetSystemMetrics(SM_CYSCREEN);*/

	uint32_t width = 640;
	uint32_t height= 480;

    HBITMAP bmp = CreateCompatibleBitmap(screen, width, height);

    SelectObject(target, bmp);

    int x = 0;
    x = BitBlt(target, 0, 0, width, height, screen, 10,10, SRCCOPY | CAPTUREBLT);
    assert(x);

	BITMAPINFO bminfo;
    bminfo.bmiHeader.biBitCount = 16;
    bminfo.bmiHeader.biSize = sizeof(BITMAPINFOHEADER);
	bminfo.bmiHeader.biCompression = BI_RGB;
    bminfo.bmiHeader.biPlanes = 1;
    bminfo.bmiHeader.biWidth = width;
    bminfo.bmiHeader.biHeight = height;
    bminfo.bmiHeader.biSizeImage = width * 4 * height; // must be DWORD aligned
    bminfo.bmiHeader.biXPelsPerMeter = 0;
    bminfo.bmiHeader.biYPelsPerMeter = 0;
    bminfo.bmiHeader.biClrUsed = 0;
    bminfo.bmiHeader.biClrImportant = 0;

    uint8_t *bmpBuffer = new uint8_t[bminfo.bmiHeader.biSizeImage];

	x = GetDIBits(target, bmp, 0, height, bmpBuffer, &bminfo, DIB_RGB_COLORS);
    assert(x);

    BITMAPFILEHEADER fheader;
    fheader.bfType = 0x4D42;
    fheader.bfSize = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER) + bminfo.bmiHeader.biSizeImage;
    fheader.bfReserved1 = 0;
    fheader.bfReserved2 = 0;
    fheader.bfOffBits = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER);

    std::ofstream bmp_file("file.bmp", std::ios_base::binary);

    bmp_file.write((const char *) &fheader, sizeof(fheader));
    bmp_file.write((const char *) &(bminfo.bmiHeader), sizeof(bminfo.bmiHeader));
    bmp_file.write((const char *) bmpBuffer, bminfo.bmiHeader.biSizeImage);
    bmp_file.close();
	ReleaseDC(NULL, screen);
	DeleteDC(target); 
	DeleteObject(bmp);

}

So habe ich es jetzt.
Er läuft jetzt schon deutlich länger.

Nur ich kann quasi zusehen, wie er den RAM voll schreibt und nicht mehr entleert.

Evtl. kann mir jemand noch einen Hint geben 😕

Danke schonmal

]]>https://www.c-plusplus.net/forum/post/2230939https://www.c-plusplus.net/forum/post/2230939Sat, 07 Jul 2012 22:09:31 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Sat, 07 Jul 2012 22:15:35 GMT]]>Es fehlt immer noch
delete [] bmpBuffer;

]]>https://www.c-plusplus.net/forum/post/2230941https://www.c-plusplus.net/forum/post/2230941Sat, 07 Jul 2012 22:15:35 GMT<![CDATA[Reply to Screenshot mit GDI in Endlosschleife beendet sich nach x durchläufen on Sat, 07 Jul 2012 23:00:12 GMT]]>das sieht gut aus...
danke...

ich werde jetzt die nacht über testen;)

]]>https://www.c-plusplus.net/forum/post/2230947https://www.c-plusplus.net/forum/post/2230947Sat, 07 Jul 2012 23:00:12 GMT