バイナリを連結する方法としてベターなやり方が知りたい。

Question

提示コードですが上のコードはバイナリの連結を実験したものです。下のコードは.pngファイルの実データ部を表示させる関数です。
下のコードで複数あるIDATチャンクの実データを一つにまとめる必要があるでバイナリの連結が発生すると思うのですがどのやり方がベターなののでしょか？上のコードでしょうか？

[これを行う目的]　
.pngデータのIDATAチャンクの実データを取り出したい。が複数存在するのでそれを一つにまとめたい。

また提示コードの// #### コメント内部のコードでアクセスエラーは発生するのですがこれはなぜでしょうか？

#include <iostream>
#include "string.h"


unsigned char* gPictureData;

void AddPictureData(const unsigned char* data)
{
	int size = sizeof((char*)gPictureData) + sizeof((char*)data);

	unsigned char* str = new unsigned char[size] {0x00};
	
	
	strcat_s((char*)str, sizeof(str), (char*)gPictureData);

	// ########################################################
	strcat_s((char*)str, sizeof(str), (char*)data);
	// ########################################################


	delete gPictureData;
	gPictureData = nullptr;





	//gPictureData = new unsigned char[size]{0x00};
	//strcat_s((char*)gPictureData, sizeof(str), (char*)str);
	//gPictureData[size + 1] = '\0';
}

int main() 
{
	gPictureData = new unsigned char[5]{0x00,0x00,0x00,0x00};
	const unsigned char* st = new unsigned char[5]{ 0x01,0x01,0x01,0x01 };
	


	for (int i = 0; i < sizeof(gPictureData); i++)
	{
		printf("%x ", gPictureData[i]);

	}

	printf("\n\n\n");

	AddPictureData(st);

	for (int i = 0; i < sizeof(gPictureData); i++)
	{
		printf("%x ", gPictureData[i]);

	}



	
	return 0;
}

// ###################### IDAT チャンクを表示  ###################### 
/*
* データのバイナリをコピーして表示
*/
void ImagePrint(const char* fileName, int pos,char* copy,int copySize)
{
	std::fstream fs(fileName, std::ios_base::binary | std::ios_base::in);

	if (fs.is_open() == false)
	{
		std::cerr << "ファイルが開けません。" << std::endl;
	}
	else
	{
		//std::cout << "ファイルを開きました。" << std::endl;

		size_t fileSize = fs.seekg(0, fs.end).tellg();	//ファイルサイズ
		printf("fileSize: %zd\n",fileSize);
		fs.seekg(0, fs.beg);	//シーク位置を初期に戻す
		unsigned char* fileData = new unsigned char[fileSize];	//ファイルデータ	
		fs.read((char*)fileData, fileSize);	//バイナリ読み込み

		/////////////////////////////////////////////////////////////////////////////
		//BytePrint("texture.png",pos - 8, pos - 4);

		int dataSize = convInt(fileData, pos - 8);
	

		printf("dataSize: %d\n",dataSize);
		
		int a = 0;
		for (int i = pos; i < dataSize; i++)
		{
			std::cout<< (unsigned int)fileData[i]<<" ";

			a++;
			if (a == 20)
			{
				printf("\n");
				a = 0;
			}
		}
		

		////////////////////////////////////////////////////////////////////////////






		fs.seekg(0, fs.beg);	//シーク位置を初期に戻す
		delete fileData;	//メモリ開放

	}

	fs.close();	//ファイルを閉じる

}

Comments

[退会済みユーザー]

> 下のコードで複数あるIDATチャンクの実データを一つにまとめる必要があるので文字列の連結が発生すると思うのですがどのやり方がベターなののでしょか？上のコードでしょうか？

すみません、「文字列の連結が発生すると思う」の箇所がよくわかりません
IDATチャンクの実データは文字列ではないと思いますが

[退会済みユーザー]

userisgodさんが言われるように、pngの実データは文字列ではないケースもあるので、このままのコードで良いと思いましたー。

[int32_t]

IDATは文字列ではありませんが、std::string はバイナリも問題なく扱えるので、episteme さんの回答で良いかと思います。

