随机数发生器只生成一个随机数

我有以下功能:

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

为什么会发生?


每次您执行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());
    

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


    为了便于在整个应用程序中重用,静态类可能会有所帮助。

    public static class StaticRandom
    {
        private static int seed;
    
        private static ThreadLocal<Random> threadLocal = new ThreadLocal<Random>
            (() => new Random(Interlocked.Increment(ref seed)));
    
        static StaticRandom()
        {
            seed = Environment.TickCount;
        }
    
        public static Random Instance { get { return threadLocal.Value; } }
    }
    

    您可以使用静态随机实例,使用代码如

    StaticRandom.Instance.Next(1, 100);
    

    Mark的解决方案可能相当昂贵,因为它需要每次同步。

    我们可以通过使用线程特定的存储模式来解决同步需求:

    
    public class RandomNumber : IRandomNumber
    {
        private static readonly Random Global = new Random();
        [ThreadStatic] private static Random _local;
    
        public int Next(int max)
        {
            var localBuffer = _local;
            if (localBuffer == null) 
            {
                int seed;
                lock(Global) seed = Global.Next();
                localBuffer = new Random(seed);
                _local = localBuffer;
            }
            return localBuffer.Next(max);
        }
    }
    
    

    衡量这两个实现,你应该看到一个重大的区别。

    链接地址: http://www.djcxy.com/p/12813.html

    上一篇: Random number generator only generating one random number

    下一篇: When to use reinterpret