输入验证(执行时循环)故障

Question

===== CS302 TOOL BOX =====
T > COIN TOSS SIMULATOR
G > GRADE ESTIMATOR
C > COLOR CHALLENGE
Q > QUIT
Type code letter for your choices: h
Invalid selection. Please try agian: 
t
Invalid selection. Please try agian: 
T

^这就是我运行程序并给出错误猜测时得到的。创建循环是为了验证用户输入，因此完全失败。对我做错了什么有什么想法吗？谢谢你的时间！

}
    System.out.println("===== CS302 TOOL BOX =====");
    System.out.println("T > COIN TOSS SIMULATOR");
    System.out.println("G > GRADE ESTIMATOR");
    System.out.println("C > COLOR CHALLENGE");
    System.out.println("Q > QUIT");
    System.out.print("Type code letter for your choices: ");
    boolean code_letter;
    String code_choice = scanner.next();
    do {
        if (code_choice.toUpperCase().equals("Q")
                || (code_choice.toUpperCase()).equals("C")
                || (code_choice.toUpperCase()).equals("G")
                || (code_choice.toUpperCase()).equals("T")) {
            code_letter = true;
        }
        else {
            System.out.println("Invalid selection. Please try agian: ");
            code_letter = false;
            scanner.next();
        }
    } while (!(code_letter));
    {
        System.out.println("you did it?");
    }
}
}

Answer 1

初始化

do-while 循环中的 code_choice 变量，否则在条件测试之前，该变量将不会为循环的每次迭代重新初始化。

String code_choice = scanner.next();

喜欢这个

//declared it outside to take care of the scope
String code_choice = null;
    do {
        //initialize it every time before you perform the conditional test.
        //This increases the readability!
        code_choice = scanner.next();
        if (code_choice.toUpperCase().equals("Q")
                || (code_choice.toUpperCase()).equals("C")
                || (code_choice.toUpperCase()).equals("G")
                || (code_choice.toUpperCase()).equals("T")) {
            code_letter = true;
        }
        else {
            System.out.println("Invalid selection. Please try agian: ");
            code_letter = false;
        }
    } while (!(code_letter));

希望这有帮助！

Answer 2

我假设扫描仪（小写 S）是扫描仪对象？下次请包含所有相关代码，否则很难调试问题。

你遇到的主要问题是在 else 条件下的行中。 scanner.next（） 不执行任何操作。您需要执行以下操作：

else {
    System.out.println("Invalid selection. Please try agian: ");
    code_letter = false;
    code_choice = scanner.next(); // This needs to be modified
}

这应该让它工作。

Answer 3

试试。

把你的

scanner.next()

在 if 之前，删除

围绕

System.out.println("you did it?"); 

最后你的时间...将其更改为..

while(code_letter !=  true);