为什么我的 switch 语句不起作用并陷入无限循环？

Question

我试图让它输出用户的财富，但它只会输出用户无限输入的数字。

我做错了什么？如何修复无限循环？

public static void main(String[] args) {

    Scanner broncos = new Scanner ( System.in );

    int x, y, a;

    System.out.println("Would you like me to tell your fortune? Type 1 for yes and any other number for no: ");
    x = broncos.nextInt();


    while (x==1)
    {   
        System.out.println("Enter a number from 1-8: ");
        y = broncos.nextInt();

        while ( y > 0 && y < 9)

        System.out.println( y );
        System.out.print ( "That's not a valid number. Try again. ");
        y = broncos.nextInt();
    }


    System.out.println ("Enter a number from 1-8: ");
    a = broncos.nextInt();


    switch (a) 
    {
        case 8 : 
            System.out.println ("You will win the lottery" );
            break; 
        case 7 : 
            System.out.println ("You are going to do great things");
            break; 
        case 6 : 
            System.out.println ("Don't get out of the house today");
            break; 
        case 5 : 
            System.out.println ("You'll get a raise at work");
            break;  

        case 4 : 
            System.out.println ("Lucky day today");
            break; 

        case 3 : 
            System.out.println ("Not so lucky day today");
            break; 

        case 2 : 
            System.out.println ("Your favorite sports team will win" );
            break; 

        case 1 :
            System.out.println ("Your car is going to break down" );
            break;

        default: 
            System.out.println ( "You didn't enter 1-8!" );
    }

    broncos.close();
}

Answer 1

导致问题的不是您的 switch 语句，而是之前的 while 循环。您像在while (x==1) 中一样测试x，但是在while 循环中您在哪里更改x？如果你从不改变x的状态，测试永远不会变成假，循环也永远不会退出。

正如 Tom 敏锐地指出的那样，您将希望验证您的 y 输入的 while 循环包含与获取 y 输入相关的所有代码。

while ( y > 0 && y < 9) {
    System.out.println("Enter a number from 1-8: ");
    y = broncos.nextInt();

    if (y > 0 && y < 9) {    
        System.out.println( y );
        System.out.print ( "That's not a valid number. Try again. ");
    }
}

现在在你的 switch 语句中使用正确的y。

另外，一旦你有一个正确的y，就没有必要得到一个a：

// System.out.println ("Enter a number from 1-8: ");
// a = broncos.nextInt();

同样，只需在 switch 语句中使用 y。

至于您的x，无需验证它（除非您需要检查它是否实际上是一个数字），因为输入的任何数字都应该有效。