Бросок костей не работает C++ deafult_random_engine

По какой-то причине я продолжаю получать 6 каждый раз. Я знаю другой способ случайного броска костей, но я хотел научиться использовать deafult_random_engine.

#include <iostream>
#include <string>
#include <random>
#include <ctime>

using namespace std;

int main()
{
    default_random_engine randomGenerator(time(0));
    uniform_int_distribution<int> diceRoll(1, 6);

    cout << "You rolled a " << diceRoll(randomGenerator) << endl;
}

Но этот фрагмент кода работает с time(0).

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;

int main()
// dice roll
{
    srand(time(0));
    for(int x = 1; x < 2; x++){
        cout << 1+(rand()%6) << endl;
    }
    return 0;
}

c++ random dice numbers simulation
person Michael Schroeder    schedule 26.02.2015    source источник
comment
начните с того, что не используйте time() в качестве семени.   -  person T.C.    schedule 26.02.2015
comment
Я вставил несколько разных семян, 150, 165, 9050, но все они возвращают значение 1 для броска.   -  person Michael Schroeder    schedule 26.02.2015
comment
Ни один из этих комментариев или ответов не объясняет, почему time(0) такое плохое начальное значение. Я бы не ожидал, что приличная функция инициализации начального числа даст такие повторяющиеся результаты, если они отличаются на секунду или более. @MichaelSchroeder - не могли бы вы предоставить свою платформу/компилятор?   -  person Brett Hale    schedule 26.02.2015
comment
Какой компилятор вы используете? У меня он отлично работает на Visual C++ 2012, скомпилированном в 32-битном режиме.   -  person MrEricSir    schedule 26.02.2015
comment
Я использую блоки кода   -  person Michael Schroeder    schedule 26.02.2015
comment
Я также разместил рабочий бросок костей, используя время (0).   -  person Michael Schroeder    schedule 26.02.2015
comment
youtube.com/ это руководство, которому я следовал   -  person Michael Schroeder    schedule 26.02.2015

Ответы (1)


arrow_upward
0
arrow_downward

Здесь почти наверняка виновато использование time(0).

Вероятно, вам следует выбрать такой метод:

#include <iostream>
#include <string>
#include <chrono>
#include <random>
#include <ctime>

using namespace std;

int main() {
    default_random_engine randomGenerator(std::random_device{}());
    // OR:
    // default_random_engine randomGenerator(
    //  (unsigned) chrono::system_clock::now().time_since_epoch().count());

    uniform_int_distribution<int> diceRoll(1, 6);

    cout << "You rolled a " << diceRoll(randomGenerator) << endl;

    return 0;
}

В то время как ваш исходный код всегда производил 6 в моей системе, этот выглядит немного более "авантюрным":

pax> for i in {1..10}; do ./qq ; sleep 1 ; done
You rolled a 5
You rolled a 5
You rolled a 6
You rolled a 1
You rolled a 6
You rolled a 5
You rolled a 2
You rolled a 3
You rolled a 5
You rolled a 4
person paxdiablo    schedule 26.02.2015
comment
Но time(0) работает, когда я пишу его по-другому без default_random_engine. Как добавить скрипт в комментарий? Я новенький... - person Michael Schroeder; 26.02.2015
comment
Значит, это проблема с моей библиотекой ‹random›, верно? - person Michael Schroeder; 26.02.2015
comment
что такое std::random_device{}()) - person Michael Schroeder; 26.02.2015
comment
Потому что это просто дает мне 1 снова и снова. - person Michael Schroeder; 26.02.2015