The analyzer has detected suspicious code initializing the pseudorandom number generator to a constant value.
// C
srand(0);
// C++
std::mt19937 engine(1);
When initialized in such a way, the generator will be producing a predictable sequence of numbers or the same number at every run.
To avoid this, assign the seed to some random number such as the current system time:
srand(time(0));
However, this approach may lead to issues in multithreaded applications: 'time(0)' may return identical values in different threads. Also, keep in mind that the user could change the time settings.
Starting with C++11, the 'std::random_device' generator is available, which implements an interface to the true-random number generator:
std::random_device rd;
std::mt19937 engine(rd());
However, if your system lacks such a generator, the regular pseudorandom number generator will be used.
This diagnostic rule is classified as:
|
Was this page helpful?
Your message has been sent. We will email you at
If you do not see the email in your inbox, please check if it is filtered to one of the following folders: