Javaスキャナ入力の有効化

Question

私は自分の入力から3つの数字、1、2、3を受け入れようとしています。それ以外のものは無効でなければなりません。私は方法を作りましたが、なぜそれがうまくいかなかったのか分かりません。何を変えなければならないのですか？

int number; 
do { 
    System.out.println("Enter 1, 2 or 3"); 
    while (!scanner.hasNextInt()) { 
     System.out.println("Invalid input!"); 
    } 
    number = scanner.nextInt(); 
} while (number == 1 || number == 2 || number == 3) 
return number; 

Answer 1

あなたのループロジック

do { 
    ... 
} while (number == 1 || number == 2 || number == 3); 

限り答えは有効あるとして用ループでの滞在が必要です。あなたはあなたの状態を反転したい：

do { 
    ... 
} while (!(number == 1 || number == 2 || number == 3)); 

または個々のコンポーネントを反転するDe Morgan's Lawを使用します。

do { 
    ... 
} while (number != 1 && number != 2 && number != 3); 

また、ScannerのhasNextInt戻りfalseは、あなたがスキャナをオフ無効な入力を取る必要がある場合たとえば、あなたが無視するnextLineとなります。そうでなければ、無限ループになります：

while (!scanner.hasNextInt()) { 
    System.out.println("Invalid input!"); 
    scanner.nextLine(); // This input is ignored 
}