2017-02-20 4 views
4

ここで何が起こっているのか分かりません。std :: uniform_int_distributionからの反復値

#include <iostream> 
#include <random> 
#include <chrono> 
using namespace std; 

unsigned number_in_range(unsigned, unsigned, default_random_engine); 

int main() 
{ 

    time_t now = chrono::system_clock::to_time_t(chrono::system_clock::now()); 

    default_random_engine rng(now); 

    // 
    // Print out 10 random numbers 
    // 
    for (int i = 0; i < 10; i++) 
    { 
     uniform_int_distribution<int> dist(0, 100); 
     cout << dist(rng) << endl; 
    } 

    cout << endl; 

    // 
    // Do the same thing, but get the numbers from `number_in_range()` 
    // 
    for (int i = 0; i < 10; i++) 
    { 
     cout << number_in_range(0, 100, rng) << endl; 
    } 

    return 0; 
} 

unsigned number_in_range(unsigned range_start, unsigned range_end, default_random_engine rng) 
{ 
    uniform_int_distribution<int> dist(range_start, range_end); 
    return dist(rng); 
} 

このコードの出力例は次のとおりです。ループのための私の最初のコードと全く同じ方法で

45 
21 
10 
3 
54 
18 
23 
72 
68 
27 

68 
68 
68 
68 
68 
68 
68 
68 
68 
68 

number_in_range()作品、そしてまだそれが同じ値を超える吐き出しともう一度。 number_in_range()バージョンとは何が違うのですか?どのように修正できますか?

答えて

12

ランダムエンジンを参照する代わりにコピーしています。したがって、常に同じ内部状態を持ちます。

試してみてください。

unsigned number_in_range(unsigned range_start, unsigned range_end, default_random_engine &rng) 
+0

素晴らしいです。ありがとうございました。 – anthropomorphic

+0

補足として、最初に生成された数値は常に「45」であることに気付きました。それは普通ですか? 'std :: default_random_engine'を使うと、私はいつも一つの価値を取り、それを放棄すべきですか? – anthropomorphic

+0

@擬人化これは純粋な播種問題です。 PRNGを理解せずに物を捨てないでください。シード入力を確認してください! – sascha

関連する問題