whileループ内での文のエラー

Question

私は、応答をコンパイルしてデータを出力する小さなプログラムに対して、事前テスト条件としてwhileループを使用する方法について読んでいますが、何を入力しても問題はありません入力ボックスは無効であると私に伝えます。私は何が間違っているのか分かりません。関連するコードは次のとおりです。

import javax.swing.JOptionPane; 

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


     int agree = 0; 
     int disagree = 0; 
     int neutral = 0; 
     int totalVotes = 0; 
     int input; 
     String inputString; 


     inputString = JOptionPane.showInputDialog("Response: \n" + 
        "(1=agree, 2=disagree, 3=no opinion, -1=exit)"); 
     input = Integer.parseInt(inputString); 

     while (input != -1) 
     { 
      if (input == 1) 
      { 
       agree += 1; 
       totalVotes += 1; 
      } 
      if (input == 2) 
      { 
       disagree += 1; 
       totalVotes += 1; 
      } 
      if (input == 3) 
      { 
       neutral += 1; 
       totalVotes += 1; 
      } 
      else { 
       JOptionPane.showMessageDialog(null, "invalid response " 
             + input); 
      } 
     } 


    } 
} 

Answer 1

elseが適切に使用されていないためです。あなたのコードを見れば、あなたの最終ifがあれば入力！= 3、無効な応答を示し

if (input == 3) 
     { 
      neutral += 1; 
      totalVotes += 1; 
     } 
     else { 
      JOptionPane.showMessageDialog(null, "invalid response " 
            + input); 
     } 

意味があります。

これを修正するには、ifをelse if (input == 2) ...（== 3と同じ）に変更します。

Answer 2

スティーブが指摘している通り、はが正しく配置されていません。私はをの場合はスタンドアロンのifsの代わりに入れようとしていたと思います。 import javax.swing.JOptionPane;

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


     int agree = 0; 
     int disagree = 0; 
     int neutral = 0; 
     int totalVotes = 0; 
     int input; 
     String inputString; 


     inputString = JOptionPane.showInputDialog("Response: \n" + 
       "(1=agree, 2=disagree, 3=no opinion, -1=exit)"); 
     input = Integer.parseInt(inputString); 

     while (input != -1) 
     { 
      if (input == 1) 
      { 
       agree += 1; 
       totalVotes += 1; 
      }else if (input == 2) 
      { 
       disagree += 1; 
       totalVotes += 1; 
      } else if (input == 3) 
      { 
       neutral += 1; 
       totalVotes += 1; 
      } 
      else { 
       JOptionPane.showMessageDialog(null, "invalid response " 
            + input); 
      } 
     } 

    } 
} 

Answer 3

あなたが知っているので、入力がの場合は、もし第一および他の最終とともに、それ以外の使用すべきであると同時に、1 3に等しくすることができません。あなたの現在のコードは、入力が1であるかどうかをチェックします。 2と等しいかどうかを確認しますが、前のステートメントは入力が1に等しいと結論づけました。したがって、== 2、または== 3をチェックする必要はありません。if/elseを使用すると、一緒に連鎖するときには単一の条件を満たすだけです。条件を満たす条件にヒットしたら、残りの条件はスキップします。

if (input == 1) 
{ 
    agree += 1; 
    totalVotes += 1; 
} 
else if (input == 2) 
{ 
    disagree += 1; 
    totalVotes += 1; 
} 
else if (input == 3) 
{ 
    neutral += 1; 
    totalVotes += 1; 
} 
else { 
    JOptionPane.showMessageDialog(null, "invalid response " + input); 
}