我有以下功能:
//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));
我的答案是:
只是重申正确的解决方案:
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();
不仅速度慢(在循环内),它的随机性……根据我的说法,不是很好。。