随机数生成器仅生成一个随机数

一尘不染

随机数生成器仅生成一个随机数

我有以下功能：

//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数组的所有成员都具有相等的值。

为什么会这样呢？

阅读 423

2020-05-19

共1个答案

一尘不染

每次您new Random()使用时钟初始化它。这意味着在紧密的循环中，您会多次获得相同的值。您应该保留一个Random实例，并在同一
实例上继续使用Next。

//Function to get a 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);
    }
}

编辑（请参阅评论）：为什么我们需要lock这里？

基本上，Next将要更改Random实例的内部状态。如果我们同时在多个线程中执行此操作，则可以争论“我们只是使结果更加随机”，但 实际上
，我们正在做的事情有可能破坏内部实现，并且我们也可能开始获得相同的数字从不同的线程，这可能是一个问题-
可能不是。但是，保证内部发生的事情是更大的问题。因为Random它不是让线程安全的任何保证。因此，有两种有效的方法：

同步，这样我们就不会同时从不同的线程访问它
Random每个线程使用不同的实例

两者都可以。但是同时使多个调用者的单个实例静音只会带来麻烦。

在lock实现这些方法的第一（和更简单）; 但是，另一种方法可能是：

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

这是每个线程的，因此您不需要同步。

2020-05-19