Почему записывается лишняя информация в палитровые изображение(bmp)?

Question

[Евгений Трофимов]

Если я все верно понял - когда глубина цвета 1,4 или 8, то после заголовков файла и изображения следует палитра размером 2 ^ BitCount (глубина цвета) цветов. А после этой палитры следует индексы цветов палитры.

Вот функция чтения

int Image::loadpaletteimg(FILE *f)
{
		int Palette_size = pow(2,BMInfoHeader.BitCount);//размер палитры - 2 в степени глубины цвета
		Palette=new RGBQUAD[Palette_size];
		
		for (int i = 0; i < Palette_size; i++) fread(&(Palette[i]),sizeof(RGBQUAD),1,f);//считываем палитру

		Rgbquad=new RGBQUAD*[BMInfoHeader.Height];//cтолбцы

		for (int i = 0; i < BMInfoHeader.Height; i++)//построчно записываем
		{
			Rgbquad[i]=new RGBQUAD[BMInfoHeader.Width];//строки
			fread(Rgbquad[i], sizeof(RGBQUAD), BMInfoHeader.Width, f);
		}
		if(Rgbquad) cout<<"\nИзображение успешно загружено!"<<endl;
		return 1;
}

А вот - для записи

void Image::writepaletteimg(FILE *f)
{
	int Palette_size = pow(2,BMInfoHeader.BitCount);
	for (int i = 0; i < Palette_size; i++)
		{
			fwrite(&(Palette[i]),sizeof(RGBQUAD),1,f);
			//cout<<i<<endl;
		}
	cout<<"Палитра записана!"<<endl;
	int size = BMInfoHeader.Height*BMInfoHeader.Width;

	for (int i = 0; i < BMInfoHeader.Height; i++)
	{
		fwrite(Rgbquad[i], sizeof(RGBQUAD), BMInfoHeader.Width, f);
	}
	fclose(f);
	return;
}

Я так понимаю, что проблема в моем двухмерном массиве, т.к для индексов он вроде как не нужен, но не совсем понимаю, что именно нужно изменить... Картинка пишется почти правильно, т.е ее можно прочитать, но вот в конец файла пишется какая-то гадость, забивая 70% картинки. Т.е если исходная картинка весит 9,05 КБ , то после перезаписи в моей программе - 32,8 КБ.


Как мне переписать, чтобы не записывало лишнего?

Answer 1

[Алексей]

От

Rgbquad=new RGBQUAD*[BMInfoHeader.Height];//cтолбцы

и ниже не понял.
Откуда там RGBQUAD и его размер...
Палитру уже прочитали на предыдущем шаге.

Comments

[Алексей]

После палитры идет само изображение, по 1-2-4 точки в одном байте.
И, возможно, сжатые алгоритмом RLE.

[Евгений Трофимов]

в Rgbquad я пишу само изображение, оно у меня в классе объявлено - RGBQUAD **Rgbquad;
RGBQUAD - это вот такая структура -
struct RGBQUAD
{
byte Blue;
byte Green;
byte Red;
byte Reserved;
};

[Алексей]

fread(Rgbquad[i], sizeof(RGBQUAD), BMInfoHeader.Width, f); - тут вы читаете Height строк по sizeof(RGBQUAD)*Width байт, что неверно. Фактически вы читаете Height*4*Width байт, а в 8 битном изображении всего Height*Width байт.

[Евгений Трофимов]

Алексей: так а как я могу считать по-другому, если у меня RGBQUAD занимает 4 байта... Я же не могу там fread(Rgbquad[i], 1, BMInfoHeader.Width, f) - т.к тогда он в RGBQUAD только значение одного цвета прочитает...

[Алексей]

Вы может чего недопоняли.
Из чего состоит indexed color BMP.
1) Заголовки
2) Палитра
3) Картинка

Вы прочитали заголовок (в прошлом посте).
Тут вы прочитали палитру циклом for (int i = 0; i < Palette_size; i++)
Далее вы должны читать картинку.
Картинка состоит из набора байт.
В случае indexed color 8 bit - 1 байт на 1 точку.
Почему вы на этом этапе вспоминаете про RGBQUAD я не понимаю.
Вы должны прочтиать Width*Height байт - это и будет картинка Width x Height.
Чтобы вывести эту картинку в True Color, нужно взять каждый байт, заглянуть в палитру, найти нв соответствующей позиции структуру RGBQUAD, взять из нее R, G, B - это и будет цвет точки.

Для случая indexed color 4 bit, палитра будет состоять из 128 записей, а в каждом последующем байте будет закодировано сразу 2 точки (по 4 бита на точку).

После того как вы прочитали палитру - забудьте про RGBQUAD, далее идут просто байты.

[Евгений Трофимов]

Алексей: да уже вроде и byte *palette_img объявил и palette_img = new byte[size] , и fread(palette_img, 1, size, f) делаю - а все равно выходной файл больше входного... byte - unsigned char . Причем что странно - картинка 127х64, стало быть нужно минимум 8,128 байт чтобы ее записать + заголовок, а она весит 4 198 байт..

[Алексей]

Скорее всего ваша картинка имеет 4 бита цвета.
Соответственно 128 цветов в палитре и по два пиксела в байте.
Отсюда и двойная разница.
Или еще веселее - она может быть сжата RLE алгоритмом.

[Алексей]

Смотрите BMInfoHeader.BitCount

Answer 2

[Евгений Трофимов]

int size = BMFileHeader.Size - BMFileHeader.OffsetBits;
	palette_img = new byte[size];

	for (int i = 0;i<size;i++)
	{
		fread(&palette_img[i], 1, 1, f);
	}

int size = BMFileHeader.Size - BMFileHeader.OffsetBits;
	for (int i = 0;i<size;i++)
	{
		fwrite(&palette_img[i], 1, 1, f);
	}

Вот это в общем сработало...

Answer 3

[AtomKrieg]

https://ru.wikipedia.org/wiki/BMP#BITMAPFILEHEADER
я тут википедию листаю и вижу что есть такое поле "Размер таблицы цветов в ячейках." последних версий формата bmp. Т.е. может стоит определять размер таблицы из заголовка, а не в степень двойки возводить.

Comments

[Евгений Трофимов]

Это вроде для 32-битных, я то пока с простенькими - 8-битными пытаюсь разобраться .. Просто я действительно пишу ширина*высота*размер структуры(4) байт, хотя нужно просто высота*ширина байт... И еще я видимо сжатие не учитываю, хотя оно видимо есть) В общем там и не разобрался, если писать в 4 раза меньше байт - то некоторые картинки все же правильно сохраняются, а некоторые нет..

[AtomKrieg]

Евгений Трофимов: в отладчике смотрите