0-1の範囲で乱数を10,000個発生させ、0.1刻みのヒストグラムを作成したいのですが、度数が明らかにおかしいです。アドバイスをください。

Question

0-1の範囲で乱数を10,000個発生させ、0.1刻みのヒストグラムを作成

0-1の範囲で乱数を10,000個発生させ、0.1刻みのヒストグラムを作成したいのですが、度数が明らかにおかしいです。アドバイスをください。

発生している問題

0 (0番目の度数)
993 (1番目の度数)
1007 (2番目の度数)
998 (3番目の度数)
958 (4番目の度数)
1001 (5番目の度数)
1049 (6番目の度数)
989 (7番目の度数)
963 (8番目の度数)
2042 (9番目の度数)

「n番目の度数」とは、n-1/10<=x<n/10で発生した乱数の個数です。

該当のソースコード

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

int main(void)
{
  int ii, rnd, k, j;
  float x;

  int dosu[10]={0};
  float ransu[10000]={0};

  for(ii=0; ii<10000; ii++){
    k=ii+1;
    rnd=rand();
    x=(float)rnd/RAND_MAX;
    ransu[ii]=x;
    /*ii番目に発生させた乱数は、変数ransuのii番目の要素として格納した*/

    if(0<=ransu[ii]<0.1) dosu[0]++;
    else if(0.1<=ransu[ii]<0.2) dosu[1]++;
    else if(0.2<=ransu[ii]<0.3) dosu[2]++;
    else if(0.3<=ransu[ii]<0.4) dosu[3]++;
    else if(0.4<=ransu[ii]<0.5) dosu[4]++;
    else if(0.5<=ransu[ii]<0.6) dosu[5]++;
    else if(0.6<=ransu[ii]<0.7) dosu[6]++;
    else if(0.7<=ransu[ii]<0.8) dosu[7]++;
    else if(0.8<=ransu[ii]<0.9) dosu[8]++;
    else if(0.9<=ransu[ii]<=1.0) dosu[9]++;

    printf("%f  (%d番目の乱数)\n", ransu[ii], k);

  }
  for(j=0; j<10; j++){
    printf("%d (%d番目の度数)\n",dosu[j] ,j);
  }

  return 0;
}

試したこと

度数の合計が10000になるので、ちゃんと10,000個の乱数が発生しているようです。

Answer 1

0<=ransu[ii]<0.1

a <= x < b は a <= x && x < b と書くべし。

Answer 2

間違えているところは、他の回答の通りです。
数式とプログラムの書き方は違います。

書き直すとすると、

    if(ransu[ii]<0.1) dosu[0]++;          // 正に決まっているので0と比べなくていい
    else if(ransu[ii]<0.2) dosu[1]++;     // elseなので0.1以上に決まっている
    else if(ransu[ii]<0.3) dosu[2]++;     // 同様
    else if(ransu[ii]<0.4) dosu[3]++;     // 同様
    else if(ransu[ii]<0.5) dosu[4]++;     // 同様
    else if(ransu[ii]<0.6) dosu[5]++;     // 同様
    else if(ransu[ii]<0.7) dosu[6]++;     // 同様
    else if(ransu[ii]<0.8) dosu[7]++;     // 同様
    else if(ransu[ii]<0.9) dosu[8]++;     // 同様
    else                   dosu[9]++;     // elseなので0.9以上に決まっているし、1未満に決まっている

else ifの意味をよく分かっていないのでは無いでしょうか。

実用的には、このように1行で書けます。

    dosu[ransu[ii]*10]++

Comments

[Murasaki_PurPle]

丁寧な回答をありがとうございます。
駆け出しの学生ですので、まだまだ理解の浅い部分が多々あるようです。

otn様の解説が丁寧だったので、ベストアンサーにさせていただきました。

Answer 3

if(0<=ransu[ii]<0.1) dosu[0]++;

明らかに間違い。
--> if ((0<=ransu[ii]) && (ransu[ii]<0.1)) dosu[0]++;

他も同様です。

---

[追記]
(0<=ransu[ii]<0.1) --> ((0<=ransu[ii]) < 0.1) で、(0<=ransu[ii]) 常に真 (1)、従って、(1 < 0.1) は、常に偽となります。(C言語の場合)

でもよく見ると、
if(0<=ransu[ii]<0.1) dosu[0]++; => if(ransu[ii]<0.1) dosu[0]++; でも良く、その他も同様、、

ですが、 一連の if()elseif()は、dosu[(int)(ransu[ii] * 10)]++; だけでも良さそう。