学生のデータの並び替えと指定する学生を表示させたい。

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
卒業課題に躓いてしまったので、皆さんに教授していただきたく投稿させて頂きました。
問1. 学籍番号を入力して、個人の学籍番号、氏名、各科目点数、合計点、平均を表示。

問2. 昇順または降順にしたい列を選んで昇順にせよ。
私なりに調べましたが…qsortを使ったコードが上手く出力できないので、例を教えていただけないでしょうか。
列を指定するコードを入れるとごちゃごちゃになってしまったので、大まかな流れを書いていただけると幸いです。

よろしくお願いいたします。

該当のソースコード

#include<stdio.h>

int main(void)
{
FILE * fp;
int gakuseki = 0;
char name[100];
int kokugo, math, english, sum;
double ave;
double sumk, summ, sume;

if ((fp = fopen("class.dat", "r")) == NULL) /* オープン */
printf("\a\aファイル見つかりませんでした。\n");
else
{
printf("\aファイルを開きます。\n");
printf("\a\n---全生徒のデータを表示---\n");
printf("  学籍番号  |  氏  名  |  国 語  |  数 学  |  英 語  |  合計点  |  個人平均点\n");

while (fscanf(fp, "%d %s %d %d %d", &gakuseki, name, &kokugo, &math, &english) == 5)
{
int sump = kokugo + math + english;
printf("%11d | %-8s |  %4d   |  %4d   |  %4d   |   %4d   |  %lf\n", gakuseki, name, kokugo, math, english, sump, (double)sump / 3);
gakuseki++;
sumk += kokugo;
summ += math;
sume += english;

}

printf(" 各科目平均 |    -     |    %3.1f |    %3.1f |    %3.1f |    -     |      -\n\n", sumk / gakuseki, summ / gakuseki, sume / gakuseki);


fclose(fp); /* クローズ */
}

int seach;
printf("閲覧したい生徒の学籍番号を入力してください。: "); scanf("%d", &seach);
seach = strstr(gakuseki, seach);
printf("  学籍番号  |  氏  名  |  国 語  |  数 学  |  英 語  |  合計点  |  個人平均点\n");
printf("%11d | %-8s |  %4d   |  %4d   |  %4d   \n", gakuseki, name, kokugo, math, english);

return 0;
}

コンパイル結果

閲覧したい生徒の学籍番号を入力しても、何も表示されませんでした。
エラーは出ないのに考えても分かりませんでした。
どなたかご教授よろしくお願いいたします。

Comments

[tatsu99]

class.datの内容はどうなっていますか？

[episteme]

あと未解決の質問に決着をつけなさい
https://teratail.com/users/Bulls#question

[BeatStar]

せめて『インデント』ぐらいは入れましょうよ…
Tabキーや半角スペースでやるアレです。

今のままでは読む気失せます。

特にfor文内にfor文があるとかのような入れ子の場合、相当読みづらい。

質問のコードもif文内にwhile文がある。
これでは面倒すぎる。

今の状態は、『ひらがなだけで表現し、かつ句読点もないし、顔文字もないような長文で、その長文が改行なしで表現されている感じ』です。

読む気失せます。

まずはインデントぐらいは入れましょう。

[episteme]

卒業課題を丸投げとは大したもんだ。

[BeatStar]

仕様が曖昧。
情報伏せてわかるわけないでしょ。

[fana]

> 卒業課題

そういうものはさすがに自力でやらなきゃでしょ…
（仮にここで答えをもらえたとして，提出先にここが発見されたら卒業できなくなる可能性…とか考えないのか？）

[Bulls]

tatsu99さん
class.datの内容を記載します。
1 Babe 44 81 63
2 Ruth 60 49 77
3 Lou 11 62 32
4 Gehrig 98 40 8
5 Joe 13 28 33
6 DiMaggio 56 60 63
7 Ian 78 69 73
8 Thorpe 38 48 0
9 Michael 5 88 96
10 Klim 61 4 36
11 Grant 41 54 10
12 Hackett 47 70 90
13 Kieren 79 26 62
14 Perkins 37 11 82
15 Mark 19 31 23
16 McGwire 55 52 87
17 Sammy 21 93 63
18 Sosa 38 13 79
19 Jim 48 45 41
20 Abott 77 61 67
21 Jordan 94 50 19
22 Scottie 7 87 14
23 Pippen 1 40 24
24 Dennis 20 9 86
25 Rodman 5 32 14
26 Pedro 85 95 25
27 Martinez 67 84 46
28 Yurisbel 75 34 29
29 Gracial 18 93 4
30 Moinelo 28 82 75