[退会済みユーザー]

複数個所でポインタのサイズを取得・使用していますが何を意図してるのでしょうか
それ以外にもnew/deleteなどかなり恐い作りになっていますが……

[退会済みユーザー]

連結がしたいのです。　そのためにはまず連結後のサイズを取得して連結後の文字列を作成してそれを連結後を保存したい変数に上書きしてます。動的確保なのでnew /delete を結構使っていますｗ間違えているやり方だと後で知りましたがあまり提示コードを書き換えるのはよくないのでこのままです。

[thkana]

これまでの質問を追っていないので把握していないのですが、そもそもとして
> 複数あるIDATチャンクの実データを一つにまとめる必要がある
本当にその必要があるのですか?

さらにそれ以前に、最終的になにをしたいのか、見失っていたりはしませんか?
そもそも何をしようとしていたのか、整理してみてはいかがですか?

[退会済みユーザー]

伝わらなかったのでやや冗長に書き直しますね

なぜ、確保した配列の大きさではなく、ポインタの大きさを取得しているのでしょうか
ポインタが8バイト(64ビット)の環境であると以下の箇所は、範囲外を参照するループとなっています

> gPictureData = new unsigned char[5]{0x00,0x00,0x00,0x00};
> for (int i = 0; i < sizeof(gPictureData); i++)
> {
> printf("%x ", gPictureData[i]);
> }

> まず連結後のサイズを取得して

ポインタのサイズを取得、それを加算しているだけですが理解していますか？
わからないようであれば、sizeof演算子について調べてみてください

[退会済みユーザー]

OpenGLでPNGを使用するという流れからきてるとは思うのですが、
その時点で、「stb_image.h」を使うという回答がついており、正直やりたいことがわかりません

このまま、PNGの圧縮やらフィルタやらまで続けていくつもりなのでしょう
（仮にIDATを連結したところで、glTexImage2Dには直接使用できないので）

[退会済みユーザー]

この人に小手先の解決法を提示したところで無意味なのは過去の質問を見ても明らか。（むしろクレクレ君状態になってるのを増長してるだけで害悪ですらある）
とりあえず行き当たりばったりで判らない事やエラーに遭遇するとまったく自分で調べず丸投げで質問するだけで、回答貰ってその場しのぎしてもその内容は理解しない（そもそも出来ない）から、また同じような質問を繰り返すだけ。

[episteme]

> unsigned char* gPictureData;
> void AddPictureData(const unsigned char* data) { ... }

ナニがしたんでしょうか?
gPictureDataの末尾にdataを連結したいんですか?
だとするとgPictureDataの末尾をどうやって得るんですか? (sizeof は使えません)

[退会済みユーザー]

そこがわからないのです。

[episteme]

"そこ”ってどこですか?

それと、gPictureDataの末尾にdataを連結するとして、gPictureDataに連結できるだけの領域が確保されている保証はどこにあるんですか?

[退会済みユーザー]

> gPictureDataの末尾をどうやって得る
です

[episteme]

もうひとつ、末尾位置を保持する変数を用意しておけばいいんじゃないですか?

[episteme]

で、どうやるかわからなくて苦戦するくらいなら std::string 使えばいいんじゃないんですか?

[退会済みユーザー]

そうですか。ちょっとそれますが.png ファイルは複数のiDATチャンクが存在する場合それらを足し合わせる必要があるのでしょうか？

[K_3578]

そもそもいっつも質問文変だし日本語の勉強からしたらいいよ。
これだけアドバイス貰っても理解出来ないんだから。

[episteme]

それはまた別の質問です。
足し合わせる必要があるかわからんのになんで足し合わせる質問するん?

[退会済みユーザー]

[XY問題]
https://ja.wikipedia.org/wiki/XY%E5%95%8F%E9%A1%8C

XY問題とは、ヘルプデスクなどで見られる「質問者が、本当に解決したい課題Xについて直接聞くのではなく、Yという二次的な課題を解決する方法を聞く」ことによって発生するコミュケーション上の問題を指す語である。 質問者はYによってXを解決することができると考えているが、しかし、Yを解決してもXは解決しないか、またはYは解決方法としては不十分であることが多い。本質的な課題を曖昧にしたり二次的問題を持ち込んだりすることは、回答者に不必要な苦労をさせたり、または不十分な解決方法が提示されることにつながる。

