Endlosschleife - Warum?

[selina55555]

Hallo,

ich habe eine kurze Methode geschrieben, die dividieren soll.
Bei der Eingabe eines Zeichens, das keine Zahl ist soll eine Fehlermeldung kommen (per Exception). Dies soll auch über eine andere Exception passieren, falls durch 0 dividiert wird.

Beim Ausführen habe ich als erstes 8 und dann a eingetippt. Dabei entsteht eine Endlosschleife und die zugehörige Meldung bei InputMismatchException wird in einer Endlosschleife ausgegeben. Woran liegt das?

Ich möchte gerne, dass nach einer falschen Eingabe lediglich der Benutzer erneut aufgefordert wird eine Zahl einzugeben. Bei dem Versuch durch 0 zu dividieren funktioniert das auch. Nur bei InputMismatchException klappt es nicht.

Ich würde mich sehr freuen, wenn mit jemand behilflich sein kann, da ich den Fehler alleine nicht finden konnte.

import java.util.*;

public class main {

    public static void main(String[] args) {
    Scanner input = new Scanner (System.in);
    int x=1;
    do {
        //try this
       try {
            System.out.println("Enter your first number");
            int n1 = input.nextInt();
            System.out.println("Enter your second number");
            int n2 = input.nextInt();
            double diff = n1/n2;
            System.out.println(diff);
            x=2; //successful diff
        }
        //if you have an error do this
        catch (InputMismatchException e) {
            System.out.println("Only idiots try to divide by other things than numbers");
        }
       
       catch(Exception e){
            System.out.println("You can't do that.");
        }
    }
    while (x==1);
}
}

 

[Kababär]

Also was hier geschieht ist, dass du, wenn du a eintippst, in den ersten catch-Block gerätst.
Dann gehst du wieder in das try und versucht, wieder a einzulesen, da sich der Zeiger des Scanners noch nicht bewegt hat, weil er eben im ersten catch-Block gelandet ist.
Also um den Zeiger weiter zu bewegen und somit das wirklich nächste Zeichen einzulesen, müsstest du im catch-Block 'input.next();' aufrufen.

Aber das geht generell auch etwas eleganter, wie man mit einem Scanner einlesen kann.

Da sich x nie änder,t würde ich lieber ein boolean nehmen.
A la

boolean b = true
whiel(b){


}