每次您
newRandom()使用时钟初始化它。这意味着在紧密的循环中,您会多次获得相同的值。您应该保留一个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());
这是每个线程的,因此您不需要同步。
欢迎分享,转载请注明来源:内存溢出
评论列表(0条)