Lösungsvorschläge | Client-Server | Desktop übertragen

net 0

babelduo (net eingeloggt) schrieb:

raus kommen allerdings auch komische farbwerte!!
21_21_21
22_22_22
23_23_23
...

kann doch sein. vielleicht irgendwelche grautöne.
aber vielleicht sind in meinem code auch fehler (operatorreihenfolge oder sowas).
das prinzip sollte jedoch klar sein: jeweils 5 bits pro farbe, alle 3 in ein WORD gequetscht...

babelduo

so - ich war nun längere zeit nicht mehr aktiv wegen urlaub und so

habe mir nun eine testView-MFC-Anwendung geschrieben die einen Screenshot
macht und ihn Pixel für Pixel ausgibt...

das problem: das Bild hat nen komischen Farb-Stich
url: http://img87.imageshack.us/img87/7452/untitledlm0.jpg

hier mein code:

das bild wird auf 800x600 geschrumpft...

HDC hdc, hdc2;
		int width,height;

		width=800;
		height=600;

		hdc=::GetDC(NULL); 

		HBITMAP aBmp; 
		BITMAPINFO bi; 
		HGDIOBJ OldObj; 
		void *dibvalues; 

		BITMAPINFOHEADER bmih; 

		hdc2=CreateCompatibleDC(hdc); 

		ZeroMemory(&bmih,sizeof(BITMAPINFOHEADER)); 
		bmih.biSize=sizeof(BITMAPINFOHEADER); 
		bmih.biHeight=height; 
		bmih.biWidth=width; 
		bmih.biPlanes=1; 
		bmih.biBitCount=16; 
		bmih.biCompression=BI_RGB; 
		bmih.biSizeImage = ((((bmih.biWidth * bmih.biBitCount) + 31) & ~31) >> 3) * bmih.biHeight; 
		bmih.biXPelsPerMeter = 0; 
		bmih.biYPelsPerMeter = 0; 
		bmih.biClrImportant = 0; 

		bi.bmiHeader=bmih; 

		aBmp=CreateDIBSection(hdc,&bi,DIB_RGB_COLORS,(void**)&dibvalues,NULL,NULL); 

		if (aBmp==NULL) 
		{ 
			//OutputDebugString("CreateDIBSection failed!\n"); 
		} 

		OldObj=SelectObject(hdc2,aBmp);

		StretchBlt(hdc2,0,0,width,height,hdc,0,0,GetSystemMetrics(SM_CXSCREEN),GetSystemMetrics(SM_CYSCREEN),SRCCOPY);

		// Screenshot ausgeben
		CDC* DC;
		DC=GetDC();

		int x=0;
		int y=height;

		int red,green,blue;

		for (int i=0; i<height*width; i++)
		{
			WORD *w = (WORD*)dibvalues + i;

			red = *w & 0x1f;
			green = (*w>>5) & 0x1f;
			blue = (*w>>10) & 0x1;

			DC->SetPixel(x,y,RGB(red,green,blue));

			x++;

			if(x==width)
			{
				x=0;
				y--;
			}
		}

woran liegt das? ansonsten klappt ja alles perfekt und ich kann
weiterarbeiten

danke schonmal!

Hi babelduo,

hast du die Möglichkeit den Code mal online zu stellen, bzw. komplett zu posten?

Mal sehen was sich da noch drehen lässt, interessiert mich das Thema.

lg
Dibmen

Hast du schon eine lösung für dein Problem mit den Farbwerten gefunden?

Meep Meep

hola

mal so ein gedanke:
wie waere es wenn man alle pixel des letzten bildes mit dem aktuellen subdrahieren wuerde ? nicht veraenderte sektionen wuerden dann '0' sein. bei wenig aenderungen koennte man dann bestimmt mit ner RLE oder so ne gute kompression bekommen und man koennte den algo auch recht gut parallelisieren. muesste von der geschwindigkeit her um einiges besser sein. was meint ihr dazu ?

Meep Meep

Das ändert nicht an der Tatsache, dass momentan die Farbwerte total durcheinander sind

Aber grundsätzlich...

babelduo

@Meep Meep
ja danke schonmal! darüber muss ich mir nun auch gedanken machen

ich habe hier mein projekt mal hochgestellt... vllt. findet ihr ja den
fehler mit den farbwerten - wär cool!

http://www.babelduo.de/screen.rar
(MFC-Anwendung Visual C++)

