每次您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
它 让线程安全的任何保证。因此,有两种有效的方法:
两者都可以。但是同时使多个调用者的 实例静音只会带来麻烦。
在lock
实现这些方法的第一(和更简单); 但是,另一种方法可能是:
private static readonly ThreadLocal<Random> appRandom
= new ThreadLocal<Random>(() => new Random());
这是每个线程的,因此您不需要同步。