其他如果和做虽然不按预期工作

Question

编辑：我最近发生的编译程序，我知道无法编译导致我相信我同时有一个问题与我的编译器。毫无疑问，由于我在Mac上运行WINE而不是原生应用程序。谢谢你的回复。我会正确地测试出所有的回应，并且在我用编译器解决了所说的错误时进行了所有更改，或者我已经将计算机移到了一个有效的计算机上。

我对编程相对比较陌生，所以请耐心等待。我的两条If语句出错，另外一条出现/我无法解决的错误。

整个程序按预期工作，但下面有两个块。问题是，当我输入字符'y'时，一切都按预期工作，并按照我的预期打印（“Results =”+ Arrays.toString（row））。它也会继续执行原始的For循环并再次启动程序。然而，当我输入任何其他字符时（即不是'y'或'n'），代码不会打印“输入必须是'y'或'n'”并且只是等待另一个输入。即使输入'n'，它也不会按照我想要的那样循环出来，它只是继续循环，而不是继续循环，如果我以为会的话。它无限期地这样做，不接受除'y'之外的任何其他输入以继续通过循环，因此我永远不会收到打印“负面”的消息。

有没有人有任何想法，为什么会发生这种情况？ 虽然不是技术上的家庭作业，但我将它标记为我想知道的，如果可能的话，发生了什么事情而不是如何解决它。

 do { 
      ans = input.next().charAt(0); 
       if (!ans.equals('y') || !ans.equals('n')) { 
        System.out.println ("Input must be either 'y' or 'n'"); 
       } 
     } while (!ans.equals('y') || !ans.equals('n')); 

和

if (ans.equals('y')) { 
      for (Object[] row : prevResults) { 
        System.out.println("Results = " + Arrays.toString(row)); 
      } 
     } // 
     else if (ans.equals('n')) { 
      System.out.println("Negative"); 
      //System.exit(0); 
     } 

的完整代码如下

import java.util.*; 

public class Averages { 
    public static void main (String [] args) { 

     //declare variables 
     int course, exam, average = 0; 
     char ans; 
     String pass; 

     //creating objects 
     Scanner input = new Scanner(System.in); 
     List<Object[]> prevResults = new ArrayList<Object[]>(); 

     //full loop 
     for (int i = 0; i < 5; i++) { 

      System.out.println ("Loop " + (++i) + " out of 5"); 

      //Course loop 
      do { 
      System.out.println ("Please enter a course mark out of 100"); 
      course = input.nextInt(); 
       if (course > 100) { 
        System.out.println ("Number entered is over 100"); 
       } 
      } while (course > 100); 

      //Exam loop 
      do { 
      System.out.println ("Please enter an exam mark out of 100"); 
      exam = input.nextInt(); 
       if (exam > 100) { 
        System.out.println ("Number entered is over 100"); 
       } 
      } while (exam > 100); 


      average = (course + exam)/2; 

      // Final Grade 
      System.out.println ("The average mark is " + average); 

      if (average >= 50 && course > 40 && exam > 40) { 
       System.out.println ("The final grade is pass"); 
       pass = "Pass"; 
      } 
      else { 
       System.out.println ("The final grade is fail"); 
       pass = "Fail"; 
      } 

      //add to array 
      prevResults.add(new Object[] { "Course mark: " + course, "Exam mark: " + exam,"Average: " + average, "Grade: " + pass}); 


      System.out.println ("Would you like to see previous results? y/n"); 


      //'Previous results' question loop 
      do { 
       ans = input.next().charAt(0); 
        if (!ans.equals('y') || !ans.equals('n')) { 
         System.out.println ("Input must be either 'y' or 'n'"); 
        } 
      } while (!ans.equals('y') || !ans.equals('n')); 


      // Close or Array if statement 
      if (ans.equals('y')) { 
       for (Object[] row : prevResults) { 
         System.out.println("Results = " + Arrays.toString(row)); 
       } 
      } // 
      else if (ans.equals('n')) { 
       System.out.println("Negative"); 
       //System.exit(0); 
      } 
     }// end for 
    }//end main 
}//end class 

编辑2：我有开关电脑，所有的建议答案确实工作。他们是

while (ans != 'y' && ans != 'n'); 

和

while (!(ans.equals('y') || ans.equals('n'))); 

和

制作一个单独的方法，由克里斯·布朗的建议。

为了读者的利益，这些解决方案非常有效，尽管我没有时间看看Greg Hewgill建议的BufferedReader，我很可能会实现它，因为它似乎是更好的选择说明。

Answer 1

首先，在你的代码中的错误：

System.out.println ("Loop " + (++i) + " out of 5"); 

你不应该增加i，因为它已经在增加你的for更新语句 - 你得到了错误的迭代计数结果。

接下来，你不应使用equals比较char值 - 使用==代替。当您使用equals时，由于auto-boxing会发生很多不必要的事情，最终导致大致characterObject.equals(anotherCharacterObject)，对象类型为java.lang.Character。只需使用例如代替ans == 'y'。

末，为乡亲们所指出的那样，你应该重写你的DO-而为：

do { 
    ans = input.next().charAt(0); 
    if (ans != 'y' && ans != 'n') { 
     System.out.println ("Input must be either 'y' or 'n'"); 
    } 
} while (ans != 'y' && ans != 'n'); 

，甚至有健康检查一个单独的方法（感谢克里斯·布朗）。

Answer 2

你的条件必须是

if (!ans.equals('y') && !ans.equals('n')) 

，而不是

if (!ans.equals('y') || !ans.equals('n')) 

Answer 3

的条件应该是：

while (!(ans.equals('y') || ans.equals('n'))); 

，因为你第一次写的条件：

while (!ans.equals('y') || !ans.equals('n')); 

总是等于true（每个字符不是y或不是n）。

并说谢谢De Morgan，也见my answer here。

Answer 4

} while (!ans.equals('y') || !ans.equals('n')); 

说虽然ans不等于'y'或者ans不等于'n'。这将始终是真实的（它总是会不等于其中之一）

Answer 5

你的条件逻辑是相反的，你说：

if (!ans.equals('y') || !ans.equals('n')) 

语义上，这是一个非常困难的线来分析，因为你在说“如果答案不是答案或答案不是n”，这是一个语句中的很多逻辑运算符。这可能是为什么你的错误悄悄进入。

真的，快速修复是将“或”换成“和”，但这并不能真正帮助你理解问题。你应该重构它使用不同的方法，如：

if(isLegalAnswer(ans)) 

其中isLegalAnswer被定义为：（！isLegalAnswer（ANS））

private static boolean isLegalAnswer(char ans) { 
    return (ans=='y' || ans=='n'); 
} 

这可以让你否定整个表达式使用单一否定。通常，我禁止自己在单个语句或表达式中使用多个否定，这有助于保持代码易于阅读，同时对执行速度做出很少或没有差别。