danke schonmal

p.s: wenn man Enter drückt gehts los...

Andromeda

DC->SetPixel (x,y, (red<<16 | green<<8 | blue)<<3);

Er hat damit aber noch immer einen Farbstich im Bild.

Hast du denn das Senden via Winsock schon hinbekommen?

hustbaer

Naja, zum Ausgeben solltest du das richtig hochskalieren.
Also (r << 19) + (g << 10) + (b << 3) wenn r, g, b in 16 bit 5-6-5 vorliegen.

Vielleicht bin ich einfach zu doof, aber ich fummel mir dabei einen ab
und erhalte, wie auch immer ich es anstelle, immer einen heftigen Farbstich
im Bild.

Bei mir seh ich auch nur ein lustig eingefärbtes Bild,...aber wems reicht

Was sagt der Threadersteller dazu? Gelöst?

hustbaer

blue = (*w) & 0x1f;
			green = ((*w)>>5) & 0x1f;
			red = ((*w)>>10) & 0x1f;

			DC->SetPixel(x,y,RGB((red * 8), (green * 8), (blue * 8)));

So, das funktioniert.
Das 16 Bit Format wo Windows verwendet wenn man bloss BI_RGB angibt ist halt RGB555. Natürlich muss das dann noch Faktor 8 hochskaliert werden...

Noch besser ist das, damit Weiss Weiss bleibt und nicht Grau wird:

DC->SetPixel(x,y,RGB((red << 3) + (red >> 2), (green << 3) + (green >> 2), (blue << 3) + (blue >> 2)));

Andromeda

hustbaer schrieb:

blue = (*w) & 0x1f;
			green = ((*w)>>5) & 0x1f;
			red = ((*w)>>10) & 0x1f;

			DC->SetPixel(x,y,RGB((red * 8), (green * 8), (blue * 8)));

[/cpp]

Ich möchte ja nicht aufdringlich sein, aber das entspricht genau meinem Vorschlag sechs Beiträge darüber

PS: Eine unterschiedliche Gewichtung der Farbkomponenten ist IMHO nur für LC-Displays nötig.

Naja, er wollte ja auch den differenten Teil ermitteln.
Dann muss er ja noch irgendwie eine Prüfung einbinden.

Ich kann mir nicht vorstellen, dass das so einfach bzw. schnell geht
und man das dann noch sinnvoll über Winsock senden kann. :xmas2:

hallo ^^

ja danke euch schonmal sind ja viele vorschläge hier... allerdings
habe ich bei den vorschlägen entweder einen blauen oder einen gelben
farbstich?

klappt denn ein vorschlag bei euch richtig?

@Fruggle
ja das mit über WinSock senden ist echt kompliziert
den vorschlag mit dem differenten teil ermitteln (also altes bild
- neue) find ich recht interessant...

man könnte das ja erstmal offline testen und halt immer wenn man ne
taste drückt (oder mit nem timer) wird nur der differente teil
neu gezeichnet...

wenn jemand auch lust hat da rumzuprobieren gerne - hier nochmal mein
projekt:

http://www.babelduo.de/screen.rar
(MFC-Anwendung Visual C++)

ich hab in den nächsten tagen wohl net so viel zeit...

aber vielen dank schonmal an euch

Bleib am Ball. Das hört sich alles sehr interessant an.
Werde mir das Projekt auch mal ansehen Poste hier doch mal,
wenn du weiter machst und bei dem Differenzieren bist.

hustbaer

babelduo, bist du doof? Ich habe es runtergeladen und ausprobiert und was ich zuletzt gepostet habe funktioniert. Nimm es oder lass es aber frag bloss nicht nochmal obs denn nu funktioniert. Probiers wenigstens aus du faules Stück *****.

@Andromeda:

Ich möchte ja nicht aufdringlich sein, aber das entspricht genau meinem Vorschlag sechs Beiträge darüber

Äh, ja, richtig. Wenn man die Korrektur von "0x1" -> "0x1f" noch dazunimmt die vom ursprünglichen Code ausgehend noch notwendig ist, hier aber schon zu sehen war... dann schon

Andromeda

hustbaer schrieb:

Äh, ja, richtig. Wenn man die Korrektur von "0x1" -> "0x1f" noch dazunimmt...

Es war nur ein Tippfehler. Das sieht doch ein Blinder mit Krückstock