Problem mit try-catch in Schleife

[Cometcola]

Guten Morgen!
Ich habe ein kleines Problem, welches mich schon viel Zeit gekostet hat, ich vermute aber die Lösung sollte relativ simpel sein. Folgenes Programm soll mein Problem verdeutlichen:

Eingabe einer Zahl, dabei Prüfung ob ok und wenn nicht, dann so lange durchlaufen lassen, bis Eingabe korrekt ist. Bei Falscheingabe(Buchstaben, Text) an den Anfang zurück.

Danke! Gruß Cometcola

import java.util.InputMismatchException;
import java.util.Scanner;

public class Jahreszeit {

	public static void main(String[] args) {

		Scanner console = new Scanner(System.in);
		int day = 0;

		while(true){
			
			try {
				System.out.print("Tag eingeben (1-31):");
				day = console.nextInt();
				if (day > 31 || day < 1) {
					System.out.println("Bitte im Bereich [1,31]! Nochmal.");
					continue;
				} else {
					break;
				}
				
			} catch (InputMismatchException e) {
				System.out.println("Keine ganze Zahl. Bitte nochmal.");

			}
			
		}

 

[TryToHelp]

Mhh ich glaube ich bin zu blöd, ich verstehe grade deine Frage nicht, sieht doch so aus, als würde dein Code machen, was du willst, bei Zahlen größer 31 bzw kleiner 1 wiederholt er die Schleife

 

[Cometcola]

genau das funktioniert perfekt. Nur wenn ich jetzt Beispielsweise einen Buchstaben eingebe, springt er in eine Endlos-Schleife....und das möchte ich nicht

Das break und continue hab ich auch eher ausm Bauchgefühl da reingeschrieben und ich denke mal, dass da das Problem liegt. Habe gefühlt schon alles durchprobiert.

 

[TryToHelp]

ich würde die Abbruchbedingung eher in den While kopf anstelle

while(true)

setzen, aber das sollte es nicht ändern, also in der art

while(day > 31 || day < 1)

und im catch block day auf 0 setzen

 

[Cometcola]

die if Bedingung in den while Kopf setzen geht glaube ich nicht, da ich ja 2 unterschiedliche Meldungen habe, wenn ich bsp. eine Zahl falsch eingegeben habe. Oder kannst du das im Quellcode deutlich machen?

 

[Michael...]

Man könnte ein Flag nutzen, dass man in der Schleife setzt wenn eine korrekte Eingabe erfolgt ist. das while(true) und break ist unschön.
Bzgl. des Problems: Wenn keine Zahl eingeben wurde, versucht die nextInt() eine Ganzzahl einzulesen, schlägt fehl --> Exception.
Der Buchstabe steht aber immer noch im Stream und Scanner versucht wieder eine Ganzzahl zu lesen --> Exception, usw.
Eine schnelle nicht unbedingt schöne Lösung wäre im catch Block einfach die falsche Eingabe auszulesen.

 

[TryToHelp]

Mhh ok, ja deine Fehlerausgabe braucht das if, da habe ich nicht dran gedacht und wie gesagt, ich glaube das sollte auch nicht wirklich dein Problem lösen, nur verstehe ich nicht, wieso er bei einem Buchstaben in einer endlosschleife endet

was auch geht, wäre dann ein boolean zu setzen, ich finde es einfach unschön, mittels break zu arbeiten und das continue, kannst du glaube ich ganz weck lassen, das brauchst du nicht ;-)

[EDIT]Mhh zu langsam
Irgendwie hatte ich sowas schon vermutet, aber wusste nicht, ob das dann der Fall ist, oder ob der stream dann gelehrt wird, was aber wohl nicht der Fall ist[/EDIT]

 

[Cometcola]

ok danke erstmal, werd mal sehen wie ich damit vorankomme.

 

[ssoul26]

Lager das Einlesen in eine Methode aus und rufe diese jeweils auf.

   public static void main(String[] args) {
      // TODO Auto-generated method stub
      while (true) {
         einlesen();
      }
   }

   private static void einlesen() {
      try {
         Scanner console = new Scanner(System.in);
         int day = 0;
         System.out.print("Tag eingeben (1-31):");
         day = console.nextInt();
         if (day > 31 || day < 1) {
            System.out.println("Bitte im Bereich [1,31]! Nochmal.");
            einlesen();
         } else {
            System.out.println("Eingabe korrekt!");
         }
      } catch (InputMismatchException e) {
         System.out.println("Keine ganze Zahl. Bitte nochmal.");
      }
   }

 

[Cometcola]

ok danke! Werd mich mit dem Material mal weiter beschäftigen!

 

[ssoul26]

Du hast aber jetzt schon verstanden, wieso das Programm bei Falscheingabe in einer Endlosschleife endet, oder?

 

[TryToHelp]

Also dann so in der Art ;-)

public int readDay(){
        Scanner console = new Scanner(System.in);
        int day = 0;
        boolean correctDay=false; // flag zum anzeigen ob die eingabe wiederholt werden muss
        while(!correctDay){
            try {
                System.out.print("Tag eingeben (1-31):");
                day = console.nextInt();
                if (day > 31 || day < 1) {
                    System.out.println("Bitte im Bereich [1,31]! Nochmal."); // deine Fehlerausgabe
                } else {
                    correctDay=true; // setzen des flags
                }             
            } catch (InputMismatchException e) {
                System.out.println("Keine ganze Zahl. Bitte nochmal."); // deine 2. Fehlerausgabe
                console.next(); // bereinigen des streams
            }         
        }
        return day;
}

und deine main

public static void main(String[] args) {
      // TODO Auto-generated method stub
      int day=readDay();
   }

 

[TryToHelp]

@ssoul26 dein Programm ist ja eine reine Endlosschleife ;-) selbst bei korrekter eingabe

 

[ssoul26]

@ssoul26 dein Programm ist ja eine reine Endlosschleife ;-) selbst bei korrekter eingabe

Ja, als Test so gedacht! Also er nimmt immer nur Eingaben entgegen.

 

[Michael...]

console.next(); // bereinigen des streams

Hier besser

nextLine()

verwenden.

 

[Cometcola]

ja konnte es jetzt nachvollziehen. Nochmals danke an alle Helfer! Hat mich gestern Abend und heute Früh zum Verzweifeln gebracht.