ファイルに16個ごとに改行を書き込みたい。C言語。

Question

ファイルのバイナリを読み込んで、そのバイナリをファイルに16個書き込んだら改行を書き込みたいのですが、うまい方法が見つかりません。

C言語でvisual stdioです。
分かる方いましたら、よろしくお願いします！

#include <string.h>
#include <stdio.h>
#include <stdlib.h>
void BinaryMake(FILE *ifp, FILE *ofp)
{
	fseek(ifp, 0L, SEEK_END);
	long fsize = ftell(ifp);
	fseek(ifp, 0L, SEEK_SET);

	unsigned char *BinaryText = malloc(fsize);

	fread(BinaryText, sizeof(*BinaryText), fsize, ifp);

	int i = 0;
	fprintf(ofp, "Binary Out.\n\n");

	// Make BinOut.
	fprintf(ofp, "unsigned char BinOut[] = {\n");
	for (i = 0; i < fsize; i++) {
		fprintf(ofp, "0x%X, ", BinaryText[i]);
		// 16の倍数で改行をファイルに書き込みたい。
		if (i != 0) {
			if (i > 16) {
				if (i % 16 == 0) {
					fprintf(ofp, "\n");
				}
			} else if (i % 15 == 0) {
				fprintf(ofp, "\n");
			}
		}

	}
	fprintf(ofp, "\n};\n\n");

	// BinOutLen.
	fprintf(ofp, "unsigned int BinOutLen = ");
	fprintf(ofp, "%d", i);
	fprintf(ofp, ";\n");

	free(BinaryText);
}

int main(int argc, char *argv[])
{
	FILE *fIN;
	FILE *fOUT;
	errno_t error;
	char buf[255 + 1];

	if ((error = fopen_s(&fIN, "C://Projects/test_curl/Debug/StinkyShitInDaMazafaka.txt", "rb")) != 0) {
		strerror_s(buf, 255, error);
		printf("fopen_s error one \n%s\n", buf);
		return 0;
	}
	if ((error = fopen_s(&fOUT, "C://Projects/test_curl/Debug/MazafakaInDaBinSource.txt", "w+")) != 0) {
		strerror_s(buf, 255, error);
		printf("fopen_s error two \n%s\n", buf);
		return 0;
	}
	BinaryMake(fIN, fOUT);
	fclose(fIN);
	fclose(fOUT);

	return 0;
}

上のソースでファイルに書き込んだ内容の一部です。
２行目が17個目で改行が書き込まれています。
よろしくお願いします！

