文字列を2進数に変換してファイル出力したい

Question

以下のfiledata[８]に格納してある文字列を２進数に変換して出力したいです。
条件として、１バイト区切りにカンマを格納したい。

イメージとしては、以下の様に出力したい
↓ファイル.txtの中身(左からABCDEF)
/////////////////////////////////////////////////////////////////////
01000001,01000010,01000011,01000100,01000101,01000110, 01000111
/////////////////////////////////////////////////////////////////////

int main()
{
	FILE *pFile;						// ビット出力用ファイルポインタ
	char filedata[8] = { "ABCDEFG" };	// 終端はNULL
	char str[71];						// ビット格納用配列 8*8ビット+1バイト区切りで【,】を格納
	int rcount = 71;					// 8バイトカウント
	// 出力用配列初期化
	memset( str, 0x00, sizeof( str ) );
	// ファイルオープン
	pFile = fopen( "./bitFile.txt", "w" );

	for ( int count = 0; count < 8; count++ )
	{
		int beforeBit = filedata[count] - '0';	// 該当番地の文字を数値として抜き出す
		int bit       = 1;						// マスクビット
		int bitcount  = 8;						// 8ビットカウント
		while (1)
		{
			// 対象が0の場合は確認完了
			if (( beforeBit == 0 ) && ( bitcount <= 0 ) )
			{
				break;
			}
			// 該当ビットを抜き出す。
			str[rcount] = beforeBit & bit;
			// 次のビットへ移動
			beforeBit = beforeBit >> 1;
			rcount--;
			bitcount--;
		}
		// 8バイト区切り文字を格納
		str[rcount] = ',';
		rcount--;
	}

	// ファイル出力
	fprintf( pFile, "%s\n", str );
}

上記処理の場合、
fprintf( pFile, "%s\n", &str );を実行後に
↓の様な例外が発生してしまいます。
Run-Time Check Failure #2 - Stack around the variable 'str' was corrupted.
strの領域問題と考えstr[256]の様に大きめに確保しても同様のエラーが出ております。
どうしたら正常にファイル出力が行えるでしょうか？
また、もっと簡単に文字列→２進数に変換してファイルへ出力する方法がございましたら教えていただきたいです。

Comments

[kazuma-s]

'F' が 00001111 なら 'G' は 00010000 ですか？

[tyapapa]

質問の内容を変更しておりますが、
ASCIIコードに従って出力したいのでG(10進数の71)の場合は、1000111になります

Answer 1

#include <stdio.h>

int main(void)
{
    char filedata[8] = "ABCDEFG";
    char str[72] = "";
	int k = 0;
    for (int i = 0; i < 8 && filedata[i]; i++) {
		for (int j = 8; --j >= 0; ) str[k++] = filedata[i] >> j & 1 | '0';
		str[k++] = ',';
    }
	str[k-1] = '\0';
    FILE *fp = fopen("./bitFile.txt", "w");
	if (!fp) return 1;
    fprintf(fp, "%s\n", str);
	fclose(fp);
}

参考になりますか？
理解できますか？

Comments

[tyapapa]

理解できました。ソース作り直しありがとうございます！

Answer 2

間違いが多くて修正が困難です。例を置いていきます。

#define N 6
int main()
{
    char filedata[N+1] = "ABCDEF";
    for (int i=0; i<N; i++) {
        int num = filedata[i]-'A'+10;
        for (int j=0; j<8; j++) {
            printf("%d", (num>>(8-j-1))%2);
        }
        if (i+1 < N) {
            printf(",");
        }
    }
}

Answer 3

文字列、というのは終端に’￥０’をおいた文字配列です
str はそうなってませんね.

って、そもそもstrには文字データがはいってませんね。
’０’の文字ってのはどういうデータかわかるでしょうか

Comments

[tyapapa]

’０’は文字の１０進数でいうところの４８にあたる認識です
文字 - '0'で数字に変換してしまっているのでstrには文字が入っていないって意味ですかね。

そして、終端が分かるようにするためにも、str[71] = '\0'も必要になるってことですか

[y_waiwai]

// 該当ビットを抜き出す。
str[rcount] = beforeBit & bit;

その１０進数の４８は入ってませんね
strには、４８と４９，最後の0、しか入ってはいけません、が、そうなってはいません