return funktioniert nicht bzw. wird ignoriert

[automatix]

Hallo Forum!

So sieht die ursprüngliche Methode aus (ich hab sie aus dem Web):

	public static char readChar() {
		BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
		String line="";
		System.out.flush();
		try {
			line=reader.readLine();
			if (line.length()>0) return line.charAt(0);
		} catch (Exception e) {
			System.out.println(e);
		}
		return ' ';
	}

Der User soll eine Eingabe machen, das erste Zeichen der Zeichenkette ist der Rückgabewert der Methode:

		InPut myInput_1 = new InPut();
		String inputString = myInput_1.readString();
		System.out.println(inputString);

Soweit so gut.

Nun lautet die Aufgabe, die Methode so umzubasteln, dass eine gültige Eingabe (ein Character) erzwungen wird.

	public static char readChar(boolean enforce) {
		System.out.println("Versuch");
		if(!enforce) {
			readChar();
		} else {
			BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
			String line="";
			System.out.flush();
			try {
				line=reader.readLine();
				if (line.length()!=1)
				{
					readChar(true);
				} else {
					System.out.println(line.charAt(0) + "-bebebe");
					return line.charAt(0);
				}
			} catch (Exception e) {
				System.out.println(e);
			}
		}
		return 'r';
		
	}

Die Methode guckt, ob die eingegebene Zeichenkette ein Zeichen lang ist; falls ja, soll das Zeichen an der nullten Stelle (also das einzige Zeichen) der Zeichenkette zurückgegeben werden; falls nein, wird die Methode wieder aufgerugen.

Der Code in der Main-Classe:

		ReadChar myInput_2 = new ReadChar();
		char inputChar = myInput_2.readChar(true);
		System.out.println(inputChar);

Nun gibt es ein Prblem: die return-Anweisung im else-Block wird ignoriert. Nur, wenn gleich beim ersten Versuch eine gültige Eigabe gemacht wurde, kriegt man das Eingegebene auch als return-Wert -- ansonsten wird die return-Anweisung im esle-Block übersprungen und man bekommt dem entsprechend immer den Standard-Rückgabewert ('r') zurück... (Es wird nicht der ganze else-Block übersprungen -- s. syso(bebebe) -- sondern nur das return.)

Was mache ich falsch?

Danke.

 

[Campino]

Nein, die return- Anweisung wird nicht ignoriert.
Bei erneutem Aufruf ist enforce ist enforce true, das Ding geht also nicht in den if- Teil (was soll der eigentlich bringen? Bei enforce=false wird im Moment IMMER 'r' zurückgegeben. Ich nehme an, dass du nicht verstehst, warum? Selber Fehler wie im else- Teil).
Im else- Teil liest er ein, prüft wie lang die Kette ist und gibt entweder das Zeichen aus ODER ruft neu auf. Die neu aufgerufene Methode gibt jetzt das Zeichen in die ursprüngliche Methode zurück, die verarbeitet es aber NICHT, sondern verlässt den else- Teil, erreicht also das return 'r'. Es fehlt schlicht und einfach ein weiteres return. Wo, musst du jetzt selbst herausfinden

 

[automatix]

Hallo! Danke für die prompte Antwort!

> was soll der eigentlich bringen?
Wird die Methode ohne Parameter bzw. mit enforce=false aufgerufen, so soll die überladene parameterlose Methode readChar aufgerufen werden. Das ist die Idee. Du hast aber Recht -- das wird so nicht funktionieren, man brauch eine return-Anweisung in Zeile 4:

			return readChar();

> Selber Fehler wie im else- Teil

Und wieder muss man Dir zustimmen... Zeile 13:

					return readChar(true);

Jetzt läuft alles. Vielen Dank!

 

[automatix]

Heute ist nicht mein Tag... Das ist jezt schon der zweite Doppelbeitrag... Liebe Moderatoren, würden Sie den bitte löschen. Danke.