C言語で乱数が毎回同じ値になってしまう

Question

前提・実現したいこと

C言語で12つの選択肢から一つをランダムに選ぼうとするコードを書いても毎回同じになってしまう。

発生している問題・エラーメッセージ

該当のソースコード

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

int GetRandom(int min,int max);
int main(){

    int ran,min=0,max=11;
    ran = GetRandom(min,max);

    /*結果発表*/
    printf("<%d>が選ばれました\n",ran);
    system("pause");

    return 0;
}

/*乱数取得用*/
int GetRandom(int min,int max)
{
	static int flag;

	if (flag == 0) {
		srand((unsigned int)time(NULL));
		flag = 1;
	}

	return min + (int)(rand()*(max-min+1.0)/(1.0+RAND_MAX));
}

<0>が選ばれました

試したこと

time()関数を使ってrand関数の初期値?(引数の値？)を指定しているはずなのですが、毎回同じ数値が出力されます。

以下のサイトを参考にしております
苦しんで覚えるC言語

Comments

[ozwk]

コピペして動かしてみましたがちゃんと違う値になりました（ideone）

環境を追記してください

Answer 1

int GetRandom(int min,int max)
{
    static int flag;

    if (flag == 0) {
        srand((unsigned int)time(NULL));
        flag = 1;
    }
    int r = rand();
    printf("rand() = %d factor = %lf\n", r, (max-min+1.0)/(1.0+RAND_MAX));
    return min + (int)(r*(max-min+1.0)/(1.0+RAND_MAX));
}

rand() と factor:(max-min+1.0)/(1.0+RAND_MAX) をプリントしてみると
factorがかなり小さい(0に近い)のが原因であるように思われます。

標準ライブラリの乱数(srand()/rand())は決して質の良いものではないので(特にVisual C++のは)、
シミュレーションなどが目的でより良い乱数がご所望でしたらメルセンヌ・ツイスタあたりを使うが吉ではないかと。

Comments

[Courange]

ご回答ありがとうございます。
丁寧で分かりやすかったですm(_　_)m

[episteme]

逆にさほどの質を求めないなら rand()%12 で十分に思えます。

Answer 2

次のコードを実行してみてください。

#include <stdio.h>   // printf
#include <stdlib.h>  // srand, rand
#include <time.h>    // time

int main()
{
	srand(time(0));
	for (int i = 0; i < 10; i++)
		printf(" %d", rand());
}

VC++ の場合、1秒以上の間をあけて何度も実行しても、
1番目の乱数は似たような値になります。
time() の返す値が似ているからでしょう。
srand() の後に rand() を 1回カラ打ちして、
2番目以降の値を使うとよいのではないでしょうか？

Comments

[Courange]

なるほど
やってみます

[Courange]

int GetRandom(int min,int max)
{
static int flag;

if (flag == 0) {
srand((unsigned int)time(NULL));
flag = 1;
}

for(int i = 0;i < 1 + (int)( rand() * (10) / (1.0 + RAND_MAX) );i++){

}
return min + (int)( rand() * (max - min + 1.0) / (1.0 + RAND_MAX) );
}

[Courange]

これで無事解決できました。
本当にありがとうございました。

[kazuma-s]

for文が必要ですか？
sand((unsigned int)time(NULL)); のすぐあとに rand(); と書くだけで 1回目の乱数を捨てられます。

Answer 3

乱数の品質というかsrandの性質が悪いのではないでしょうか。
長時間おいて実行してみるとか、
srand((unsigned int)time(NULL)*(unsigned int)time(NULL)*(unsigned int)time(NULL));
とかにしてみるとか。