"为什么我的随机数生成器在 C# 中看起来不是随机的？"

Question

我正在使用 Microsoft Visual C# 2008 Express。

我找到了这段代码：

    public static int RandomNumber(int min, int max)
    {
        Random random = new Random();

        return random.Next(min, max);
    }

Answer 1

min = 0和max = 1的问题是min是包含的，max是独占的。 因此，该组合的唯一可能值为0。

Answer 2

random = new Random();

这将启动具有当前时间（以秒为单位）的随机数生成器。 在系统时钟更改之前多次调用函数时，随机数生成器将以相同的值启动，因此它返回相同的值序列。

Answer 3

不要为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）。

Answer 4

Next（）的重载返回：

大于或等于minValue且小于maxValue的32位有符号整数; 也就是说，返回值的范围包括minValue但不包括MaxValue。 如果minValue等于maxValue，则返回minValue。

0是唯一可能返回的值。 也许你想要random.NextDouble（），它将在0和1之间返回一个double。

Answer 5

最小值是包容性的，但最大值是独家的。 查看API

Answer 6

你总是得到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类; 看到这篇文章

编辑：线程安全

Answer 7

除了在其他答案中已经提到的0-1问题之外，当你正在寻找0-10范围并且连续50次获得相同结果时，你的问题是真正的问题。

new Random()应该返回一个带有从定时器初始化的种子的随机数（当前秒），但显然你每秒调用50次这个代码。 MSDN建议：“为了提高性能，创建一个随机数以随着时间的推移生成许多随机数，而不是重复创建一个新的随机数来生成一个随机数。” 如果您在方法之外创建一次随机生成器，那么应​​该修复“非随机性”问题以及提高性能。

如果你需要“更高质量”的伪随机数，还要考虑这篇文章 ，以获得比系统提供的更好的伪随机数发生器。

Answer 8

你误解了“random.Next（min，max）”这一行。 “min”代表允许随机生成的最小数字。 虽然“max”代表不允许生成的最小数字，但它不在允许绘制的最大数字的位置。 所以当行是random.Next（0,1）时，你基本上只允许绘制0。

Answer 9

正如其他人所提到的，每秒多次构建的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;
}

Answer 10

这是对任何答案的补充，因为这个特定问题的答案是边界应该是（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
}

Answer 11

我找到了一种非常简单但有效的方法来生成随机数，只需取当前日期时间毫秒的后两位数：

   int seed = Convert.ToInt32(DateTime.Now.Millisecond.ToString().Substring(1, 2));
   int cnr = new Random(seed).Next(100);

这很粗糙，但它确实有效！ :-)

当然，它每100次统计生成相同的数字。 或者，你可以取所有三位数或连接其他日期时间值，如秒左右。

Answer 12

你的范围不正确。 minValue 包含在范围内，而 maxValue 在范围内不包含（意味着它不会包含在范围内）。 所以这就是为什么它只返回 0。

另一个有用的注意事项：在方法中创建一个 Random 实例并不理想，因为它可能在调用时获得相同的种子。 所以我会说使用：

static Random gen = new Random();

public static int RandomNumber(int minValue, int maxValue){
    return gen.Next(minValue, maxValue);
}

Answer 13

一些海报已经声明Random（）使用基于系统时钟当前秒的种子，并且在同一秒中创建的任何其他Random实例将具有相同的种子。 这是不正确的。 Random的无参数构造函数的种子基于滴答计数或自启动时间以来的毫秒数。 此值大多每15毫秒在大多数系统上更新，但可能会因硬件和系统设置而异。

Answer 14

在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

希望这可以帮助！ :)