Неправильно отображается RGBA?

Question

[Gokilla]

Есть исходное изображение bmp размера 960*495

Вот такое в программе

Вот код:

void FileReader::FileReader__(const char__ *filename,const char__ *state)
{
	std::string ext="";
	int__ l=strlen(filename);
	while(--l>0&&filename[l]!='.')
		ext+=filename[l];
	if(strstr(ext.c_str(),"fnoc") || strstr(ext.c_str(),"lmx"))
	{
		ext_=true;
		file=fopen(filename,state);
		open_=true;
		char__ ch;
		while((ch=fgetc(file))!=-1)
		{
	        if(ch==10)
				strokoffile++;
			countbyte++;
		}
		fclose(file);
		file=fopen(filename,state);
	}
	else
	{
		life=open(filename,_A_ARCH);
		ext_=false;
		org=0;
	}
}
void__ FileReader::ReadFile(int__ size_,void__ *buf)
{
	lseek(life,0,org);
    read(life,buf,size_);
	org=1;
}
void__ Scene::EnableTexture(float__*texcoor, float__*vercoor)
{
    glBegin(GL_QUADS);
    glTexCoord2f(texcoor[0],texcoor[2]);
    glVertex2f(vercoor[0],vercoor[2]);
    glTexCoord2f(texcoor[1],texcoor[2]);
    glVertex2f(vercoor[1],vercoor[2]);
    glTexCoord2f(texcoor[1],texcoor[3]);
    glVertex2f(vercoor[1],vercoor[3]);
    glTexCoord2f(texcoor[0],texcoor[3]);
    glVertex2f(vercoor[0],vercoor[3]);
    glEnd();
}
    NewData=new uchar__[4*960*495];
    NewTexture=new uint__[1];
void__ Scene::ShowBackGround()
{
	filereader->FileReader__("J:\\фукрпкявамчс\\content\\Загрузки\\table.bmp","r");
	filereader->ReadFile(960*495*4,NewData);
	for(float__ i=0;i<960*495;i++)
    {
        int__ index=i*4;
        uchar__ R,G,B,A;
        B=NewData[index];
        G=NewData[index+1];//Green(B)
        A=NewData[index+2];//Red(G)
        R=NewData[index+3];//blue(R)
        NewData[index]=R;
        NewData[index+1]=G;//(G)
        NewData[index+2]=B;//(B)
        NewData[index+3]=A;//(R)
        //GBR
        //BGsR
    }
    glBindTexture(GL_TEXTURE_2D,NewTexture[0]);
    gluBuild2DMipmaps(GL_TEXTURE_2D,GL_RGBA,960,495,GL_RGBA,GL_UNSIGNED_BYTE,NewData);
    EnableTexture(GetTextureCoordinats(10),GetVertexCoordinats(0));
	return;
}

В чем ошибка?

Comments

[Михаил Усоцкий]

А вы уверены, что RGBA? На моей памяти встречал формат ARGB. Судя по всему, так и есть. У вас каналы сместились. Например, альфа стал красным, а красный стал зелёным, зелёный - синим, синий - красным.

[Gokilla]

Михаил Усоцкий, подскажи что в цикле поменять?

[Евгений Шатунов]

Михаил Усоцкий , упаковка каналов может быть любой. RGBA, BGRA, ARGB, ABGR. Еще может быть черезрядная и поканальная упаковка.

Зеленый цвет в центре доски имеет RGB значение: 4, 142, 4. Альфа, очевидно: 255.
После рендеринга в том же месте RGB уже такой: 142, 255, 4. Альфа, очевидно: 4.
Какие каналы где находятся - очевидно. Это действительно ARGB.

Gokilla , чтобы у тебя был RGBA, тебе надо поменять порядок каналов в момент вычитки картинки.

[Михаил Усоцкий]

Gokilla, у вас эта часть некорректна, если они загружаются с файла (а там должна быть чёткая последовательность):

B=NewData[index];
        G=NewData[index+1];//Green(B)
        A=NewData[index+2];//Red(G) <- здесь должно быть красный по смещению
        R=NewData[index+3];//blue(R) <- здесь должно быть альфа по смещению

// здесь решили перемешать что-то? То есть ARGB => ABGR
        NewData[index]=R;
        NewData[index+1]=G;//(G)
        NewData[index+2]=B;//(B)
        NewData[index+3]=A;//(R)

[Gokilla]

Евгений Шатунов, это не argb вся доска стала малиновой

for(float__ i=0;i<960*495;i++)
{
int__ index=i*4;
uchar__ R,G,B,A;
A=NewData[index]; //R
R=NewData[index+1];//Green(B) //G
G=NewData[index+2];//Red(G) //A
B=NewData[index+3];//blue(R) //B
NewData[index]=R;
NewData[index+1]=G;//(G)
NewData[index+2]=B;//(B)
NewData[index+3]=A;//(R)
//GBR
//BGsR
}

[Михаил Усоцкий]

Gokilla, неправильно. Должно быть:
B=NewData[index];
G=NewData[index+1];
R=NewData[index+2];
A=NewData[index+3];
Ведь у нас формат по-умолчанию должно быть вот так: ARGB [+3 +2 +1 +0]

[Gokilla]