unsigned char BinOut[] = {
0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F,
0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F,
0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F,
0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F,
0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F, 0x2F,

Answer 1

こんな手もある。

    // Make BinOut.
    fprintf(ofp, "unsigned char BinOut[] = {");

    for (i = 0; i < fsize; i++) {
        if (i % 16 == 0) {
            fprintf(ofp, "\n");
        }
        fprintf(ofp, "0x%X, ", BinaryText[i]);
    }

    fprintf(ofp, "\n};\n\n");

追記：誰かが「余計なカンマは出さない」と言うので

    // Make BinOut.
    fprintf(ofp, "unsigned char BinOut[] = {");
    char[2] delim = "";
    for (i = 0; i < fsize; i++) {
        fprintf(ofp, delim);
        if (i % 16 == 0) {
            fprintf(ofp, "\n");
        }
        fprintf(ofp, "0x%X", BinaryText[i]);
        strcopy(delim, ", ");
    }
    fprintf(ofp, "\n};\n\n");

以上、行頭判断形式。

Comments

[tkturbo]

strcopyのオーバーヘッドがわからんけど「無駄な出力しない」版も追記してみました。

Answer 2

余分なスペースとか改行は許しません。

fprintf(ofp, "unsigned char BinOut[] = {\n");
for (i = 0; i < fsize; i++)
{
    //16個目と最後は改行
    fprintf(ofp, "0x%02X,%c", BinaryText[i], (i%16==15||i==fsize-1)?'\n':'\0');
}
fprintf(ofp, "};\n\n");

Comments

[AnywhereGorilla]

僕がやりたかったことができてます！！！回答ありがとうございました！勉強します！

Answer 3

あまり難しく考えずに「16個毎に改行する」というのをそのままコードにすれば良いです。

ただ、単純に16個出力したら改行すると、データの数がちょうど16の倍数の時に空行が出力されるので、見栄えを考えてこんな感じでしょうでしょうか。

fprintf(ofp, "unsigned char BinOut[] = {\n");
for (i = 0; i < fsize; i++) {
    fprintf(ofp, "0x%X, ", BinaryText[i]);
	if(i % 16 == 15 || i == fsize - 1)	// 16個出力するか最後のデータを出力したら改行
        fprintf(ofp, "\n");
}
fprintf(ofp, "};\n\n");		// 先頭の改行は不要

あるいは、改行の位置を工夫するとif文が簡単になります。

fprintf(ofp, "unsigned char BinOut[] = {");	// 最初の行では改行しない
for (i = 0; i < fsize; i++) {
	if(i % 16 == 0)			// 改行してから16個並べる感じ
        fprintf(ofp, "\n");
    fprintf(ofp, "0x%X, ", BinaryText[i]);
}
fprintf(ofp, "\n};\n\n");

Comments

[catsforepaw]

2番目はすでに tkturbo さんが回答したのと同じでしたね。

Answer 4

この辺り、「16個かいたら1回改行」に対して条件判定が多すぎます。

    // Make BinOut.
    fprintf(ofp, "unsigned char BinOut[] = {\n");
    for (i = 0; i < fsize; i++) {
        fprintf(ofp, "0x%X, ", BinaryText[i]);
        // 16の倍数で改行をファイルに書き込みたい。
        if (i != 0) {
            if (i > 16) {
                if (i % 16 == 0) {
                    fprintf(ofp, "\n");
                }
            } else if (i % 15 == 0) {
                fprintf(ofp, "\n");
            }
        }

    }
    fprintf(ofp, "\n};\n\n");

if (i % 16 == 0) 改行 は心情として正しいです。

if (i != 0) は、0 % 16 == 0 になってしまうのを防ぐためだと思いますが、
ここにガードを入れるよりも、最初からi==0 にならないようにする、つまりiを1から
始めるほうが効果的です。if文が１つ減ります。

if (i % 15 == 0)も、0から初めて16回書いたら改行、のために入っているのかな？と
思いますが、iが1から始まれば不要です。

ということで、こういうのはいかがでしょうか。

for (i = 1; i <= fsize; i++) {
    fprintf(ofp, "0x%X, ", BinaryText[i-1]);
    if (i % 16 == 0) {
        fprintf(ofp, "\n");
    }
}

テストはしていません。

Comments

[AnywhereGorilla]

気持ちを汲んだ回答ありがとうございました！！！

Answer 5

こんにちは。

テストしてませんが、たぶん下記でいけると思います。

// Make BinOut.
fprintf(ofp, "unsigned char BinOut[] = {\n");
for (i = 0; i < fsize; i++) {
    fprintf(ofp, "0x%X, ", BinaryText[i]);
    // 16の倍数で改行をファイルに書き込みたい。
    if ((i % 16) == 15)
    {
        fprintf(ofp, "\n");
    }
}
fprintf(ofp, "\n};\n\n");

---

【追記】
catsforepawさんの回答を見て気が付きました。,が最後についてしまいますね。
もし、生成されたファイルをコンパイルするようでしたら、コンパイラ(例えばVisual strudio)によっては適切にコンパイルしてくれないことがあります。

フラグが必要になることが難点ですが、このような','等を省略したい時、結構使えます。

// Make BinOut.
fprintf(ofp, "unsigned char BinOut[] = {\n");
bool first=true;
for (i = 0; i < fsize; i++) {
    if (first) {
        first=false;
    } else {
        // 16の倍数で改行をファイルに書き込みたい。
        if ((i % 16) == 0) {
            fprintf(ofp, "\n");
        }
        fprintf(ofp, ", ");
    }
    fprintf(ofp, "0x%X", BinaryText[i]);
}
fprintf(ofp, "\n};\n\n");

tkturbotさんとcatsforepawさんの回答を見ても感じるかと思いますが、「継続処理」は「本処理」の後で実行するのではなく、「本処理」の前で実行した方が「綺麗」に書けることが多いようです。
そして、最初だけは「継続処理」が不要になるので、それを運良くごまかせない場合は、フラグを使って分岐するのが良さそうです。

---

理想は、フラグを使わないでループ先頭の処理を分けることができると良いのです。
どなたか良い方法をご存知でしたら是非教えて下さい。

Comments

[AnywhereGorilla]

できました！ありがとうございました！！！

[Chironian]

あっと、追記が間に合いませんでした。
もし，','でハマったら参考にされて下さい。

[AnywhereGorilla]

分かりました！ありがとうございました！！！