C言語：乱数の式の場所について

Question

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#define N 1000
int main(){
        int i,sum,t,max;
        float average;
        int a[N];
        printf("input number of students in Class=");
        scanf("%d",&t);
        max=a[0];
        for(i=0;i<t;i++){
                printf("input point of Student%d=",i+1);
                scanf("%d",&a[i]);
        }
        for(i=0;i<t;i++){
        if(max<a[i]){
        max=a[i];
        }
        }
        printf("maximum point=%d\n",max);
        sum=0;
        for(i=0;i<t;i++){
                sum=sum+a[i];
        }
      average=(float)sum/(float)t;
        printf("average point=%f\n",average);
        return 0;
}
問題点：乱数を発生させてa[i]=rand()%Nの式を入れようとしたが、どこに式を入れても
　　　　上手くいかなかった。
やってみたこと：とにかくいろいろな位置にa[i]=rand()%N_1を入れてみた

Comments

[archiver]

上手くいかなかったとは、どうなっているのでしょうか？期待する動作と現状発生している現象を明記してください。

[r0_.-]

乱数を入れているのに、自分で値を入力して平均・最高点数を求めるようになってしまうんです。

[dodox86]

Cygwin上で実行しているのでしょうが、問題はCygwinに関することではないので、「Cygwin」タグは必要ないです。

[r0_.-]

わかりました

[r0_.-]

今度から気をつけます。

[episteme]

> 乱数を入れているのに、自分で値を入力して平均・最高点数を求めるようになってしまうんです。

scanf("%d",&a[i]); を a[i]=rand()%N に差し替えてもそうなるんですか?

[r0_.-]

はい

[episteme]

修正の後コンパイルしてないとか?

[r0_.-]

しました

[episteme]

scanf("%d",&a[i]); を a[i]=rand()%N に差し替えたのなら、
コード中に scanf("%d",&a[i]); は存在しないのだから
「自分で値を入力して平均・最高点数を求めるようになってしまう」わけがないのだが。

[r0_.-]

すみません。乱数にはなっていました。正しい値は出ませんでした。

[episteme]

「自分で値を入力して平均・最高点数を求めるようになってしまう」ではなかったのか...

Answer 1

• max=a[0];は、a[0]に値を設定してから実行しないと、(どんな値が入っているかわからない)a[0]の初期値をmaxに設定するため、偶然大きな値が入っていると想定外の結果になる。
• おそらく、「一人一人の点数をキー入力している部分」の代わりに、疑似乱数で点数を設定したいのであろうから、疑似乱数の式を書く場所は最初のforループの中。
• 疑似乱数のシードをそれより前に一度実行する。わざわざtime.hをincludeしているから、time(NULL)を使うつもりだろう。
• しかし、Nは最大人数の値であって、最大点数ではないので% Nで「0以上N未満の疑似乱数」にするのはおかしいだろう。最大点数を示す定数か何かを追加すべきだ。
• maxとsumの計算は並行して行なえる。
• こうした計算にfloatをわざわざ使う必要はないので、doubleを使うべき。

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#define MAX_STUDENTS 1000
#define MAX_POINT 100

int main(void)
{
    srand(time(NULL));

    int total;
    printf("input number of students in Class=");
    scanf("%d", &total);
    if (total > MAX_STUDENTS) {
        printf("%d exceeds maximum number %d\n", total, MAX_STUDENTS);
        return 1;
    }

    int a[MAX_STUDENTS];
    for (int i = 0; i < total; i++) {
        /*
        printf("input point of Student%d=", i + 1);
        scanf("%d", &a[i]);
        */
        a[i] = rand() % (MAX_POINT + 1);
        printf("point of Student%d=%d\n", i + 1, a[i]);
    }

    int max = a[0];
    int sum = 0;
    for (int i = 0; i < total; i++) {
        sum += a[i];
        if (max < a[i]) {
            max = a[i];
        }
    }

    double average = (double)sum / total;
    printf("maximum point=%d\n", max);
    printf("average point=%.2lf\n", average);

    return 0;
}

Comments

[r0_.-]

ありがとうございました。