Q&A
whileループの部分、どういう意図で書いているのか、説明してみてください。説明しているうちに、間違いに気づくかと思います。
この課題が分かりません。
「テキストファイルscore.txtには、以下のように、ゲームの得点と、プレイヤーの名前の一覧が記述されている。このファイルを読み込み、プレイヤー名とスコアを表示し、中で最高得点を取得したプレイヤーの名前と点数を表示しなさい。
<score.txtの内容> James,1000 Tom,1200 Steve,2800 Bob,900
<実行結果> プレイヤー 得点 -------------------- James 1000点 Tom 1200点 Steve 2800点 Bob 900点 -------------------- 最高得点：Steve(2800点) ``` 」(問題文ここまで) 以下のようなコードを書いてみました。 ```#include<stdio.h> #include<stdlib.h> #include<string.h> #pragma warning(disable:4996) #define size 2048 void main(){ FILE* file; char line[size]; char str1[size], str2[size], str3[size], str4[size]; int n1, n2, n3, n4; int max; line[0] = '\0'; file = fopen("c:\\test\\score.txt","r"); if (file == NULL){ printf("ファイルが開けません\n"); exit(1); } printf("プレイヤー得点\n--------------\n"); while (fgets(line, size, file) != NULL){ sscanf(line,"%[^,],%d,%[^\r\n,],%d,%[^\r\n,],%d,%[^\r\n,],%d",str1,&n1,str2,&n2,str3,&n3,str4,&n4); printf("%s %d点\n%s %d点\n%s %d点\n%s %d点\n", str1, n1, str2, n2, str3, n3, str4, n4); } max = n1; if (n2 > max){ max = n2; } if (n3 > max){ max = n3; } if (n4 > max){ max = n4; } printf("--------------\n最高得点:"); if (n1 == max){ printf("%s(%d点)", str1, n1); } if (n2 == max){ printf("%s(%d点)", str2, n2); } if (n3 == max){ printf("%s(%d点)", str3, n3); } if (n4 == max){ printf("%s(%d点)", str4, n4); } fclose(file); getchar(); }
score.txtの内容が「James,1000,Tom,1200,Steve,2800,Bob,900」ならば望み通りの実行結果が得られましたが、問題文の通りのscore.txtの内容だと上手く行きません。
fgets()は文字列読み込みの際に\nがあればそこで読み込みを止めるというのは知っているのですが、sscanf()でコンマを飛ばしつつ改行飛び越して4行の文字列と数字を読み込む方法が分かりません。fgetどうか教えて下されば幸いです。
＜追伸＞
fgets()がファイルの最後にNULLが出てくるまでファイル全て読み込ませる意図でwhile文を書きました。ただfgets()は1行ごとにしか読み込めず、1行目読み込んだ後lineに1行目の文字列が格納されるが、それで2行目読み込んだ後lineに1行目の文字列が消された上で2行目の文字列が格納されるのだと思います(ここら辺の細かい仕組みについてはまだあまり理解できていません)。4行目まで読み込んだ後は4行目の文字列しか格納されてないのでしょう(実行時、最高得点の所は表示されたが「最高得点:Bob (900点)」と表示されていた)。fgets()に改行を読み込ませないか、あるいはfgetc()で読み込むという手がありますが、sscanf()とどう結び付ければよいのかが分かりません。
質問書式の体をなしておらずすいませんでした。確かに読む気を起こさせるようなものでは無かったと思っております。これを機会にマナーを守って質問していきます。
あなたは自分の質問のソースコードを読む気が起きますか？私は起きません。https://teratail.com/questions/41441
WoodenHamletさんが指摘されたからか、ようやくソースが。でもotnさんの意味はまだ通じていないですね。一言、一行で読もうとしていながら、ループの意味？それが改行と関係あるのだが。。。
4行目のファイル終端に行ってNULLになるまで、行ごとに\nまで読むのを繰り返すということです。文字列配列の数がline1個だけなら、whileで4回ループしても4行分入らない.....?
そうですね。whileループの中では1行毎に処理することになります。
’score.txt’の要素を分解すると’名前’+','+'スコア’＋改行ですね。（出血大サービスなコメント）貴殿の’ sscanf(line,"%[^,],%d,%[^\r\n,],%d,%[^\r\n,],%d,%[^\r\n,],%d",str1,&n1,str2,&n2,str3,&n3,str4,&n4);’のどこに、そのロジックが？
%[^,]がカンマのみ読み込ませずに1番最初の文字列(str1)を読み取って、%dがスコア(n1)を読み取ります(ここまで1行分)。%[^\r\n,]がカンマと改行を読み込ませずに文字列(str2)を読み取って、また%dがスコア(n2)を読み取って(ここまで2行目)というように4行目までこの操作を繰り返すということです。
すなおに１行毎の処理をすれば？これがotnさんの’while'の話です。
また、回答されたmajiponiさんのソースにも。
もしデータが100個の場合はどうしますか？
whileを100個並べるわけにもいかないので、配列を使うだろうとは想像できます。
otnさんが指摘したのは、'while'って繰り返し。これが理解できるか？です。つまり1行を100回繰り返せば私の言った回答に。頑張って欲しいからのコメントです。
ご丁寧なご指導本当にありがとうございました。
回答1件
0
ベストアンサー
今のプログラムを要約すると、
ファイルを開く
最終行まで読み飛ばす
最終行の内容から4人のデータを解析する
最終行に従って得点表示
最高得点者表示
ファイルを閉じる
となってます。
問題点は、「データを読み飛ばしている」(ヒント: whileの条件式の()と、ブロック{}の間にある謎の記号)ことと、「すべての行を一度に処理しようとしていること」です。ロジックをこう変えてみてはどうですか?
C
1char 最高得点者名[大きさ]; 2int 最高得点 = 0; 3while(fgets(行, 大きさ, file)) 4{ 5 sscanf(行,"%[^,],%d", 名前, &得点); 6 printf("%s %d点"); 7 これまでで最高得点者だったら 8 最高得点者名と最高得点を更新する。 9} 10最高得点者の情報を表示する。
投稿2016/11/11 07:32
編集2016/11/11 07:39
総合スコア1720
```
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#pragma warning(disable:4996)
#define size 2048
void main(){
FILE* file;
int i = 0;
char line[size][4];
char str[size][4];
int n[4];
int max=0;
char saidai[size];
line[0][i] = '\0';
file = fopen("c:\\test\\score.txt","r");
if (file == NULL){
printf("ファイルが開けません\n");
exit(1);
}
printf("プレイヤー得点\n--------------\n");
while (fgets(line[i], size, file) != NULL){
sscanf(line[i],"%[^,],%d",str[i],&n[i]);
printf("%s %d点\n", str[i], n[i]);
if (max < n[i]){
max = n[i];
strcpy(saidai, str[i]);
}
i++;
}
printf("--------------\n最高得点:");
printf("%s (%d点)", saidai, max);
fclose(file);
getchar();
}
```
と書いたら望み通りの結果が得られました。皆様、懇切丁寧に質問に付き合って頂き本当にありがとうございました。お陰様で非常に基本的な所でもまだよくわかってないことがあるとわかったので精進していきたいです。
良かったです。
あなたの回答
tips
プレビュー
