200人のスコアの評価を求める

Question

200人分の評価(-1〜100点)が記載されたファイルがs.txtであるとします。そのときに、s.txtを読み込んで、集計結果を表示させたいです。
点が-1点の人は評価対象から外すとする場合に、その他0〜100点の人の人数をカウントcountし、その中で最大値max、最小値min、平均点averageを求めます。さらに、その平均点から20点以上下回っている評価対象の人数をfailureとしてカウントして求めたいです。

以下ソースコードを書いたのですが、平均点から20点以上下回っている評価対象の人数が0人になってしまいます。わかる方教えていただきたいです。(このとき、評価対象の平均点は50点で、s.txtの中には20点のスコアの人がいるので、0にはならないはずなのですが、0になってしまいます。)

#include <stdlib.h>

int main(void) {
    FILE *fp;
    fp = fopen("s.txt","r");
    if(fp == NULL) {
        printf("Open error\n");
        exit(1);
    }
    int i, score_data, score_sum = 0, count = 0, max = 0, min = 100, failure = 0;
    for(i = 0; i < 200; i++) {
        fscanf(fp, "%d\n", &score_data);
        if (score_data >= 0) {
            count++;
            score_sum += score_data;
            if (max < score_data) {
                max = score_data;
            }
            if (min > score_data) {
                min = score_data;
            }
        }
    }
    
    double average = (double)score_sum / count
    
    for(i = 0; i < 300; i++) {
        fscanf(fp, "%d\n", &score_data);
        if (score_data >= 0) {
            if(score_data <= (average - 20.0) {
            failure++;
            }
        }
    }
    
    fclose(fp);
    printf("Count: %d\n", count);
    printf("Average: %f\n", average);
    printf("Max: %d\n", max);
    printf("Min: %d\n", min);
    printf("Failure: %d\n", failure);
    return 0;
}
コード

Comments

[y_waiwai]

このままではコードが読みづらいので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

Answer 1

ファイルから読込み処理の結果として，点数の度数分布を作ればよいのではないでしょうか．

最大値max、最小値min、平均点averageを求めます。さらに、その平均点から20点以上下回っている評価対象の人数

これらはそこから求めればよいでしょう．

Answer 2

Cを勉強したいのはともかく、データプロセスが目的だとしたら、他のツールの方が楽だと思います。Pythonとか、Excelとか。Python のコードを置いておきます。

def average(l):
    return sum(l) / len(l)

with open("s.txt") as fp:
    scores = [int(line) for line in fp if not line.isspace() and int(line) != -1]
    print(f"Count: {len(scores)}")
    print(f"Average: {average(scores)}")
    print(f"Max: {max(scores)}")
    print(f"Min: {min(scores)}")
    print(f"Failure: {len([x for x in scores if x <= (average(scores) - 20)])}")

Cの勉強を応援したいから、ファイルを配列に読み込むコードも置いておきます。

#include <stdio.h>
#include <stdlib.h>

int main() {
  FILE *fp = fopen("s.txt", "r");
  if (fp == NULL) { printf("Open error\n"); return 1; }

  int score_arr[200];
  int count = 0;
  for (int i = 0; i < 200; i++) {
    int score = 0;
    fscanf(fp, "%d\n", &score);
    if (score >= 0) { score_arr[count] = score; count++; }
  }

  for (int i = 0; i < count; i++) {
    // データ処理
    printf("%d\n", score_arr[i]);
  }
}

Comments

[SHOMI]

>ファイルを配列に読み込むコード
-1を読み捨てたうえで不定値にしていますが…

[YufanLou]

ご指摘ありがとうございます。そうですね、本当の長さを変数でキープしなければいけません。編集し直しました。まぁ、ですからCがこんなデータ処理の問題に似合わないと思います。みんなにRustを薦めたい。

[SHOMI]

>score_arr[i] = score;
countで管理するなら
score_arr[count] = score;
では

[SHOMI]

fseekを習っていないとコメントされているのでC言語の課題でしょうかね？

[YufanLou]

ご訂正ありがとうございます！
Pythonを書いてからコメント読んで課題かもと気づいたから、Cコードを追加に書きました。もしかしたら配列もまだ習っていないと思って。

Answer 3

ひとつ目のループでfscanfを使って200人分のデータを読み終わった後、
ふたつ目のループで再度fscanfを使って読もうとしていますが、既にファイルを読み終わっているので読み取れません。
一度目のループでは配列に読み取っておきそれを使用するか、fseekで読み取り位置をファイル先頭に戻してください。

また、ふたつ目のループは300回になっています。

Comments

[pocpocpoc]

返信いただき、ありがとうございます。fseekというものを習っていないのですが、以下のようなソースコードではうまくいったのですが、おかしいところはありますでしょうか。また、簡単化できますでしょうか。見ていただきたいです。

#include <stdio.h>
#include <stdlib.h>

int main(void) {
FILE *fp;
fp = fopen("s.txt","r");
if(fp == NULL) {
printf("Open error\n");
exit(1);
}
int i, score_data, score_sum = 0, count = 0, max = 0, min = 100, failure = 0;
for(i = 0; i < 200; i++) {
fscanf(fp, "%d\n", &score_data);
if (score_data >= 0) {
count++;
score_sum += score_data;
if (max < score_data) {
max = score_data;
}
if (min > score_data) {
min = score_data;
}
}
}
fclose(fp);

double average = (double)((int)(10 * ((double)score_sum / count))) / 10;

fp = fopen("s.txt","r");
if(fp == NULL) {
printf("Open error\n");
exit(1);
}
for(i = 0; i < 200; i++) {
fscanf(fp, "%d\n", &score_data);
if ((score_data >= 0) && (score_data <= (average - 20.0))) {
failure++;
}
}

fclose(fp);
printf("Count: %d\n", count);
printf("Average: %f\n", average);
printf("Max: %d\n", max);
printf("Min: %d\n", min);
printf("Failure: %d\n", failure);
return 0;
}

[SHOMI]

>簡単化
回答にも書いていますが、読み取ったものを配列に格納しておき使用しましょう

Answer 4

気がついたのですが、一度ファイルを開いて読み込んだら、その後closeさせて、再度ファイルを開き読み込みをしなければいけないのでしょうか？

Comments

[episteme]

わざわざ訊くまでもない、そう思うならやってみれば?

[pocpocpoc]

上記のようにやってみて成功したのですが、他にいい方法はないのかと思ったため、聞いてしまいました。申し訳ないです。

[episteme]

二度読まずに済ませるため配列に読み込んでおけばいい。
二度目のファイル読み込みの代わりに、配列の各要素を使ってfailureを探す。