命令提示冻结后按回车，无效循环

我目前正在制作一个程序，允许您选择4个选项中的1个，为了正确执行此操作，我必须使用循环。这个简单的程序的要点是允许用户选择1 4种选择：命令提示冻结后按回车，无效循环

1 - 由 2设置％至AVG - 输入级（并采取进入那些成绩平均） 3 - 获取平均（我假设平均等级和平均百分比） 4 - 退出

目前，我没有得到编译错误，并且能够运行该程序。每当我输入1并按回车，出于某种原因，号码进入，但它只是一个空格，我不得不按CTRL-C解冻它。我也不知道如何获得“while（choice！= 1）;”行来正确执行。

我需要让程序循环，允许用户按他们想要的次数执行所有的选项，直到他们按下4退出，所以我使用了一个定点控制的循环。这是我的代码，我是初学者，所以我可能没有完成整个“循环”过程。谢谢！

import java.util.Scanner; 

public class ExerciseThree 
{ 
public static void main (String[] argsv) 
{ 

    float percent = 0; 
    double grade = 0; 
    double totalAvg = 0; 
    double total = 0; 
    double gradeAvg = 0; 

    int gradeCounter = 0; 
    int quit; 
    int choice; 
    int choiceOne; 


    Scanner input = new Scanner (System.in); 

    System.out.println("Please choose one of the following: \n 1 - Set percentage of total for new grades \n 2 - Enter new grades \n 3 - Get average \n 4 - Quit "); 
     choice = input.nextInt(); 


    while (choice != 4); 

     switch (choice) 
     { 

      case 1: 

      if(choice == 1) {  
       System.out.println("Enter a percentage to multiply by"); 
       percent = input.nextFloat(); 

      break; 
      } 

      case 2: 

      if (choice == 2) { 
       System.out.println("Enter grades"); 
       grade = input.nextDouble(); 
       total = total + grade; 
       gradeCounter = gradeCounter + 1; 
       gradeAvg = (double) total/gradeCounter; 
      break;  
      } 



      case 3: 

      if (choice == 3) { 
       System.out.println("You have chosen to get the average"); 
       totalAvg = totalAvg + percent * grade; 
       totalAvg = input.nextDouble();   
      break; 
      } 


      default: 

      if (choice == 4){ 
       System.out.println("You have chosen to quit"); 
       quit = input.nextInt(); 
      break; 
      } 



     }

来源

2014-02-17 user3304333

'如果（选择！= 4 ||选择== 4）' - 这总是'true'：/ – Maroun

哦，我明白了！因为任何选择！= 4也会使1,2或3计数为！= 4 – user3304333

此外，当您切换（选择）时，此后不需要“if”。因为......你开关（选择）已经做到了。 – Maroun

我甚至不知道从哪里开始...有你的代码的多个错误：

while (choice != 4);

删除分号和包装你switch内{}

if (choice == 1)

没有必要在case内部放置if语句，这是多余的

if (choice != 4 || choice == 4){

这始终是真实的，而且前面已经说过，if在每个case代码块的结尾在case代码块

case 1: 

if (choice == 1) {   
    System.out.println("Enter a percentage to multiply by"); 
    percent = input.nextFloat(); 

}

多余的，你需要放置一个break;声明

来源

2014-02-17 14:52:42 Cirou

谢谢！我还有另外一个问题，无论何时按1并输入百分比，我将如何返回4个选项的菜单？现在它只是循环“输入一个百分比乘以” – user3304333

发生这种情况是因为你的'choice'变量的值永远不会改变，所以你进入一个无限循环，其中'choice'变量总是等于1 ... I认为你应该对你的代码的逻辑做一些修改 – Cirou

我在想我可以做一个反控制的循环，所以一旦他们输入一个百分比，它就会知道，然后它可以返回到菜单后，因为这是所有他们会做。 – user3304333

在每种情况下，你应该把一个“休息”声明。

case 1: 

     if (choice == 1) {   
      System.out.println("Enter a percentage to multiply by"); 
      percent = input.nextFloat();

因此，与其把：

case 1: 

     if (choice == 1) {   
      System.out.println("Enter a percentage to multiply by"); 
      percent = input.nextFloat(); 
      break;

这是我看到的唯一问题。如果您有任何其他问题，请回答此问题。

来源

2014-02-17 14:44:25 user3319502

删除'if'请，它是多余的。 – Maroun

我相应地改变了它，虽然我以前有断言，并删除它们，看看它是否可以解决问题（Idk XD）但即使我删除了if语句，当我运行该程序时，问题仍然存在，输入一个数字，它显示数字，然后显示一个空格，我必须按Control-C退出该程序。 – user3304333

我在修改之前加了一个{在开关之前！ – user3304333

命令提示冻结后按回车，无效循环

回答

相关问题