Михаил Усоцкий,
for(float__ i=0;i<960*495;i++)
{
int__ index=i*4;
uchar__ R,G,B,A;
R=NewData[index]; //R
G=NewData[index+1];//Green(B) //G
A=NewData[index+2];//Red(G) //A
B=NewData[index+3];//blue(R) //B
NewData[index]=R;
NewData[index+1]=G;//(G)
NewData[index+2]=B;//(B)
NewData[index+3]=A;//(R)
//GBR
//BGsR
}

Вот так если?

[Gokilla]

Михаил Усоцкий, ваш вариант

B=NewData[index]; //R
G=NewData[index+1];//Green(B) //G
R=NewData[index+2];//Red(G) //A
A=NewData[index+3];//blue(R) //B
NewData[index]=R;
NewData[index+1]=G;//(G)
NewData[index+2]=B;//(B)
NewData[index+3]=A;//(R)

[Михаил Усоцкий]

Gokilla, а вот это почему решили перемешать?
NewData[index]=R;
NewData[index+1]=G;//(G)
NewData[index+2]=B;//(B)
NewData[index+3]=A;//(R)

Ведь должно быть по формату ARGB [+3 +2 +1 +0]
NewData[index]=B;
NewData[index+1]=G;
NewData[index+2]=R;
NewData[index+3]=A;

Если у вас используется обратный порядок байтов, то:
NewData[index]=A;
NewData[index+1]=R;
NewData[index+2]=G;
NewData[index+3]=B;

[Gokilla]

Михаил Усоцкий, потому что
gluBuild2DMipmaps(GL_TEXTURE_2D,GL_RGBA,960,495,GL_RGBA,GL_UNSIGNED_BYTE,NewData);
А ваш вариант вся фиолетовая доска опять

[Евгений Шатунов]

Gokilla , ты так долго ответа ждать будешь. Что самое интересное, твои действия сейчас немного напоминают попытку троллинга. :)
Я тебе советую найти все каналы по отдельности. Сперва найти красный, потом зеленый, потом - синий, альфа сама найдется. Когда ищешь канал - все остальные каналы в 255, так проще.
Есть подозрение, что ты вообще не с того места начинаешь читать цвета, в результате чего у тебя просто мешанина в каналах.

[Gokilla]

Евгений Шатунов, хочу сказать что первое исходное изображение это png ,bmp не загрузилось на этот сайт

[Gokilla]

Евгений Шатунов, в том что не с того места читаю это скорее всего да

[Михаил Усоцкий]

Gokilla, интересно... решил изучить данную функцию, у него несколько отличается от общепринятых. Получается, что вам надо ARGB => RGBA.
NewData[index]=A;
NewData[index+1]=B;
NewData[index+2]=G;
NewData[index+3]=R;
Где-то так должно быть.

[Gokilla]

Михаил Усоцкий, я уже написал что не подошло

[Михаил Усоцкий]

Gokilla, да что же такое?! Всегда привык видеть формат ARGB. А тут нате. Хмм... А попробуйте методом тыка перемешать каналы, чтобы получить нужное изображение. Тут вариантов перебора не так уж и много.

[Gokilla]

Михаил Усоцкий, уже нашел, но к сожалению он не универсальный для каждого изображения по разному

[Михаил Усоцкий]

Gokilla, посмотрел ваш ответ. Довольно интересно. В поиске нашёл ответ, что для bmp может встречаться формат GBAR.

[Михаил Усоцкий]

Похоже, я ошибся. С bmp всё работает иначе. Надо сначала прочитать маску, а потом строить изображение. А изображение начинается с некоторого смещения.
Формат Bmp-файла
Описание формата BMP
Короче, вы пропустили важную часть в файле, которая отвечает, как строить изображение.

Answer 1

[Gokilla]

for(float__ i=54;i<960*495;i++)
    {
        int__ index=i*4;
        uchar__ R,G,B,A;
        
		R=NewData[index];				//R
        A=NewData[index+1];//Green(B)	//G
        B=NewData[index+2];//Red(G)		//A
        G=NewData[index+3];//blue(R)	//B
        NewData[index]=R;
        NewData[index+1]=G;//(G)
        NewData[index+2]=B;//(B)
        NewData[index+3]=A;//(R)
		
        //GBR
        //BGsR
    }

вот ответ

Comments

[Евгений Шатунов]

Gokilla , ошибка чтения на 2 байта, видно как в воде. Артефактов - как грязи.

[Gokilla]

Евгений Шатунов, поправка на 54,вот это помогло
opengl-tutorial.blogspot.ru/p/5.html

Answer 2

[VoidVolker]

filereader->FileReader__("J:\\фукрпкявамчс\\content\\Загрузки\\table.bmp","r");
  filereader->ReadFile(960*495*4,NewData);

Формат BMP полностью документирован и описан — готовых библиотек полно. Так что либо используйте готовый код для чтения файла либо реализуйте самостоятельно. Данный формат поддерживает палитры, а цвет пикселя может быть от одного до 64 бит (т.е. до восьми байт). В данном случае явно происходит некорректная интерпретация данных о цвете. Кроме того, он еще и заголовок содержит, в котором полностью описана вся информация об изображении и формат пикселя, и которая в данном коде полностью игнорируется. А еще там строки выравниваются по степени четверки - так что в любом случае все пиксели читаются в два цикла (второй - вложенный).