Как сделать членную функцию класса в C++ генерировать различное случайное число каждый раз, когда оно вызывается?

Question

У меня есть класс, который также включает в себя случайное число двигателя и его распределение:

#include <iostream> 
#include <cmath> 
#include <random> 
#include <chrono> 

class C 
{ 
    public: 
     typedef std::mt19937_64 engine; 
     typedef std::uniform_real_distribution<double> distribution; 
     . 
     . 
     . 

    protected: 
     engine rng; 
     distribution dist; 
     void func(); 
     . 
     . 
     . 
}; 

Так как конструктор вызывается только один раз, я положил семена в нем:

C::C() 
{ . 
    . 
    . 
    unsigned seed = std::chrono::system_clock::now().time_since_epoch().count(); 
    distribution dist(0.0, pow(10,12)); 
    engine rng(seed); 
} 

Следующая функция член предполагается генерировать случайное число и будет вызываться много раз в одном запуске программы:

void C::func() 
{ . 
    . 
    . 
    double randNum = floor(dist(rng)); 
    std::cout << randNum << std::endl;  
    . 
    . 
    . 
} 

Однако каждый время генерирует число 0 как случайное число. Кажется, что dist(rng) не выполняет свою работу.

Мне действительно нужно найти проблему и исправить выход. Буду признателен за любую помощь.

Answer 1

Вы определяете ваши dist и eng переменные в классе

protected: 
    engine rng; 
    distribution dist; // Here 
    void func(); 

И в конструкторе вы определяете другой с таким же именем

C::C() 
{ . 
    . 
    . 
    unsigned seed = std::chrono::system_clock::now().time_since_epoch().count(); 
    distribution dist(0.0, pow(10,12)); // Here 
    engine rng(seed); 
} 

Я предполагаю, что вы хотите, чтобы последние присваивание, а не новая переменная. Затем вы получите номера, которые вы хотите.

C::C() 
{ . 
    . 
    . 
    unsigned seed = std::chrono::system_clock::now().time_since_epoch().count(); 
    dist = distribution(0.0, pow(10,12)); // Here 
    rng = engine(seed); 
} 

Или с инициализацией

C() : dist(0.0, pow(10, 12)), 
    rng(std::chrono::system_clock::now().time_since_epoch().count()) 

Answer 2

В конструкторе вы создаете локальную переменную dist, скрывающую член класса, в то время как член класса получает по умолчанию инициализируется. По умолчанию инициализируется uniform_real_distribution генерирует значения в диапазоне [0,1), который floor всегда превратится в 0.

Answer 3

Самих близко, но вы действительно должны быть инициализация этих членами:

C::C() 
    : dist(0.0, pow(10, 12)) 
    , rng(std::chrono::system_clock::now().time_since_epoch().count()) 
{}