我有以下功能:

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

其他回答

1) 正如马克·格雷维尔所说,尝试使用一个随机生成器。将此添加到构造函数System.Environment.TickCount中总是很酷的。

2) 一个提示。假设您要创建100个对象,并假设每个对象都应该有自己的随机生成器(如果您在很短的时间内计算随机数的负载,则很方便)。如果要在循环中执行此操作(生成100个对象),可以这样做(以确保完全随机性):

int inMyRandSeed;

for(int i=0;i<100;i++)
{
   inMyRandSeed = System.Environment.TickCount + i;
   .
   .
   .
   myNewObject = new MyNewObject(inMyRandSeed);  
   .
   .
   .
}

// Usage: Random m_rndGen = new Random(inMyRandSeed);

干杯

从.NET 6开始,Random类现在配备了一个名为Shared的静态属性:

提供可从任何线程并发使用的线程安全随机实例。

你可以这样使用:

// Function to get random number
public static int RandomNumber(int min, int max)
{
    return Random.Shared.Next(min, max);
}

访问线程安全对象的开销很小,因此如果您计划在单个线程上尽可能快地生成数百万个随机数,最好创建一个专用的random实例,而不是依赖Shared。

您可以使用以下代码:

public static class ThreadSafeRandom
{
    private static readonly Random _global = new Random();
    private static readonly ThreadLocal<Random> _local = new ThreadLocal<Random>(() =>
    {
        int seed;
        lock (_global)
        {
            seed = _global.Next();
        }
        return new Random(seed);
    });

    public static Random Instance => _local.Value;
}

此代码可以按原样使用,也可以通过NuGet包ThreadSafeRandomizer使用。

编辑:从.NET 6.0开始,您可以改用Random.Shared.Next()。您仍然可以使用上面的包,它可以在上面的代码或Random.Shared与预处理器指令之间进行选择。

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

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

我使用这个:

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

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

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