随机对象随后的调用变得非随机?

时间:2013-01-14 15:00:04

标签: c# memory optimization

  

可能重复:
  Random number generator only generating one random number

我的工作中有一个方法,它通过在一组单词之间随机选择产生一个句子,如下所示:

private string generateTest(string test)
{
    test = test.ToLower();
    string sentence = "";
    for (int c = 0; c < 30; c++)
    {
        Random r = new Random();
        int len = r.Next(3, 7);
        string word = "";
        for (int i = 0; i < len; i++)
        {
            word += test[r.Next(0, test.Length)];
        }
        sentence += word + ' ';
    }
    return sentence;
}

然而,问题在于,在随机生成前6个单词后,所有后续代都完全相同。 e.g。

  

aqaaaz,qqzaq,aqqza,azqq,aazzq,aqqa,azzzq,azzzq,azzzq,azzzq,   azzzq,azzzq,azzzq,azzzq,azzzq,azzzq,azzzq,azzzq,azzzq,azzzq,   azzzq,azzzq,azzzq,azzzq,azzzq ......

然后,即使我重新启动程序,同样的事情也适用。我在想这是.NET Runtime做一些自动优化。谁能解释得更好?我该如何解决这个问题?

1 个答案:

答案 0 :(得分:5)

您正在为循环的每个循环生成实例。将您的代码更改为:

Random r = new Random();
for (int c = 0; c < 30; c++)
{
...

此外,您可能希望为其分配有意义的种子。现在,您没有将任何内容传递给构造函数,如果重新创建相同的实例,这可能会产生非常随机的结果。

编辑:正如Shadow Wizard正确指出的那样,给它相同的种子产生相同的结果。当我说“有意义的种子”时,我暗示了具有随机或伪随机属性的东西。有many种方法可以产生更多不可预测的结果,但是要解决您的问题,将实例移出循环 - 或者将其实例化为包含类的成员 - 就​​足够了。您对种子或RNG的选择应取决于您的要求。