我有以下功能:

//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类的新副本(因此它返回相同的值)。

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

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

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

其他回答

有很多解决方案,这里有一个:如果你只想要数字,那么就删除字母,然后方法会收到随机数和结果长度。

public String GenerateRandom(Random oRandom, int iLongitudPin)
{
    String sCharacters = "123456789ABCDEFGHIJKLMNPQRSTUVWXYZ123456789";
    int iLength = sCharacters.Length;
    char cCharacter;
    int iLongitudNuevaCadena = iLongitudPin; 
    String sRandomResult = "";
    for (int i = 0; i < iLongitudNuevaCadena; i++)
    {
        cCharacter = sCharacters[oRandom.Next(iLength)];
        sRandomResult += cCharacter.ToString();
    }
    return (sRandomResult);
}

在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

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

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);

每次执行时

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);

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);
    }
}

衡量这两种实现,您应该会看到显著的差异。