Euklid Funktion

[scmo]

ich bin hier an der Berechnung des ggt's nach Euklid. und dan noch der den kgv.

Problem, i will 2 Zahlen mittels Scanner einlesen. Wenn ein Buchstabe geschriben wird, dann soll kommen, bitte zahl eingeben. und nochmals nach der zahl gefragt werden.
Das Funktioniert auch alles. Jedoch! Wenn es falsch eingabe und dann richtig. kommt ein fehler!
Kann mir sagen, was da genau passiert.
Warum der Inputstring immernoch der alten Eingabe entspricht.

// name ist nur der name der zahl
public int getParam(String name){
		String tmp = "";
		Scanner in = new Scanner(System.in);
		boolean isNum = false;
		
		System.out.println("Geben Sie ihre  Zahl für "+ name+" ein: ");
		tmp  = in.nextLine();
		
		if(this.isParsableToInt(tmp)==false){
			this.getParam(name);
		}
		return Integer.parseInt(tmp);
	}
	public boolean isParsableToInt(String i){
		try{
			Integer.parseInt(i);
			return true;
		}
		catch(NumberFormatException nfe){
			return false;
		}
	}

Eingabe:
Geben Sie ihre Zahl für a ein:
1
Geben Sie ihre Zahl für b ein:
a
Geben Sie ihre Zahl für b ein:
2
Fehler meldung:

Exception in thread "main" java.lang.NumberFormatException: For input string: "a"
	at java.lang.NumberFormatException.forInputString(NumberFormatException.java:48)
	at java.lang.Integer.parseInt(Integer.java:449)
	at java.lang.Integer.parseInt(Integer.java:499)
	at Euklid.getParam(Euklid.java:33)
	at Euklid.<init>(Euklid.java:15)
	at Euklid.main(Euklid.java:99)

 

[Marco01_809]

        if(this.isParsableToInt(tmp)==false){
            return this.getParam(name);
        }

Müsste das eigentlich schon lösen. (Nur ein return eingefügt)

 

[scmo]

öh merci! funktioniert!
wenn man es einfach betrachtet gibt es bei jedem buchstaben ein return mehr
also 3 mal falsch wäre:
return return return return (int)

oder wie muss ich das verstehen?

mit

this.getParam(name);

wollte ich eigentlich einfach nochmals die ganze funktion aufrufen.

 

[Marco01_809]

Wenn das return fehlt ist das Problem, dass du einfach die Funktion erneut aufrufst, das Ergebnis dann aber im Nichts verschwinden lässt und gleich daunter versuchst du den alten, fehlerhaften String zu parsen

 

[Final_Striker]

Das Funktioniert auch alles. Jedoch! Wenn es falsch eingabe und dann richtig. kommt ein fehler!

Ich würde das an deiner Stelle iterativ mit einer Schleife machen, die so lange wiederholt wird, bis eine richtige Eingabe erfolgt als rekursiv.

 

[scmo]

Ich würde das an deiner Stelle iterativ mit einer Schleife machen, die so lange wiederholt wird, bis eine richtige Eingabe erfolgt als rekursiv.

also mit einer while schleife?

 

[Final_Striker]

also mit einer while schleife?

Ja, mit einer while-Schleife.

 

[scmo]

Ja, mit einer while-Schleife.

while(isNum == false){
			System.out.println("Geben Sie ihre  Zahl für "+ name+" ein: ");
			tmp  = in.nextLine();
			if(this.isParsableToInt(tmp)==true){
				isNum = true;
			}
		}
		
		return Integer.parseInt(tmp);

Jap! Passt! Vielen Dank!

 

[Landei]

[c]while(isNum == false)[/c] ist gleich [c]while(! isNum)[/c] und [c]if(this.isParsableToInt(tmp)==true)[/c] ist gleich [c]if(this.isParsableToInt(tmp))[/c]