我有以下功能:

//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 = new Random (15);

执行数百万次并不重要,您将始终使用相同的种子。

如果您使用

Random random = new Random ();

如果黑客猜到了种子,并且你的算法与系统的安全性有关,你会得到不同的随机数序列——你的算法被破坏了。如果你处决了骡子。在该构造函数中,种子由系统时钟指定,如果在极短的时间(毫秒)内创建了多个实例,则它们可能具有相同的种子。

如果你需要安全的随机数,你必须使用这个类

System.Security.Cryptography.RGCryptoServiceProvider

public static int Next(int min, int max)
{
    if(min >= max)
    {
        throw new ArgumentException("Min value is greater or equals than Max value.");
    }
    byte[] intBytes = new byte[4];
    using(RNGCryptoServiceProvider rng = new RNGCryptoServiceProvider())
    {
        rng.GetNonZeroBytes(intBytes);
    }
    return  min +  Math.Abs(BitConverter.ToInt32(intBytes, 0)) % (max - min + 1);
}

用法:

int randomNumber = Next(1,100);

其他回答

每次执行新的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);
    }
}

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

基本上,Next将更改Random实例的内部状态。如果我们同时从多个线程执行这一操作,你可能会认为“我们只是让结果变得更加随机”,但我们实际上正在做的事情可能会破坏内部实现,我们也可能会从不同的线程获得相同的数据,这可能是一个问题,也可能不是。不过,内部情况的保证是更大的问题;因为Random不保证线程安全。因此,有两种有效的方法:

同步,以便我们不会同时从不同的线程访问它每个线程使用不同的随机实例

两者都可以;但同时从多个调用方互斥一个实例只是自找麻烦。

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

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

这是每个线程,因此不需要同步。

为什么会这样?

正如前面所回答的,每次调用new Random()时,都会得到用相同时钟初始化的Random类的新副本(因此它返回相同的值)。

现在,从.NET6开始,有一个易于使用且线程安全的替代方案:Random.Shared

在您的示例中,您可以删除所有函数RandomNumber,然后使用以下代码(使用相同的逻辑,但现在它工作正常):

byte[] mac = new byte[6];
for (int x = 0; x < 6; ++x)
    mac[x] = (byte)(Random.Shared.Next(0, 255));

我的答案是:

只是重申正确的解决方案:

namespace mySpace
{
    public static class Util
    {
        private static rnd = new Random();
        public static int GetRandom()
        {
            return rnd.Next();
        }
    }
}

因此,您可以拨打:

var i = Util.GetRandom();

自始至终。

如果您严格需要一个真正的无状态静态方法来生成随机数,那么可以依赖Guid。

public static class Util
{
    public static int GetRandom()
    {
        return Guid.NewGuid().GetHashCode();
    }
}

它会慢一点,但可能比random更随机。接下来,至少从我的经验来看。

但不是:

new Random(Guid.NewGuid().GetHashCode()).Next();

不必要的对象创建会使其变慢,尤其是在循环下。

永远不要:

new Random().Next();

不仅速度慢(在循环内),它的随机性……根据我的说法,不是很好。。

在Visual Basic中,这是可行的(可能可以转换为C#,如果不是,DLL引用可以是一个解决方案):

Private Function GetRandomInt(ByVal Min As Integer, ByVal Max As Integer) As Integer
     Static Generator As System.Random = New System.Random()
     Return Generator.Next(Min, Max)
End Function

我使用这个:

int randomNumber = int.Parse(Guid.NewGuid().ToString().FirstOrDefault(Char.IsDigit).ToString().Replace("\0", "0"));

性能:在我的电脑上生成100万个随机数:711毫秒。

如果Guid不包含任何数字(我不知道这是否可能),则将使用0作为结果。