2017-02-20 1 views
4

Ich verstehe nicht, was hier vorgeht.Wiederholte Werte von 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); 
} 

Ein Beispiel für die Ausgabe dieses Codes ist:

45 
21 
10 
3 
54 
18 
23 
72 
68 
27 

68 
68 
68 
68 
68 
68 
68 
68 
68 
68 

number_in_range() Werke in genau der gleichen Weise wie der Code in meinem ersten for-Schleife, und doch spuckt es über den gleichen Wert und erneut. Was ist anders an der number_in_range() Version, und wie kann ich es beheben?

Antwort

12

Sie kopieren die Zufallsmethode, anstatt einen Verweis darauf zu nehmen. Daher hat es immer denselben inneren Zustand.

Versuchen:

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

Excellent. Vielen Dank. – anthropomorphic

+0

Als Randnotiz habe ich bemerkt, dass die erste generierte Zahl immer '45' ist. Ist das normal? Soll ich immer einen Wert nehmen und wegwerfen, wenn ich die 'std :: default_random_engine' benutze? – anthropomorphic

+0

@anthropomorph Dies ist ein reines Seeding-Problem. Werfen Sie keine Sachen weg, ohne PRNGs zu verstehen. Überprüfen Sie Ihre Sameneingabe! – sascha

Verwandte Themen