Answer 1

せっかくC++使うてるんなら std::string で楽してはいかがか。

#include <iostream>
#include <string>

int main() {
    using namespace std;

    /* apple と banana を つなぐ */
    string a = "apple";
    string b = "banana";
    string c = a + b;
    cout << "[" << a << "] + [" << b << "] = [" << c << "]\n";
}

[追記] char型でやってみたいとのことなので:

#include <iostream>
#include <algorithm>

int main() {
    using namespace std;

    char a[5] = { 3, 2, 1, 0, 1 };
    char b[6] = { 2, 3, 0, 0, 2, 1 };
    char c[11];
    /* a と b をつないで c へ */
    char* out = c;
    out = copy(a, a+5, out);
    out = copy(b, b+6, out);
    /* できたかな? */
    for ( char byte : c ) {
      cout << (int)byte << ' ';
    }
}

Comments

[退会済みユーザー]

そもそもバイナリの連結方法としては正しいのでしょうか？

[episteme]

なにか問題がありますか?

[退会済みユーザー]

いえ正しいのでしょか？そこが気になります。まぁやり方がなんだかめんどいというか無駄に複雑な気がするのですがそれはまぁそういうものなので仕方ないですが

[episteme]

どのコードに対する疑問/質問ですか?

[退会済みユーザー]

AddPictureData()関数です。

[episteme]

間違ってますね。

[退会済みユーザー]

そうですか。ではどういった方法があるのでしょうか？

[episteme]

回答のとおり、std::stringを使ってはいかがですか? + で繋がるんだから楽でしょ?

[退会済みユーザー]

確かにそうですが。char型でやってみたいのです。

[episteme]

どうぞやってみてください。memcpy/std::copyあたりで実現できるでしょう。

Answer 2

strcat_sだと難しいと感じたので、関数を自作しました。

#include <iostream>
#include "string.h"


unsigned char* gPictureData;

//サイズを追加
int gPictureDataSize;


char* bincat(char*s1, size_t s1_size, size_t s1_max_size, char*s2, size_t s2_size) {
	int i;
	for(i=0;i<s2_size;i++) {
		if((s1_size + i) > s1_max_size) {
			break;
		} else {
			s1[s1_size+i] = s2[i];
		}

	}
	return (&s1[s1_size+i]);
}
void AddPictureData(const unsigned char* data, int data_size)
{
    int size = gPictureDataSize + data_size;

    unsigned char* str = new unsigned char[size] {0x00};

    bincat((char*)str, 0, size, (char*)gPictureData, gPictureDataSize);
    for (int i = 0; i < size; i++)
    {
        printf("%x ", str[i]);

    }
	printf("\n");

    // ########################################################
    bincat((char*)str, 4, size, (char*)data, 4);
    // ########################################################


    for (int i = 0; i < size; i++)
    {
        printf("%x ", str[i]);

    }
	printf("\n");

	//deleteは移動

    //delete gPictureData;
    //gPictureData = nullptr;

    //gPictureData = new unsigned char[size]{0x00};
    //strcat_s((char*)gPictureData, sizeof(str), (char*)str);
    //gPictureData[size + 1] = '\0';
}

int main() 
{
    gPictureData = new unsigned char[5]{0x00,0x00,0x00,0x00};
	gPictureDataSize = 4;

    const unsigned char* st = new unsigned char[5]{ 0x01,0x01,0x01,0x01 };
	int st_size = 4;

    for (int i = 0; i < gPictureDataSize; i++)
    {
        printf("%x ", gPictureData[i]);

    }
	printf("\n");

    AddPictureData(st, st_size);

    for (int i = 0; i < gPictureDataSize; i++)
    {
        printf("%x ", gPictureData[i]);
    }

	printf("\n");


    delete gPictureData;
    gPictureData = nullptr;


    return 0;
}