Java:溢出和打印两次

Question

当我执行这部分代码时，我得到一个StackOverflowError：

/**
 * Determine the story based on the time.
 */
private void timeTell()
{
    if(replay == 0){
    long rndNum = System.currentTimeMillis() % 10;
    chooseStory();
    }
}
/**
 * Randomly choose which story to tell based on the current system time.
 */
private void chooseStory()
{
    if(rndNum == 1&& rndNum == 6){
        storyOne();
    }
    else if(rndNum == 2&& rndNum == 7){
        storyTwo();
    }
    else if(rndNum == 3&& rndNum == 8){
        storyThree();
    }
    else if(rndNum == 4&& rndNum == 9){
        storyFour();
    }
    else if(rndNum == 5&& rndNum == 0){
        storyFive();
    }
    else{
        timeTell();
    }
}

我知道我不需要timeTell()方法，解决这个问题后我会将其添加到chooseStory()方法中。这对于测试来说更容易。我试图找出问题发生的位置，所以我用System.out.println(rndNum);替换了chooseStory();，它会打印两次数字。变量replay用于查看程序是否已经运行过一次。如果用户决定再次播放，replay从默认值 0 更改为 1，并跳过生成新rndNum。我使用时间而不是随机数生成器的原因是，每次我运行程序时，生成器每次都会给我相同的序列。任何帮助将不胜感激。

Answer 1

您使用了错误的布尔运算符。

rndNum == x && rndNum == y

当且仅当 rndNum 等于 x 和 y 时，才会通过 - 如果 x 和 y 是不同的值，这将永远不会发生。 您需要使用 OR 运算符"||"：

rndNum == X || rndNum == y

此外，正如@Sotirios Delimanolis指出的那样，你也有可变的掩蔽。 他还解释了为什么会发生StackOverflowError。

Answer 2

如果只有一个变量rndNum则一次只能为其分配一个值。所以rndNum == 1&& rndNum == 6和以下所有if都是错误的。这就是为什么timeTell()总是被称为。

Answer 3

第一个 rndNum 是 timetell 的本地。所以可能实例级 rndNum 没有初始化。

但即使它被正确初始化，你的条件检查也必须是或不是 AND。一个变量一次只能有一个值。

private void chooseStory()
{
if(rndNum == 1|| rndNum == 6){
    storyOne();
}
else if(rndNum == 2|| rndNum == 7){
    storyTwo();
}
else if(rndNum == 3|| rndNum == 8){
    storyThree();
}
else if(rndNum == 4|| rndNum == 9){
    storyFour();
}
else if(rndNum == 5|| rndNum == 0){
    storyFive();
}
else{
    timeTell();
}

所以目前你所有的条件都出错了，控制每次都到达else块，因此对timetell的无休止的递归调用，因此stackOverFlow。