进程因 StackOverflowExeption 而终止 - C#

Question

我的代码有这个问题，我无法修复。

我正在做一个测验，但这些问题不能被问两次，所以我避免了这种情况，但是当它用完选项时它会崩溃。

也可能是后面的代码，但我不这么认为。

这是我的代码：

private static void chooseQuestion()
{
    Random randomQuestion = new Random();

    int returnValue = randomQuestion.Next(1, 3);

    switch (returnValue)
    {
        case 1:
            if (randomValues.questionOneChosen != 1)
            {
                questionOne();
            }else
            {
                chooseQuestion();
            }
            break;
        case 2:
            if (randomValues.questionTwoChosen != 1)
            {
                questionTwo();
            }else
            {
                chooseQuestion();
            }
            break;
    }

    endQuiz();
}

这是它之后的内容：

private static void endQuiz()
{
    Console.Clear();
    Console.WriteLine();
    text.centered("You completed the QUIZ, well done!");
    Console.WriteLine();
    text.centeredWrite("Press ENTER to go back to the menu");
    string input = Console.ReadLine();
    if (input == "")
    {
        Menu.main();
    }else
    {
        endQuiz();
    }
}

如果您需要更多代码来帮助我，请询问我。

提前致谢！

Answer 1

StackOverflow 上的 StackOverflowException，很好！ Menu.main() 最终会调用chooseQuestion() 吗？ 调用函数时，会在堆栈上创建一个新条目，其中包含所有局部变量的空间以及返回时要返回的地址。如果一个函数调用自己（递归调用），它的原始堆栈条目不会被释放。如果它经常调用自己而不返回，堆栈最终会用完空间并给出这个异常。话虽如此，堆栈非常大 - 我认为默认值为 1 兆字节 - 所以你需要大量的递归调用来耗尽它。

你有两个函数都调用自己。如果他们经常这样做，您将耗尽堆栈空间。只需用循环替换递归调用。这是您的第一个函数：

private static void chooseQuestion() 
{    
    bool endQuizChosen = false;
    while ( !endQuizChosen ) {
        Random randomQuestion = new Random();
        int returnValue = randomQuestion.Next(1, 3);  
        // ... the rest of the function ...
        endQuizChosen = endQuiz();
    }
}

并编辑endQuiz() 使其不调用自身，但如果用户想停止则返回true，如果用户想继续则返回false。

Answer 2

您的chooseQuestion 方法的逻辑如下：

a. Pick a 1 or 2, randomly
b. If 1, and I haven't asked question 1 yet, show question 1
c. if 1, and I have asked question 1, go to step (a)
d. repeat (b) and (c) for question 2
...

一旦你回答了这两个问题，你就会尝试得到另一个随机问题。但是没有更多可用的问题。所以你递归到一个异常中。

不要这样写你的逻辑。使用集合，并在您显示该问题后从集合中删除该项目。当集合为空时，您就完成了。

Answer 3

这永远不会设置。

randomValues.questionOneChosen

所以无论它是什么，如果它不是 1 或 2，则该方法将再次调用自身。它可能总是为零。