私は最近、スキャナーが期待した結果をもたらさないという質問を投稿し、私の問題はスキャナーを.nextLine(). スキャナを適切にフラッシュしているため、作業中のプログラムと混同していますが、プログラムをテストするときに、数字の入力を求められたときに文字列を入力すると、誤った出力が表示されます。同じループを 2 回繰り返します。
ループの先頭には nextLine() の呼び出しがあり、入力中の文字列などの無効な入力を処理する else ブロックにも nextLine() の呼び出しがあります。しかし、それでもどういうわけか私は悪い出力を得ています
具体的に言うと、ユーザー入力を太字で示し、問題のある出力をイタリック体で示した悪い出力のサンプルを次に示します。
左側の値を入力してください: 2
演算子を入力してください: -
右側の値を入力してください: t
無効入力
演算子を入力してください(+ - * または / :
無効な演算子
演算子を入力してください(+ - * または / :
上記の 4 行は、コンソールに自動的に吐き出されます。
これはコード スニップです。悪いコードの場所には大きなコメントが付いています。問題のある while ブロックだけを投稿したはずですが、while ブロックはプログラムの大部分を占めており、プログラム全体はそのセクションよりも少し大きいだけなので、すべてを投稿する方がよいと考えました。
import java.util.Scanner;
import javax.swing.JOptionPane;
public class Calculator{
public static void main(String[] args){
double leftHandVal = 0.0;
//Output Title & Instructions
System.out.print("!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!\n");
System.out.print("!\t\t\t\t!\n");
System.out.print( "!\t INSTRUCTIONS\t\t!\n");
System.out.print("!\t\t\t\t!\n");
System.out.print("! INPUT\t\tOUTPUT\t\t!\n");
System.out.print("! *******\t *********\t\t!\n");
System.out.print("! c or C\t\tClear\t\t!\n");
System.out.print("! q or Q\t\tQuit\t\t!\n");
System.out.print("! +\t\tAddition\t\t!\n");
System.out.print("! -\t\tSubtraction\t!\n");
System.out.print("! *\t\tMultiplication\t!\n");
System.out.print("! /\t\tDivision\t\t!\n");
System.out.print("!\t\t\t\t!\n");
System.out.print("!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!\n\n");
while(true){
Scanner input = new Scanner(System.in);
char op = '\n';//(+, -, *, or /) will use in switch statement for their ascii decimal values
System.out.print("Enter the left-hand value: ");
//these blocks allow the code at the very bottom to not erroneously ask the user for extra input with hasNext() calls
if(input.hasNext("c") || input.hasNext("C")){//even though its unlikely for a user to clear so early...just in case
leftHandVal = 0.0;
}
else if(input.hasNext("q") || input.hasNext("Q")){//even though its unlikely for a user to quit so early...just in case
op = 113;//assign q for quit code
}
else if(input.hasNextDouble()){
leftHandVal = input.nextDouble();
/*
*
*BAD CODE INSIDE WHILE BELOW
*BAD CODE INSIDE WHILE BELOW
*BAD CODE INSIDE WHILE BELOW
*BAD CODE INSIDE WHILE BELOW
*
*/
while(true){
input.nextLine();
double rightHandVal = 0.0;
System.out.print("\nEnter operator (+ - * or / : ");
if(input.hasNext()){
op = input.next().charAt(0);
}
//if user wishes to cancel or quit on operator prompt, break out of inner while to access the clear and quit code
if(op == 99 || op == 67){
op = 99;
break;
}
else if(op == 113 || op == 81){
op = 113;
break;
}
else if((op != 43) && (op != 45) && (op != 42) && (op != 47)){//if invalid operator, restart inner while
System.out.print("Invalid Operator");
continue;
}
System.out.print("Enter the right-hand value: ");
if(input.hasNextDouble()){
rightHandVal = input.nextDouble();
switch(op){
case 43:
System.out.printf("%.3f + %.3f = %.3f", leftHandVal, rightHandVal, (leftHandVal + rightHandVal));
leftHandVal += rightHandVal;
break;
case 45:
System.out.printf("%.3f - %.3f = %.3f", leftHandVal, rightHandVal, (leftHandVal - rightHandVal));
leftHandVal -= rightHandVal;
break;
case 42:
System.out.printf("%.3f * %.3f = %.3f", leftHandVal, rightHandVal, (leftHandVal * rightHandVal));
leftHandVal *= rightHandVal;
break;
case 47:
System.out.printf("%.3f / %.3f = %.3f", leftHandVal, rightHandVal, (leftHandVal / rightHandVal));
leftHandVal /= rightHandVal;
break;
}
}
//if clear or quit requested from prompt for right-hand value, break to reach the clear and quit code
else if(input.hasNext("c") || input.hasNext("C")){
op = 99;
break;
}
else if(input.hasNext("q") || input.hasNext("Q")){
op = 113;
break;
}
else{
System.out.print("Invalid Input");
}
}
}
//if c || C reset op to null and restart outer while
if(op == 99 || op == 67){
op = '\n';
leftHandVal = 0.0;
continue;
}
//else if q || Q, prompt user with a popup to confirm.
if(op == 113 || op == 81){
int response = JOptionPane.showConfirmDialog(null, "QUIT CALCULATOR?", null, JOptionPane.YES_NO_OPTION);
if(response == 0){
System.exit(0);
}
continue;
}
}
}
}