GetDIBits

[Алексей++]

Не пойму, что я не так делаю. Вроде бы всё по докам, а вот косяк...

код доставания массива битов такой:

Код:

bool CMy1Dlg::TakeScreenData()
{
	CDialog* m_Dlg=this;

	std::vector<BYTE> DATA;

	bool bResult=false;

	CDC* dc=0;
	for(;;)
	{
		DATA.clear();
		RECT dstRect;

		m_Dlg->GetClientRect(&dstRect);

		LONG wid=dstRect.right -dstRect.left+1;
		LONG hig=dstRect.bottom-dstRect.top +1;

		dc=m_Dlg->GetDC();

		BITMAPINFO bi;
		::memset(&bi,0,sizeof(bi));
		bi.bmiHeader.biSize          =sizeof(bi.bmiHeader);

		DWORD dwdLenInBytes=0;
		const CBitmap* bmp=dc->GetCurrentBitmap();

		if(!::GetDIBits(*dc,*bmp,0,0,0,&bi,DIB_RGB_COLORS) && bi.bmiHeader.biBitCount)break;
		
		bi.bmiHeader.biWidth=4;
		bi.bmiHeader.biHeight=4;

		dwdLenInBytes=
		(
			(
				bi.bmiHeader.biWidth*bi.bmiHeader.biBitCount+31
			)
			/32
		)
		*4
		*bi.bmiHeader.biHeight;

		if(!dwdLenInBytes)break;

		DATA.resize(dwdLenInBytes);

		if(DATA.size()<dwdLenInBytes)break;

		DWORD dwdLines=::GetDIBits(
			*dc,*bmp
			,0,bi.bmiHeader.biHeight
			,&DATA[0]
			,&bi
			,DIB_RGB_COLORS);


		bResult=true;
		break;
	}
	
	if(dc)m_Dlg->ReleaseDC(dc);
	return bResult;
}

далее. Вот скрины из дебажного режима
1 - до чтения битового массива
2 - после

на первом в просмотрщике памяти содержимое переменной bi выделено серым. 8 байтов со значением 0xcc - это дебажный маячок на порчу памяти, он не должен быть затронут.
Но смотрим на скрин 2
0x00ff0000 - нормально, на месте,  а вот 8 байтов 0xcc затёрты {00 ff 00 00 ff 00 00 00}

и я не понимаю, почему! Дебаггер то просто ругается, а релиз падает...

Если сделать финт ушами, а именно: объявить массив из двух BITMAPINFO, но работать только с первым, то лишней затирки, естественно, не будет (испортится второй , неиспользуемый, элемент). Но это костыль.

Всё-таки хочется узнать, в чём причина то

[Ochkarik]

потерто.

[Алексей++]

я знаю ) Вот хочется узнать - почему

[Ochkarik]

If the lpvBits parameter is a valid pointer, the first six members of the BITMAPINFOHEADER structure must be initialized to specify the size and format of the DIB. The scan lines must be aligned on a DWORD except for RLE compressed bitmaps.

- где инициализация этих шести полей?)

а....все, вижу)

[Алексей++]

да, вот тут есть

Код:

if(!::GetDIBits(*dc,*bmp,0,0,0,&bi,DIB_RGB_COLORS) && bi.bmiHeader.biBitCount)break;

[Ochkarik]

 bmiColors[] -  массив. в зависимости от...
они в структуре только первый элемент определили. похоже?

[Алексей++]

это элемент под один 32-битный пиксел (ну и ещё варианты, в зависимости от битности)

Но почему так сделано - тоже не могу понять. Что-то тут хитрое

Добавлено через 4 минуты и 9 секунд:
пришлось выдрать готовый код отсюда. Код страшненький, но рабочий, тут этой затирки не происходит
http://msdn.microsoft.com/en-us/library/dd183402(v=VS.85).aspx

разницу при работе со структурой я не увидел...

Добавлено через 44 секунды:
(процедура CaptureAnImage)

[Ochkarik]

там жешь написано.
The bmiColors member contains one of the following:
...
An array of RGBQUAD. The elements of the array that make up the color table.
An array of 16-bit unsigned integers that specifies indexes into the currently realized logical palette.
The number of entries in the array depends on the values of the biBitCount and biClrUsed members of the BITMAPINFOHEADER structure.

там просто union на самом деле...
про остальное не ведаю.... не разбираюсь я в графике...

