你可以重复使用 if 语句吗？程序只循环到第一个 if 循环

Question

这个程序是通过取 A 和 B 边的值来计算三角形的斜边。如果程序运行，它会为用户提供 4 种选择，在用户输入 A、B、 C，或Q。假设用户输入A，它允许用户输入A侧的值，它将返回4个主要选择。问题是，当用户输入 A、B、C 或 Q 时，它只是循环再次询问 A 面。以下是我目前拥有的代码。我如何使它不仅接受值A？谢谢

public class Newest_Exercise_1 {

     public static void main(String[] args) {    
         Scanner input = new Scanner(System.in);
         char letter;
         int valueOfA;
         int valueOfB;
         double hypotenuse;


         Boolean loop =true;

         letter = GettingUserInput(input);
            System.out.println("value of letter: " +letter);

         while(loop){
         if(letter=='A' || letter =='a'){
                valueOfA = InputWasA();
                System.out.println(valueOfA);
                letter = GettingUserInput(input);


         }
         if(letter=='B' || letter =='b'){
                valueOfB = InputWasB();
                System.out.println(valueOfB);
                letter = GettingUserInput(input);

         }

         if(letter=='C' || letter =='c'){
                System.out.println("made it to C");
                hypotenuse = Math.sqrt((valueOfA*valueOfA)+(valueOfB*valueOfB));
                System.out.println("Hypotenuse is: "+hypotenuse);
                letter = GettingUserInput(input);


         }
         if(letter=='Q' || letter =='q'){
                System.out.println("made it to Q");
                System.out.println("Program Closed");
                System.exit(0);
         }  
         }
     }

     public static char GettingUserInput(Scanner input){


            System.out.println("A-Enter value of side A");
            System.out.println("B-Enter value of side B");
            System.out.println("C-Calculate");
            System.out.println("Q-Quit the program");

            String s = input.next();
            System.out.println("value of s: "+s);
            char letter = s.charAt(0);

            while(letter != 'A' && letter != 'a' && letter != 'B' && letter != 'b' && letter != 'C' && letter != 'c' && letter != 'Q' && letter != 'q'){
                System.out.println("Invalid entry, Please try again");
                System.out.println("A-Enter value of side A");
                System.out.println("B-Enter value of side B");
                System.out.println("C-Calculate");
                System.out.println("Q-Quit the program");
                s = input.next();
                letter = s.charAt(0);
            }


        return letter; 
     }

     public static int InputWasA(){
         Scanner input = new Scanner(System.in);
         Boolean loop1 = true;
         int valueOfA = 0;

         while(loop1){
             try{
                 System.out.println("Enter value of side A");
                 valueOfA=input.nextInt();
                 loop1 = false;
             }
             catch(Exception e){
                 System.out.println("That was not an integer!");
                 input.next();
             }

         }
         return valueOfA;
     }

     public static int InputWasB(){
         Scanner input = new Scanner(System.in);
         Boolean loop2 = true;
         int valueOfB = 0;

         while(loop2){
             try{
                 System.out.println("Enter value of side B");
                 valueOfB=input.nextInt();
                 loop2 = false;
             }
             catch(Exception e){
                 System.out.println("That was not an integer!");
                 input.next();
             }

         }
         return valueOfB;
     }

     public static void InputWasC(){
            System.out.println("Made it to InputWasC!");
     }





}

Answer 1

你只执行一次letter = GettingUserInput(input)，所以letter当然永远是同一个值。

所以：

• 将此行移到内循环开始处
• 不要在其他任何地方重复 - 它现在在每次迭代的顶部运行
• 并废弃loop 变量，只需使用true

使代码看起来像：

while (true) {
    letter = GettingUserInput(input);
    System.out.println("value of letter: " + letter);

    if (letter=='A' || letter =='a') {
        valueOfA = InputWasA();
        System.out.println(valueOfA);
    }

    // etc

我拒绝提出其他改进建议，只保留导致所述问题的那个，但是...

因为你的循环有：

• 初始状态（读信）
• 终止测试（不是“q”）
• 迭代操作（阅读另一封信）

它是最清晰和最好的编码为for 循环。此外，为了避免所有这些双重大小写测试，GettingUserInput() 方法应该只返回小写字母，即它的最后一行应该是：

return Character.toLowerCase(letter); 

鉴于这种变化，我认为您的循环应该如下所示：

for (char letter = GettingUserInput(input); letter != 'q'; letter = GettingUserInput(input)) {
    if (letter =='a') {
        valueOfA = InputWasA();
        System.out.println(valueOfA);
     }

     if (letter =='b') {
         valueOfB = InputWasB();
         System.out.println(valueOfB);
     }

     if (letter =='c') {
         System.out.println("made it to C");
         hypotenuse = Math.sqrt((valueOfA*valueOfA)+(valueOfB*valueOfB));
         System.out.println("Hypotenuse is: "+hypotenuse);
     }
}

System.out.println("Program Closed");

注意字母 q 的代码是如何简单地跟随循环的。