ファイルから読込んだ文字列が文字化け

Question

C言語で、タイピングゲームを作成しています。問題文が書かれたtxtファイルを読込んで表示させたいのですが表示させようとすると文字化けしてしまいます。文字化けしないようにするにはどのようにしたらよいでしょうか？以下に、ソースコード、txtファイルの中身、文字化けの結果をupしますので、ご教授よろしくお願いいたします。

int game(){
	int level, sent, random, loop;
	clock_t start, end;
	int SENT[7];
	char Sent[7][100];
	char a[50], a1[50];
	FILE *A;

	printf("Please select level\n");
	printf("       1.Easy\n");
	printf("       2.Normal\n");
	printf("       3.Difficult\n");
	scanf("%d", &level);

	srand((unsigned int)(time(NULL)));
	for(loop=0;loop<7;loop++){
		SENT[loop] = (rand()%7)+1+(level-1)*10; //問題文をランダムで選択(1~10の中から7つ)
		sprintf(Sent[loop],"Sent\%d.txt", SENT[loop]);
		//printf("%d\n", SENT[loop]);
	}

	A = fopen(Sent[0],"r");
	for(loop=0;loop<50;loop++){
		fscanf(A, "%s", &a[loop]);
	}

	printf("\n");
	sleep(1);
	printf("3 ");
	sleep(1);
	printf("2 ");
	sleep(1);
	printf("1 ");
	sleep(1);
	printf("Start!!\n");

	start = clock();

	for(loop=0;loop<50;loop++){
	        printf("%s", &a[loop]);
	}
	printf("\n");
	scanf("%s", a1);
	if (strcmp(a,a1) == 0){
		printf("GOOD!\n");
	}else{
		printf("MISS!\n");
	}

    end = clock();
}

7.txt

boal

Comments

[t_obara]

実行環境は何ですか？Windows？txtファイルの文字コードは何ですか？UTF8?

[hokusou7500]

Windowsで、MinGWを使用しています。txtファイルの文字コードはUTF8です。情報が足りず申し訳ありません。

Answer 1

こんにちは。

まず、下記の読み込みが間違っています。

    A = fopen(Sent[0],"r");
    for(loop=0;loop<50;loop++){
        fscanf(A, "%s", &a[loop]);
    }

「文字列(%s)」を50個読んでいますね。
そもそも文字列自体は１つしかないですし、しかもたった4文字です。完全に読みすぎです。
読み過ぎの部分はエラーが発生して読み込まれていないと思います。
ですので、最初の１回だけ機能し、最初の１行だけが読み込まれている筈です。

UTF-8エンコードし、かつ、BOM有りにあすると最初の3文字はBOMとなります。通常のテキスト・エディタでは表示されませんが、このように読み込むと「謎」の３文字が読まれます。それがboalの前に表示されている３文字です。
その後、１文字づつずらしながら、繰り返しa配列を「文字列」として出力しているので、頭から１文字づつ減った同じ文字列が繰り返しでています。
そして、BOM+boalがNULL終端した後は、未初期化のままですので「ゴミ」がはいっていて、それらがでています。

「文字（%cで処理する）」と「文字列（%sで処理する）」の相違をきちんと把握できれば修正できると思いますよ。

Comments

[hokusou7500]

丁寧に説明していただきありがとうございます。Chironian様のアドバイスのおかげでうまく動作させることができました！自分の勉強不足で恥ずかしい限りです・・・。ベストアンサーにさせていただきました。今後もよろしくお願いいたします。

Answer 2

    for(loop=0;loop<50;loop++){
        fscanf(A, "%s", &a[loop]);
    }

これはなにを意図してるんでしょうか。
まともに動くコードではないです

＃ツッコミどころは他にもあります

Comments

[hokusou7500]

txtファイルから文字列を読み込むために書いたコードです。一文字ずつ読み込んで配列に格納しようと思って書いたコードでしたが、それならば指定子は%c、ですかね・・・。もしよろしければ他のツッコミどころも教えていただきたいです。

[y_waiwai]

ファイルに一つの文字列があってそれを読むなら
fscanf(A, "%s", &a[0]);
だけでいけます。
ただし、ファイルの文字列のバイト数がaのサイズを超えると破綻します

同じようなツッコミどころは、
for(loop=0;loop<50;loop++){
printf("%s", &a[loop]);
}
とかですね

[hokusou7500]

ありがとうございます！修正し動作させることができました。アドバイスありがとうございました！！

Answer 3

実行しているコマンドプロンプトの文字コードと、
問題文のtxtファイルの文字コードが異なっているためです。

コマンドプロンプトの文字コードは、chcpというコマンドでわかります。
txtファイルの文字コードは、保存するテキストエディタの使い方を調べてください。

Comments

[hokusou7500]

回答ありがとうございます。コマンドプロンプトの文字コードを変更しtxtファイルの文字コード(UTF-8)と合わせたのですが、表示させる文字列が以下のようになってしまいました。これは、ファイルを開くときに問題が起きているのでしょうか？

freefreefreefreereeeee4a4a4aaasLsLsLsLLmsmsssaaa (txtファイルの中身：free）

[taka-saan]

文字コードは一致しましたが、今度はプログラムに原因があります。
y_waiwaiさんからの指摘にもあるように、a[]の使い方に誤りがありますね。
問題文が50行あるのであれば、ファイル名の生成のところと同様に、2次元配列にしないといけません。
下の方のprintfのところもです。

あと、変数名にはもう少し変数の内容にふさわしいものをつけるように
心がけるとよろしいかと思います。
こういう場で人見せるときのためや、後で自分でもわからなくなります。

[taka-saan]

問題文が1ファイルに50行あるのかと思った。低評価くやしいorz

[hokusou7500]

アドバイスありがとうございます！修正し動作させることができました。変数名を適当につけちゃうのは悪い癖なので直したいと思います・・・。質問してすぐの回答ありがとうございました！！