宜しくお願い致します。

[bboydaisuke]

質問の中に含めるべきものを「質問の追記依頼」の所に書く質問者のセンスがすごいと思いました。

[bboydaisuke]

「卒業課題」ってことは学校に行ってるんだから、先生とか学生の仲間に聞けばいいのに。これから卒業するってことは何年も行っているんだから、一人も知っている人がいないってことはないでしょう。

[dodox86]

なぜこう、できていない・理解できていない複数のことを同時に片付けようとするのか。ひとつずつ理解して順番に片付けましょうよ。

[WoodenHamlet]

卒業するべきでない人間は卒業するべきではない（トートロジー）

Answer 1

問1. 学籍番号を入力して、個人の学籍番号、氏名、各科目点数、合計点、平均を表示。
問2. 昇順または降順にしたい列を選んで昇順にせよ。(解釈不能だが原文ママ)

これを実現するには何らかの**"表(table)”が必要だが、コード中にそれが見当たらない**。
それが卒業課題に躓いてしまった主因(と断言する)。

Answer 2

恐らく問1を解き始める段階にも到達していないのではないでしょうか
ファイルを読んで中身を表示しようとしているだろう部分ですでに崩壊の兆しが見受けられます。
コンパイルもしてない時点での所見ですが、

• sumk, summ, sume が初期化されていないため、表示される各科目平均が多分でたらめな値になる
• CSVの中身がわからないので何とも言えないが、平均を出すときの計算が「（上記のでたらめな）合算値／最後に読んだ人間の学籍番号＋１」になるので、でたらめな値になる。点数リストが学籍番号順で並んでいて、かつ0始まりなら大丈夫なのか？
• ファイルが読めなかった時でも問１の学籍番号を訪ねる処理に入ってしまう

Answer 3

「必ずqsortを使え」という隠された条件が存在するのでもないならば，
わざわざよくわからんqsortを使う義理も無ければ義務もないのではあるまいか．
qsortがわからんなら他の方法を採れば良い．

e.g. バケットソート：

問2. 昇順または降順にしたい列を選んで昇順にせよ。

という話において「選べる列の個数」がn個であれば，n次元ボクセル空間に各生徒をマッピングすればよい．
これで実質「ソート」の作業は完了だ．
あとは選ばれた列に従ってボクセル空間を走査し，生徒を拾い集めて表示するだけだ．

Answer 4

一番の問題点は、読み込んだ生徒の数分のデータを格納する変数を定義していないことです。
int kokugo, math, english, sum;　の定義はでは、一人分のデータだけしか格納できません。
昇順、又は降順に並べ替えるためには、全員分のデータを確保できるように変数を配列で定義する必要があります。
以下のようにしてください。
提示したソースでは、①氏名を昇順に並べ替え、②国語の点数で昇順に並べ替えを行っています。
他の科目での並び替え、及び、降順での並び替えは、提示されたソースを元にあなたの方でトライしてください。
トライした結果、うまくいかなければ、再度質問してください。
尚、100人分まで格納できますが、それを超えた場合のチェックは行っていません。必要であれば、あなたが修正してください。

#include<string.h>
#include<stdio.h>
#include<stdlib.h>
struct person {
    int     gakuseki;   //学籍番号
    char    name[100];  //氏名
    int     kokugo;     //国語
    int     math;       //数学
    int     english;    //英語
    int     sum;        //合計
};
int comp_name(const void *e1,const void *e2);
int comp_kokugo(const void *e1,const void *e2);


