Problem mit dem initialisieren meines Strings in einer Schleife

[underio]

Hallo 🙂,
Das Programm rechnet sich die Ziffernsumme der eingegebenen Zahl aus und soll nach der Berechnung fragen ,ob man eine neue Zahl ausrechnen möchte.
Sobald man "ja" schreibt ,sollte es das in den String "yesJa" speichern. Wenn "yesJa" dann "ja" wäre ,sollte sich die Schleife wiederholen und immer so weiter gehen bis man am Ende etwas anderes schreibt als "ja". Das Programm an und für sich läuft und berechnet alles richtig.
Jedoch lässt es mich nach der Zeile 27 nichts mehr machen/eingeben und beendet den Vorgang einfach(es läuft immer nur einmal).

Wie kann ich dieses Problem beheben oder bessergesagt wie kann ich das Programm möglichst einfach richtig zum Funktionieren bringen?
Danke schonmal für die Hilfe 👍

[CODE lang="java" highlight="29"]import java.util.Scanner;

public class Ziffernsumme{

public static void main(String[] args) {

Scanner sc = new Scanner(System.in);

int number;
int x;
int xOutcome = 0;
String yesJa;


do{
System.out.print("Please enter a number: ");
number = sc.nextInt();
System.out.format("\n%1$d:Ziffernsumme = ", number);
while(number > 10){
x = number % 10;
number = number / 10;
System.out.print(x + " + ");
xOutcome = xOutcome + x;
}
xOutcome = xOutcome + number;
System.out.println(number + " = " + xOutcome );
System.out.println("Would you like to calculate another digit sum?");
yesJa=sc.nextLine();
}while(yesJa.equals("ja"));
}
}[/CODE]

 

[kneitzel]

Scanner kann Token oder Zeilenbasiert arbeiten. Wenn man das mischt, dann rennt man aber in gewisse Probleme.

Tokenbasiwrt bedeutet, dass der Scanner Token für Token ausliest und dann nachdem Token stehen bleibt. Das Trennzeichen bleibt dabei aber im Stream.

Was bedeutet das bei Dir?

Du gibst ein:
123<Enter>
nextInt liest nun das nächste Token:123
Das <Enter> bleibt im Stream drin.

Dann kommt dein nextLine Aufruf und da wird dann das Enter ausgelesen.

Um das zu umgehen könntest du ein nextLine Aufruf nach dem nextInt Aufruf durchführen.

 

[httpdigest]

Typisches Problem beim Mixen von tokenbasierter vs. zeilenbasierter Eingabe.
Ändere z.B. die Zeile:

number = sc.nextInt();

nach:

number = Integer.parseInt(sc.nextLine());

Der Grund: nextInt() liesst _nur_ die eingegebene Zahl, es bleibt aber noch ein Zeilenumbruch im Eingabepuffer.

EDIT: @kneitzel hat's schon gut erklärt. 🙂

 

[underio]

Scanner kann Token oder Zeilenbasiert arbeiten. Wenn man das mischt, dann rennt man aber in gewisse Probleme.

Tokenbasiwrt bedeutet, dass der Scanner Token für Token ausliest und dann nachdem Token stehen bleibt. Das Trennzeichen bleibt dabei aber im Stream.

Was bedeutet das bei Dir?

Du gibst ein:
123<Enter>
nextInt liest nun das nächste Token:123
Das <Enter> bleibt im Stream drin.

Dann kommt dein nextLine Aufruf und da wird dann das Enter ausgelesen.

Um das zu umgehen könntest du ein nextLine Aufruf nach dem nextInt Aufruf durchführen

Das Ding ist ,dass ich das ja nicht mal eingeben kann.... Vielen Dank für die Erklärung, jedoch verstehe ich sie nicht wirklich. Tut mir leid.

 

[kneitzel]

Stell Dir die Eingabe wie eine Warteschlange vor. Jeder Tastendruck ist wie eine Person, die wartet.

Der Aufruf nextLine bearbeitet alle Personen in der Schlange bis einschließlich der Return/Enter Taste.

Wenn Du also die "Personen" in der Schlange hast:
- 1 Taste
- 2 Taste
- 3 Taste
- Enter Taste
- ...
dann werden diese ganzen Tastendrücke gelesen und alles bis auf die Enter Taste landet im String: "123".

Der Aufruf nextInt arbeitet jetzt ähnlich, jedoch werden nur Zeichen gelesen bis zu einem sogenannten Whitespace (Enter, Tab, Leertaste, ...)
Daher werden nur
- 1 Taste
- 2 Taste
- 3 Taste
gelesen und der Rest
- Enter Taste
- ...
Bleibt in der Schlange.

Wenn nun bei der Schlange
- Enter Taste
- ...
nextLine aufgerufen wird, dann wird alles bis zur Enter Taste gelesen. Da aber sofort die Enter Taste dran ist, wird ein "" zurück gegeben.
Und da die Enter taste ja bereits in der Schlange ist, gibt es keinen weiteren Aufruf.

Daher zwei mögliche Lösungen:
a) nach dem nextInt noch ein nextLine aufrufen:

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

b) Statt dem nextInt generell nextLine aufrufen und die Eingabe selbst umwandeln:

number = Integer.parseInt(sc.nextLine());

 

[underio]

Stell Dir die Eingabe wie eine Warteschlange vor. Jeder Tastendruck ist wie eine Person, die wartet.

Der Aufruf nextLine bearbeitet alle Personen in der Schlange bis einschließlich der Return/Enter Taste.

Wenn Du also die "Personen" in der Schlange hast:
- 1 Taste
- 2 Taste
- 3 Taste
- Enter Taste
- ...
dann werden diese ganzen Tastendrücke gelesen und alles bis auf die Enter Taste landet im String: "123".

Der Aufruf nextInt arbeitet jetzt ähnlich, jedoch werden nur Zeichen gelesen bis zu einem sogenannten Whitespace (Enter, Tab, Leertaste, ...)
Daher werden nur
- 1 Taste
- 2 Taste
- 3 Taste
gelesen und der Rest
- Enter Taste
- ...
Bleibt in der Schlange.

Wenn nun bei der Schlange
- Enter Taste
- ...
nextLine aufgerufen wird, dann wird alles bis zur Enter Taste gelesen. Da aber sofort die Enter Taste dran ist, wird ein "" zurück gegeben.
Und da die Enter taste ja bereits in der Schlange ist, gibt es keinen weiteren Aufruf.

Daher zwei mögliche Lösungen:
a) nach dem nextInt noch ein nextLine aufrufen:

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

b) Statt dem nextInt generell nextLine aufrufen und die Eingabe selbst umwandeln:

number = Integer.parseInt(sc.nextLine());

Vielen vielen Danke. 🙂)