C言語のファイルread, writeについて

Question

visual studio2017 でcのプログラムを作成しています。
test.txtが１行データなので、65byteずつに改行したtext_new.txtを作ろうと思っています。

	fp = fopen(fname, "r");
	fp2 = fopen(fname2, "w");

	while (fread(str, 1, 50, fp) != NULL) {
		//表示
		printf("%s\n", str);
		fprintf(fp2, "%s\n", str);
	}
	fclose(fp);
	fclose(fp2);

Debug Assertion Failed!エラーが出ます。
fread.cppのLine:48
がダメとなっているのですが、回避法を教えてください。
よろしくお願いします。

Comments

[mattn]

コードブロックを「必ず」使って下さい。今からでも修正可能です。あと48行目がどの行か分かりません。

[mattn]

コードブロックにするには、編集ボタンを押してソースコードの始めと終わりにそれぞれ ``` だけの行を追加します。間違いやすいので ``` はコピーして使って下さい。

[cateye]

test.txtが文字データのは分かりますが、文字コードは何でしょう？

[mattn]

コードを全部載せて下さい。str が容量分宣言されていない、fname が "test.txt" でない、など落ちる可能性がいくらでもあります。

Answer 1

ソースコードを全部載せて頂けないので、落ちないコードを書いておきます。

int n;

fp = fopen(fname, "r");
if (fp == NULL) {
  perror("fopen(fname)");
  exit(1);
}
fp2 = fopen(fname2, "w");
if (fp2 == NULL) {
  perror("fopen(fname2)");
  exit(1);
}

while ((n = fread(str, 1, sizeof(str)-1, fp)) > 0) {
  str[n] = 0;
  //表示
  printf("%s\n", str);
  fprintf(fp2, "%s\n", str);
}

fclose(fp);
fclose(fp2);

Answer 2

freadをするならバイナリモードでfopenしましょう。fprintfならばテキストモードです。

このほかはプログラムの全体像、ファイルのデータフォーマットがわからないので確定的なことは言えません。とりあえず、配列の長さが足りない可能性があります。ちなみにエラーメッセージでは行番号を教えてくれているので、その行がヒントになります。

Answer 3

こんにちは。

strの型宣言がないので良く分かりませんが、char str[50];と宣言していると仮定して回答します。
また文字はASCII文字と仮定します。（漢字等のワイド文字を含まないという意味です。）

①1度に65文字読み込んでそれを改行して出力するのであれば、char str[65];は最低必要と思います。
②printf("%s\n", str);で出力するのであれば、'\0'で終端する必要があります。
readで「1バイト単位」で最大65バイト読み込み、読み込めた文字列の最後に'\0'を書きましょう。
この場合、char str[66];とし、65文字読み込めた時に'\0'を設定するための1バイトを確保しましょう。

後、mattnさんのアドバイスにも対応するとより正確な回答が返ってきやすくなりますよ。

Answer 4

fread(str, 1, 50, fp)は50バイトデータを読み込みますが、終端文字'\0'は付きません。なので、次のprintf("%s\n", str);でこけるのでは？
「追記」
strの領域がいくらあるかりませんが、最低51バイトとって最後（str[50]）に'\0'を入れましょう。