为什么我的随机数生成器在C#中不是随机的?
我在Microsoft Visual C#2008 Express中工作。
我找到了这段代码:
public static int RandomNumber(int min, int max) { Random random = new Random(); return random.Next(min, max); }
问题是我运行了100多次,当我的min = 0和max = 1时,它总是给我相同的答案。我每次都得到0。 (我创建了一个测试函数来运行它 – 真的 – 我每次都得到0)。 我很难相信这是巧合…我还能做些什么来检查或测试这个? (我确实用min = 0和max = 10重复测试,前50次,结果总是“5”,第二次50次,结果总是“9”。
?? 我需要更随意随意的东西……
-Adeena
min = 0和max = 1的问题是min是包含的,max是独占的。 因此,该组合唯一可能的值是0。
random = new Random();
这将启动具有当前时间(以秒为单位)的随机数发生器。 在系统时钟更改之前多次调用函数时,随机数生成器将以相同的值启动,因此它返回相同的值序列。
不要为Next创建包装器方法。 它浪费周期创建Random类的新实例。 只需使用同一个!
Random myRand = new Random(); for(int i = 0; i < 10; i++) { Console.WriteLine(myRand.Next(0, 10).ToString()); }
这应该给你十个随机值。
如前所述 - Random是伪随机的(如同所有实现一样),如果您使用相同的种子创建100个实例,您将获得100个相同结果的实例。 确保你重复使用课程。
此外,正如人们所说,请注意MinValue是包容性的,MaxValue是独家的。 根据你的需要,做myRand.Next(0,2)。
Next()的重载返回:
大于或等于minValue且小于maxValue的32位有符号整数; 也就是说,返回值的范围包括minValue但不包括MaxValue。 如果minValue等于maxValue,则返回minValue。
0是唯一可能返回的值。 也许你想要random.NextDouble(),它将在0和1之间返回一个double。
最小值是包容性的,但最大值是独家的。 查看API
你总是得到0因为Random.Next
返回整数。 您需要调用Random.NextDouble
,它将返回0到1之间的数字。此外,您应该重用Random实例,如下所示:
[ThreadStatic] static Random random; public static Random Random { get { if (random == null) random = new Random(); return random; } } public static int RandomInteger(int min, int max) { return Random.Next(min, max); } public static double RandomDouble() //Between 0 and 1 { return Random.NextDouble(); }
如果您想要加密安全的随机数,请使用RNGCryptoServiceProvider
类; 看到这篇文章
编辑:线程安全
除了在其他答案中已经提到的0-1问题之外,当你正在寻找0-10范围并且连续50次获得相同结果时,你的问题是真正的问题。
new Random()
应该返回一个带有从定时器初始化的种子的随机数(当前秒),但显然你每秒调用50次这个代码。 MSDN建议:“为了提高性能,创建一个随机数以随时间生成许多随机数,而不是重复创建一个新的随机数来生成一个随机数。” 如果您在方法之外创建一次随机生成器,那么应该修复“非随机性”问题以及提高性能。
如果你需要“更高质量”的伪随机数,还要考虑这篇文章 ,以获得比系统提供的更好的伪随机数发生器。
正如其他人所提到的,每秒多次构建的Random使用与种子相同的秒,因此我将Random构造函数放在循环之外,并将其作为参数传递,如下所示:
public static int RandomNumber(Random random, int min, int max) { return random.Next(min, max); }
另外如其他人所提到的,max是独占的,所以如果你想要0或1,你应该使用[0,2]作为[min,max]或更大的max,然后用1做二进制AND。
public static int RandomOneOrZero(Random random) { return random.Next(0, int.MaxValue) & 1; }
这是对任何答案的补充,因为这个具体问题的答案是界限应该是(0,2)而不是(0,1)。
但是,如果要使用静态包装器方法,则必须记住Random
不是线程安全的,因此您需要提供自己的同步机制或提供每线程实例。 这是一个很大程度上非阻塞的实现,它使用一个生成器为每个每个线程生成器设定种子:
public static class ThreadSafeRandom { private static readonly Random seed = new Random(); [ThreadStatic] private static Random random; public static int Next(int min, int max) { if (random == null) { lock (seed) { random = new Random(seed.Next()); } } return random.Next(min, max); } // etc. for other members }
你误解了“random.Next(min,max)”这一行。 “min”代表允许随机生成的最小数字。 虽然“max”代表不允许生成的最小数字,但它不在允许绘制的最大数字的位置。 所以当行是random.Next(0,1)时,你基本上只允许绘制0。
一些海报已经声明Random()使用基于系统时钟当前秒的种子,并且在同一秒中创建的任何其他Random实例将具有相同的种子。 这是不正确的。 Random的无参数构造函数的种子基于滴答计数或自引导时间以来的毫秒数。 此值大多每15毫秒在大多数系统上更新,但可能会因硬件和系统设置而异。
我找到了一种非常简单但有效的方法来生成随机数,只需取当前日期时间毫秒的最后两位数:
int seed = Convert.ToInt32(DateTime.Now.Millisecond.ToString().Substring(1, 2)); int cnr = new Random(seed).Next(100);
它很粗糙,但它有效! 🙂
当然,它每100次统计生成相同的数字。 或者,你可以取所有三位数或连接其他日期时间值,如秒左右。
在VB中我总是从Randomize()函数开始。 只需调用Randomize()然后运行随机函数。 我也做了以下事情:
Function RandomInt(ByVal lower As Integer, ByVal upper As Integer) As Integer Return CInt(Int((upper - lower + 1) * Rnd() + lower)) End Function
希望这可以帮助! 🙂