C# 方法在应返回唯一值时未返回唯一值

Question

我有两种方法，generateNounPhrase() 和generateVerbPhrase()。 VerbPhrase 有一半时间会调用 NounPhrase，它的输出应该具有以下效果：

空地重新激活这个金字塔

（粗体表示在逻辑上调用generateNounPhrase()的位置）。然而，真正的输出是以下形式：

空地重新激活空地

起初我以为我的 randomIndex 方法没有按我的预期工作，但如果我再次运行这两个方法，我确实得到了不同的名词短语，但它们在句子的开头和结尾并不像应有的那样唯一。

知道我做错了什么才能得到一种方法来显示相同​​的结果吗？

private string generateNounPhrase()
{
    string nounPhraseString = "";
    nounPhraseString = nounMarkersStringList[randomIndex(0,nounMarkersStringList.Count-1)];
    if (included(1, 4, 2) == true)
    {
        nounPhraseString += " " + adjectivesStringList[randomIndex(0, adjectivesStringList.Count - 1)];
    }
    nounPhraseString += " " + nounsStringList[randomIndex(0, nounsStringList.Count - 1)];
    return nounPhraseString;
}

private string generateVerbPhrase()
{
    string verbPhraseString = "";
    if (included(1, 4, 2) == true)
    {
        verbPhraseString = intransitiveVerbsStringList[randomIndex(0, intransitiveVerbsStringList.Count - 1)];
    }
    else
    {
        verbPhraseString = transitiveVerbsStringList[randomIndex(0, transitiveVerbsStringList.Count - 1)] + " " + generateNounPhrase();
    }
    return verbPhraseString;
}

Answer 1

如果没有看到 randomIndex 的代码，我无法确定，但看起来您每次调用 randomIndex 时可能都会创建 Random 类的新实例。如果您在很短的时间内执行此操作两次，就像您在这里一样，它将两次为随机数生成器播种相同的值（因为它是用当前时间播种的），并且您将获得相同的“随机”数两次都回来了。

相反，您应该对所有 randomIndex 调用使用单个 Random 实例。

Answer 2

我高度怀疑您的 randomIndex 方法会产生以下效果：

Random r = new Random();
return r.Next (0, max);

上述代码的问题是，当您在不同时间运行它时，它会返回不同的值。但是，如果您像下面这样运行它，它几乎总是返回相同的值：

Console.WriteLine ("{0} == {1}?", randomIndex(0,10), randomIndex(0,10));

这是因为随机数生成器使用当前时间作为其种子。 2 个随机化器同时创建，将始终返回相同的值。正确的方法是：

class MyRand  // Assuming MyRand is the class name
{
   private Random r = new Random();
   public int GetRand(int min, int max)
   {
       return r.Next(min, max);
   }
}

或者，您可以使成员静态。