c# – 随机数生成器只生成一个随机数

我具有以下功能:

//Function to get random number
public static int RandomNumber(int min, int max)
{
    Random random = new Random();
    return random.Next(min, max);
}

我如何称呼它:

byte[] mac = new byte[6];
for (int x = 0; x < 6; ++x)
    mac[x] = (byte)(Misc.RandomNumber((int)0xFFFF, (int)0xFFFFFF) % 256);

如果我在运行时我使用调试器步循环,我得到不同的值(这是我想要的)。
但是,如果我在该代码下面两行放一个断点,“mac”数组的所有成员都具有相等的值。

为什么会这样呢?

最佳答案
每次你做新的Random()它使用时钟初始化。这意味着在严格的循环中,你得到相同的值很多次。您应该保留一个Random实例,并在同一实例上继续使用Next。

//Function to get random number
private static readonly Random random = new Random();
private static readonly object syncLock = new object();
public static int RandomNumber(int min, int max)
{
    lock(syncLock) { // synchronize
        return random.Next(min, max);
    }
}

编辑(见评论):为什么我们需要一个锁在这里?

基本上,Next将要改变Random实例的内部状态。如果我们从多个线程同时做到这一点,你可以说“我们刚刚使结果更随机”,但是我们实际上做的是潜在地打破内部实现,我们也可以开始获得相同的数字从不同的线程,这可能是一个问题 – 而可能不是。内部发生的事情的保证是更大的问题,因为Random不保证线程安全。因此,有两种有效的方法:

>同步,以便我们不会在不同的线程同时访问它
>每个线程使用不同的Random实例

可以是罚款;但是同时改变多个调用者的单个实例只是要求麻烦。

锁实现了这些方法的第一(和更简单);然而,另一种方法可能是:

private static readonly ThreadLocal<Random> appRandom
     = new ThreadLocal<Random>(() => new Random());

这是每线程,所以你不需要同步。

转载注明原文:c# – 随机数生成器只生成一个随机数 - 代码日志