methode soll int zurückgeben

[phobos]

Hallo, ich lerne seit kurzem Java und würde mich freuen wenn ihr mir bei meinem Problem weiterhelfen könnt.

	private static int intEingabe(String txt) {
		int eingabe=0;
		boolean fehler=true;
		Scanner sc = new Scanner(System.in);
		while (fehler) {
			fehler=false;
			try {
				System.out.print(txt);
				eingabe=sc.nextInt();			
			} catch (Exception e) {
				System.out.println("Du musst eine ganze Zahl eingeben.");
				fehler=true;
			    sc.nextLine();
			}
			return eingabe;
		}
			
	}

Ich bekomme die Meldung dass ich ein int zurückgeben muss, das tue ich doch aber imo.
woran liegt das und wie kann ichs beheben?

 

[Wishmaster51]

Du musst auch außerhalb der While-Schleife (d.h. nachdem diese abgelaufen ist) etwas zurückgeben.

 

[Gism0]

	private static int intEingabe(String txt) {
		int eingabe=0;
		boolean fehler=true;
		Scanner sc = new Scanner(System.in);
		while (fehler) {
			fehler=false;
			try {
				System.out.print(txt);
				eingabe=sc.nextInt();			
			} catch (Exception e) {
				System.out.println("Du musst eine ganze Zahl eingeben.");
				fehler=true;
			    sc.nextLine();
			}
			
		}
		return eingabe;	
	}

So gehts

 

[pro2]

Was halt vorher dein Fehler war:
Methoden müssen in jedem Fall etwas zurückgeben, wenn sie einen Rückgabewert haben.

Falsch:

int methode()
{
    int i = Math.random();
    if(2 > i)
        return i;
}

Gibt nicht zweifelsfrei etwas zurück.

Richtig:

int methode()
{
    int i = Math.random();
    if(2 > i)
        return i;
    else
        return 2;
}

Gibt immer etwas zurück.

Oder:

int methode()
{
    int i = Math.random();
    if(2 > i)
        return i;
    return -1;
}

Würde am Ende immer zumindest -1 zurückgeben, gibt natürlich vorher aber auch was anderes zurück, wenn i < 2 ist.

Das Gleiche gilt natürlich auch für alle Arten von Schleifen und Sonstiges, wo eine Rückgabe verhindert werden kann.

 

[phobos]

Danke
So einfach wenn mans weiss, da hab ich wohl die ganze zeit am falschen Ende überlegt und gesucht.

 

[hüteüberhüte]

Hallo, ich lerne seit kurzem Java und würde mich freuen wenn ihr mir bei meinem Problem weiterhelfen könnt. [...]

Hab das einfach mal kopiert:

    public static int intEingabe(String txt) throws IllegalStateException {
        Scanner sc = new Scanner(System.in);
        int result;
        while (true) {
            System.out.print(txt);
            try {
                result = sc.nextInt();
                break;
            } catch (InputMismatchException ime) {
            } catch (NoSuchElementException nsee) {
            }
            System.out.println("Du musst eine ganze Zahl eingeben.");
        }
        return result;
    }

System.out.println(intEingabe("Int eingeben: "));

Dabei kommt es jetzt bei mir/Netbeans zu der komischen Eigenschaft, dass der Scanner die Ausgabe wieder als Eingabe verwendet...

 

[phobos]

das selbe hatte ich auch.
Soweit ich dass verstanden habe kommt das daher weil bei

.nextInt()

der buffer nicht geleert wird.
Deswegen ist bei mir das

.nextline()

noch mit dabei. dann gehts.

 

[hüteüberhüte]

dann gehts.

Das Merkwürdige daran ist, dass es sich um zwei verschiedene Streams (System.in und System.out ) handelt. Eigentlich dürften solche "Anomalien" gar nicht sein. Vielleicht kann uns das jemand erklären

 

[phobos]

Der Fehler ist nicht im System-Stream sondern im Scanner-Buffer.
Wenn du nach Zeile 11

sc.nextLine();

eingibst dann funzt es ja.

siehe hier: Bug ID: 6351708 unempty buffer makes Scanner.nextLine as if it is skipped!

 

[hüteüberhüte]

siehe hier:

Also ist es doch ein Bug! Genau, wie ich vermutet hatte:

STEPS TO FOLLOW TO REPRODUCE THE PROBLEM :
in the implementation of the following methods: Scanner.nextInt(), Scanner.nextFloat() and Scanner.nextDouble(), you must clear the buffer from the new line character entered by a user.

:

result = sc.nextInt();
sc.nextLine();

Funktioniert aber auch nicht, oder ich habe etwas immernochnicht verstanden ???:L

 

[...ButAlive]

Ne ein Bug ist es nicht, sondern ein Feature. Sonst könnte man nicht sowas bauen:

Scanner scanner = new Scanner(System.in);
		
try {
   int i = scanner.nextInt();
} catch (InputMismatchException e) {
   System.out.println("Deine Eingabe "+scanner.nextLine()+" ist kein Integer");
}

Wenn der Nutzer etwas falsches eingibt, hätte man keine Chance in einer Fehlermeldung/Log die eigentliche Eingabe zu protokolieren, in dem Fall dass der Puffer geleert werden würde.

 

[hüteüberhüte]

Warum funktioniert dann

    public static int intEingabe(String txt) throws IllegalStateException {
        Scanner sc = new Scanner(System.in);
        int result;
        while (true) {
            System.out.print(txt);
            try {
                result = sc.nextInt();
                sc.nextLine();
                break;
            } catch (InputMismatchException ime) {
            } catch (NoSuchElementException nsee) {
            }
            System.out.println("Du musst eine ganze Zahl eingeben.");
        }
        return result;
    }

nicht? Probier doch mal aus, einfach

 

[...ButAlive]

Weil durch

result = sc.nextInt();

eine Exception fliegt und so niemals die nächste Zeile mit

sc.nextLine();

erreicht wird. Wenn du

sc.nextLine();

in den Catch-Block oder danach einbaust funktioniert es.

 

[hüteüberhüte]

Ach so! Du meinst das so oder?:

    public static int intEingabe(String txt) throws IllegalStateException {
        Scanner sc = new Scanner(System.in);
        int result;
        while (true) {
            System.out.print(txt);
            try {
                result = sc.nextInt();
                break;
            } catch (InputMismatchException ime) {
            } catch (NoSuchElementException nsee) {
            } finally {
                sc.nextLine();
            }
            System.out.println("Du musst eine ganze Zahl eingeben.");
        }
        return result;
    }
    
    public static void main(String[] args) {
        System.out.println(intEingabe("Int eigeben: "));
    }

Damit funktionierts jetzt. Danke!