int main(void)
{
    FILE *fp;
    int gaku_count = 0; //生徒数
    int gakuseki;
    char name[100];
    int kokugo, math, english, sum;
    double ave;
    double sumk, summ, sume;
    struct  person  all[100];   //生徒100人分
    int i;
    if ((fp = fopen("class.dat", "r")) == NULL)     /* オープン */
        printf("\a\aファイル見つかりませんでした。\n");
    else {
        printf("\aファイルを開きます。\n");
        printf("\a\n---全生徒のデータを表示---\n");
        printf
            ("  学籍番号  |  氏  名  |  国 語  |  数 学  |  英 語  |  合計点  |  個人平均点\n");

        while (fscanf
               (fp, "%d %s %d %d %d", &gakuseki, name, &kokugo, &math,
                &english) == 5) {
            int sump = kokugo + math + english;
            printf
                ("%11d | %-8s |  %4d   |  %4d   |  %4d   |   %4d   |  %lf\n",
                 gakuseki, name, kokugo, math, english, sump,
                 (double) sump / 3);
            sumk += kokugo;
            summ += math;
            sume += english;
            all[gaku_count].gakuseki = gakuseki;
            strcpy(all[gaku_count].name,name);
            all[gaku_count].kokugo = kokugo;
            all[gaku_count].math = math;
            all[gaku_count].english = english;
            all[gaku_count].sum = sump;
            gaku_count++;
        }

        printf
            (" 各科目平均 |    -     |    %3.1f |    %3.1f |    %3.1f |    -     |      -\n\n",
             sumk / gakuseki, summ / gakuseki, sume / gakuseki);


        fclose(fp);                                 /* クローズ */
    }

    int seach;
    int fx = -1;
    printf("閲覧したい生徒の学籍番号を入力してください。: ");
    scanf("%d", &seach);
    for (i = 0; i < gaku_count; i++){
        if (seach == all[i].gakuseki){
            fx = i;
            break;
        }
    }
    if (fx ==  -1){
        printf("該当者なし\n");
    }else{
        printf
            ("  学籍番号  |  氏  名  |  国 語  |  数 学  |  英 語  |  合計点  |  個人平均点\n");
        printf("%11d | %-8s |  %4d   |  %4d   |  %4d   |  %4d    |  %lf   \n", all[fx].gakuseki, all[fx].name,
               all[fx].kokugo, all[fx].math, all[fx].english, all[fx].sum, (double)all[fx].sum / 3.0);
    }

    int column;
    printf("昇順にしたい列を入力してください。: ");
    scanf("%d", &column);
    switch (column){
        case 1: //学籍番号
            break;
        case 2: //氏名
            qsort(all,gaku_count,sizeof(struct person),comp_name);
            break;
        case 3: //国語
            qsort(all,gaku_count,sizeof(struct person),comp_kokugo);
            break;
        //以下省略
        default:
            break;
    }
    printf
        ("  学籍番号  |  氏  名  |  国 語  |  数 学  |  英 語  |  合計点  |  個人平均点\n");
    for (i = 0; i < gaku_count;i++){
        printf("%11d | %-8s |  %4d   |  %4d   |  %4d   |  %4d    |  %lf   \n", all[i].gakuseki, all[i].name,
               all[i].kokugo, all[i].math, all[i].english, all[i].sum, (double)all[i].sum / 3.0);
    }
    return 0;
}
int comp_name(const void *e1,const void *e2)
{
    struct person *p1 = e1;
    struct person *p2 = e2;
    int ret;
    ret = strcmp(p1->name,p2->name);
    return ret;
}
int comp_kokugo(const void *e1,const void *e2)
{
    struct person *p1 = e1;
    struct person *p2 = e2;
    int ret;
    ret = p1->kokugo - p2->kokugo;
    return ret;
}

Answer 5

qsortを使ったコードが上手く出力できない

ということなので、qsortの使い方のサンプルだけ提示します。
こちらを参考に、各データを構造体にして、それを並べ替えるようにしました。

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

// 並べ替え対象の構造体
struct score_data
{
    int id;
    char name[100];
    int score;
};

// scoreを比較する関数
int compare_score_asc(const void *a, const void *b)
{
    return ((struct score_data *)a)->score - ((struct score_data *)b)->score;
}

// データの出力
void print_datas(struct score_data *datas, int length)
{
    int loop;
    
    for (loop = 0; loop < length; loop++)
    {
        printf("%11d | %-8s |  %4d   \n",
            datas[loop].id,
            datas[loop].name,
            datas[loop].score);
    }
    printf("----------\n");
}

int main(void){
    struct score_data datas[3];
    
    // データの入力
    datas[0].id = 0;
    strcpy(datas[0].name, "sato");
    datas[0].score = 50;
    
    datas[1].id = 1;
    strcpy(datas[1].name, "goto");
    datas[1].score = 100;
    
    datas[2].id = 2;
    strcpy(datas[2].name, "takeda");
    datas[2].score = 0;
    
    // 最初の状態を出力
    print_datas(datas, 3);
    
    // 並べ替え
    qsort(datas, 3, sizeof(struct score_data), compare_score_asc);
    
    // 出力
    print_datas(datas, 3);

    return 0;
}

なお、コードにはあえて記載していませんが、文字列同士の比較はstrcmp()を使うのが楽です。

Comments

[fiveHundred]

っと回答したけれども、問1.の表示すら出来ていない状況なのか…。
これだとqsortの使い方以前の問題ですね。

やり方としては、どこかに全てのデータを保持しておき、入力されたらそれらのデータの中から同じ学籍番号のものを出力すればいいと思います。