C++使用<random>库生成首个随机数始终相同的问题求助

问题根源 #

你每次调用randomSpawn()时，都会重新创建std::default_random_engine实例并使用time(NULL)初始化种子。time(NULL)返回的是秒级时间戳，如果短时间内多次调用该函数（比如1秒内多次执行），种子值完全相同，导致std::default_random_engine生成的随机序列完全一致，第一个输出值自然固定为17。

另外，你同时混用了<random>库和rand()函数，这完全没必要，而且rand()本身也存在类似的种子重复问题（只是你测试时可能间隔超过1秒，所以没显现）。

修正代码 #

正确的做法是将随机引擎和分布声明为静态变量，确保它们只在第一次调用函数时初始化一次，后续调用复用同一个引擎实例：

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

// 假设LENGHT是已定义的常量，比如const int LENGHT = 30;

int randomSpawn()
{    
    // 静态变量，仅第一次调用时初始化
    static unsigned seed = time(NULL);
    static std::default_random_engine rng(seed);
    static std::uniform_int_distribution<int> d(1, LENGHT);
    
    // 移除不必要的srand和rand()调用
    int result = d(rng);
    std::cout << result << std::endl;

    return result;
}

额外说明 #

• 静态变量的生命周期是整个程序运行期间，所以引擎只会被初始化一次，种子也只会设置一次，后续调用会持续生成不同的随机数。
• 如果你的程序需要更高质量的随机数，可以考虑替换std::default_random_engine为std::mt19937（梅森扭结器），它的随机性更好：

  static std::mt19937 rng(seed);
  

• 避免在循环或频繁调用的函数内重复创建随机引擎和分布，这不仅会导致重复序列，还会降低性能。

内容的提问来源于stack exchange，提问作者Michael