PS http://msdn.microsoft.com/en-us/library/dd183376(VS.85).aspx
biBitCount = 32
The bitmap has a maximum of 2^32 colors. If the biCompression member of the BITMAPINFOHEADER is BI_RGB, the bmiColors member of BITMAPINFO is NULL. Each DWORD in the bitmap array represents the relative intensities of blue, green, and red, respectively, for a pixel. The high byte in each DWORD is not used. The bmiColors color table is used for optimizing colors used on palette-based devices, and must contain the number of entries specified by the biClrUsed member of the BITMAPINFOHEADER.
If the biCompression member of the BITMAPINFOHEADER is BI_BITFIELDS, the bmiColors member contains three DWORD color masks that specify the red, green, and blue components, respectively, of each pixel. Each DWORD in the bitmap array represents a single pixel.
When the biCompression member is BI_BITFIELDS, bits set in each DWORD mask must be contiguous and should not overlap the bits of another mask. All the bits in the pixel do not need to be used.

может так...?)

[Алексей++]

Ochkarik, там вовсе не юнион

Код:

typedef struct tagBITMAPINFO {
  BITMAPINFOHEADER bmiHeader;
  RGBQUAD          bmiColors[1];
} BITMAPINFO, *PBITMAPINFO;

Добавлено через 1 минуту и 55 секунд:
>>может так...?)

может, но почему тогда там под одну маску место ? Вручную что ли обеспечивать нужно ? ))

Добавлено через 6 минут и 56 секунд:
struct s_BITMAPINFO_2RGBQUAD:public BITMAPINFO
{
   RGBQUAD GM;
   RGBQUAD BM;
};

как-то так

[Вад]

Места там под 1 пиксел, потому что надо выделять сколько необходимо, а потом кастовать. Типичный майкрософтовский подход, чтобы все поля уже в структуре были объявлены, но при этом структура имеет неизвестный заранее размер.
типа

Код: (C++)

BITMAPINFO * info = (BITMAPINFO*)new char[sizeof(BITMAPINFO) + sizeof(RGBQUAD) * PIXEL_COUNT];

[Алексей++]

Вад, первый раз такой типичный подход встречаю И вот это

+ sizeof() * PIXEL_COUNT

ну никак не соответствует. Добавляется 2 лишних двойных слова, а по твоей формуле - их будет (PIXEL_COUNT-1) лишних.
А ведь под пикселы другой массив указуется

А эти 3 RGBQUAD - там действительно маски цветов получаются

Добавлено через 2 часа, 58 минут и 53 секунды:
собственно, разница нашлась:

перед капитальным, вторым, вызовом GetDIBits нужно установить
bi.bmiHeader.biCompression = BI_RGB

(по умолчанию после первого вызова там BI_BITFIELDS)

[Ochkarik]

Алексей1153++,
PS майкрософт действительно оооочень часто так делает со структурами  неопределенной длины) собственно с их подачи даже я стал так поступать в некоторых случаях) только обычно я еще сразу определяю дефенишн для структуры в замену обычного sizeof.

[Алексей++]

Ochkarik, я тоже делаю похожие вещи, но не так, как у них сделано в этом случае. У меня поля указывают размеры всего - и самой структуры, и внешней части. А вот такие перекрытия я не делаю. Если уж внешняя часть, то внешняя

[Ochkarik]

Алексей1153++, неэкономный ты Леша!) кроме того, не всегда нужен размер из которого потом количество элементов придется каждый раз считать, иногда удобнее количество элементов указывать. или тип обработки элементов...
размер майкрософт обычно указывает только когда структура может проходить через функции, которым содержимое структуры до лампочки)

а насчет флага, там написано было, я уже копипастил)
If the biCompression member of the BITMAPINFOHEADER is BI_BITFIELDS, the bmiColors member contains three DWORD color masks that specify the red, green, and blue components, respectively, of each pixel. Each DWORD in the bitmap array represents a single pixel.
внимательней надо быть

[Алексей++]

про экономность не соглашусь, ну а про цитату - это я уже тоже увидел ) Вот , если указать BI_RGB, то этого довеска не будет

[Вад]

Вад, первый раз такой типичный подход встречаю И вот это

+ sizeof() * PIXEL_COUNT

ну никак не соответствует. Добавляется 2 лишних двойных слова, а по твоей формуле - их будет (PIXEL_COUNT-1) лишних.
А ведь под пикселы другой массив указуется

Ну, за точность размера я не ручаюсь - суть в том, что выделяешь размер, равный размеру основной структуры + нужный размер буфера для данных.

В принципе, это удобно при дальнейшем обращении со структурой: работаешь, как ни в чём не бывало. Только выделять память слегка неаккуратно и неудобно.