ループ処理での乱数生成

Question

ループ処理での乱数生成

現在モンティーホール問題を実装していますが、ループ処理のところで全て同じ値になってしまいます。

モンティホール問題

実装

以下のようなコードを書いています。
forでのループ処理で都度乱数を生成して各ループで異なる結果を出したいのですが、
出力がwin_count = 10000, lose_count = 0
のようになってしまいます。おそらく1度目の結果をそのまま返してしまっています

記載コードはドアを変えない場合のシミュレーションです。

#include<iostream>
#include <stdlib.h> // rand, srand関数
#include <time.h>   // time関数
using namespace std;

// 0:ひつじ, 1:くるま
int door[3] = {0, 0, 1};
int choose_num;
int win_count = 0;
int lose_count = 0;

// ドアを開ける
int choose_door() {
    // 乱数を生成してreturn
    unsigned int now = (unsigned int)time( 0 );
    srand(now);
    return rand() % 3;
}

// ひつじのドアを開ける
int releave_door() {
    int n;
    for(int i=0; i<3; i++) {
        if(door[i] == 1 || i == choose_num) continue;
        n = i;
    }
    return n;
}

// ドアを変えなかった場合
void not_change_door_result() {
    if(door[choose_num] == 1) win_count++;
    else lose_count++;
}

int main() {
    // 試行回数10000回で実験
    for(int i=0; i<10000; i++) {
        choose_num = choose_door();
        int sheep_door = releave_door();
        not_change_door_result();
    }
    cout << win_count << " " << lose_count << endl;
}

##　質問
ループの度に異なる乱数を生成して、正しいシュミレーションを行いたいです。
アドバイスをよろしくお願いいたします。

Comments

[episteme]

× シュミレーション
〇 シミュレーション

[episteme]

必要な変数をなんもかんもグローバルにするのはお行儀悪い。

Answer 1

choose_doorに以下の記述がありますが、このままだとchoose_doorを呼び出すたびに、疑似乱数のシードが現在日時に基づいて設定されます。

    unsigned int now = (unsigned int)time(0);
    srand(now);

すると、現在時刻が同じ秒数の間は、同じシードが設定されるため、最初のrand()で得られる値は同一になります。現実的に、現在のPCの処理速度では1秒間にすべての試行(10000回)が終了してしまうため、(途中で秒数が変わらなければ)すべて同一のrand()の返り値でシミュレーションが行なわれ、同一の結果が10000個得られることになります。

解決策は、他の方の回答にあるように、シードの設定をmainの冒頭で行ない、choose_doorではrand()だけを行なうことです。

Answer 2

    unsigned int now = (unsigned int)time( 0 );
    srand(now);

をmainの頭に移動するとよいと思います。

Answer 3

unsigned int now = (unsigned int)time( 0 );
srand(now);

seedの設定↑はmainのアタマで一回だけ。