Datentypen String.equals funktioniert nicht

[xant95]

Hallo, ich würde gerne machen dass wenn man als Input "0" eingibt, die Methode beendet wird, jedoch passiert nichts wenn ich "0" eingebe.
Ich habe bereits versucht die schleife bei "// FALLS DER NAME UNGÜLTIG IST" und die Methode "isDigit" wegzulassen, jedoch hatte das keinen Einfluss.

Danke im Vorraus! 👀

[CODE lang="java" highlight="6"]// BEGLEITUNGEN ERSTELLEN
public static void makePassenger() {
String userInputName = "";
String userInputEmailadr = "";

while (!userInputName.equals("0") && !userInputEmailadr.equals("0")) {
System.out.println("Gib bitte die Namen und die dazugehörigen Email-Adressen deiner Begleitungen ein: \n"
+ "Drücke 0 zum abbrechen!");
System.out.println("Name: ");
userInputName = usefullTools.scanString().toLowerCase();

// FALLS DER NAME UNGÜLTIG IST
while (userInputName.contains(".") || userInputName.contains("@") || usefullTools.isDigit(userInputName)) {
System.out.println("Ungültiger Name!\n"
+ "Gib bitte den Namen erneut ein: ");
userInputName = usefullTools.scanString().toLowerCase();
}



System.out.println("Email-Adresse: ");
userInputEmailadr = usefullTools.scanString().toLowerCase();

// FALLS DIE EMAIL-ADRESSE UNGÜLTIG IST
while (!userInputEmailadr.contains("@") && !userInputEmailadr.contains(".")) {
System.out.println("Ungültige Email-Adresse!\n"
+ "Gib bitte die Email-Adresse erneut ein: ");
userInputEmailadr = usefullTools.scanString().toLowerCase();
}

// FALLS DIE PERSON BEREITS EINGETRAGEN WURDE
for (int j = 0; j < bookedPassengerList.size(); j++) {
if (bookedPassengerList.get(j).getEmailadr().equals(userInputEmailadr)) {
System.out.println("Diese Person wurde bereits für diesen Flug eingetragen.");
} else {
// BEGLEITUNG HINZUFÜGEN
Passenger passenger = new Passenger(userInputName, userInputEmailadr);
Booking.getBookedPassengerList().add(passenger);
System.out.println("Danke!\n"
+ Booking.getBookedPassengerList().get(Booking.getBookedPassengerList().size() - 1)
.getName().substring(0, 1).toUpperCase()
+ Booking.getBookedPassengerList().get(Booking.getBookedPassengerList().size() - 1)
.getName().substring(1)
+ ", wurde erfolgreich hinzugefügt!\nWenn du fertig bist, gib bitte 0 ein!");
reservedSeats++;
}
}
}
}[/CODE]

 

[Staarfightaar]

kann mir vorstellen dass du als string ein "0\n" dabei hast also 0 mit line break oder end of line zeichen, lass dir mal die länge des strings ausgeben
wenn die nicht 1 ist wenn du 0 eingibst sollte es in die richtung gehen

 

[KonradN]

Die Logik ist doch unstimmig!

Spiel es doch af einem Zettel durch und übergege Dir: Wan wird denn auf die "0" geprüft? Und was wird mit der Eingabe ansonsten gemacht?

Also ein gutes Beispiel, dass aufzeigt, wieso man so Code auf Methoden aufteilen soll um eben den Überblick nicht zu verlieren.

 

[Jw456]

Du versuchst in der ersten while zu prüfen ob die Eingabe vom User "0" ist.
Nur zu dem Zeitpunkt hast du dem User noch garkeine Möglichkeit gegeben eine Eingabe zumachen. Also die Schleife wird auf jeden Fall einmal durchlaufen was du eigentlich nicht willst.

Also wo müsste denn die Eingabe vom User sein?

 

[Neumi5694]

Nein, sein Code ist zwar seltsam, passt grundsätzlich aber schon - zumindest was die "0" angeht.
Er prüft ja, ob die Eingabe nicht "0" ist und das trifft am Anfang zu.
Ist zwar etwas komisch, erfüllt aber die Absicht.

Problematisch ist, dass "0" Eingaben z.B. währen der Auswertung der Email-Adresse nicht akzeptiert werden.
"0" ist keine gültige Email-Adresse, also wird ewig weitergemacht.

Außerdem wird der Name nie eingelesen, er geht gar nicht in die Schleife rein, so lange der String leer ist.

@xant95
Die while Schleife, so wie du sie anwendest, kann dazu führen, dass der Code darin niemals ausgeführt wird.
Wenn etwas mindestens einmal gemacht werden soll, verwende eine do-while Schleife.

String name;
do {
    name = nameEinlesen();
} while (wasAuchImmerEineWiederholungAuslöst);
System.out.println(name);

Hier eine Version mit Eingabeaufforderung und Fehlermeldung

System.out.println("Bitte eingeben");
String name;
boolean wasValid;
do {
    name = nameEinlesen();
    wasValid = prüfungObGültigerWert(name);
    if (!wasValid) {
        System.out.println("Bitte nochmal eingeben");
    }
} while (wasValid);
System.out.println(name);

 

[KonradN]

Evtl. auch hier erneut einmal das generelle Vorgehen bei der Programmierung.

1. Schreibe das Verhalten auf. Dabei einfach aufschreiben, was getan werden soll. Das muss auch nicht zu sehr ins Detail gehen.
Das kann also hier etwas sein wie:

Eingabe von Name / EMail Kombinationen
In einer Schleife:
- Name := Eingabe eines gültigen Names oder "0" durch User.
- Wenn Name ein "0" ist, dann Abbruch.
- EMail := Eingabe einer gültigen EMail oder "0" durch User.
- Wenn EMail "0" ist, dan Abbruch
- ....

Das lässt sich dann eigentlich 1:1 so in Java übersetzen:

public static void inputOfNamesAndEmailCombinations() {
    String name = getValidNameOr0Input();
    if (name.equals("0")) return;

    String email = getValidEmailOr0Input();
    if (email.equals("0")) return;

    // ...
}

Jetzt sind die Methoden aber noch nicht geschrieben. Da kann dann jetzt noch etwas mehr rein. Das geht aber dann genau so!

Eingabe gültiger Name oder 0:
In einer Schleife:
- Name := Eingabe User.
- Ist Name = "0" -> Gib Name zurück
- Ist Name Valide? -> Gib Name zurück
- Ausgabe: Name war ungültig, bitte gültigen Namen eingaben oder 0 zum Abbruch

Und ist Name Valide ist jetzt wieder nicht genau angegeben. Kann man natürlich schon direkt abhandeln oder es ist jetzt eine neue Methode:

Ist Name Valide:
- Enthält der Name Zahlen? -> Gib Nein zurück.
- Enthält der Name ([]){}? -> Gib Nein zurück
- Gib Ja zurück.

Und so weiter. Du bekommst dann ganz viele kleine Methoden. Jede ist total enfach zu schreiben. Und wenn Du Dich mit Unit Tests beschäftigst, dann wird dies auch relativ einfach zu testen sein. (Naja - relativ. Eingabe und so ist blöd zu testen.)

Ud so gehst Du auch mit dem vor, was Du da schon gemacht hast:

Ist der Name schon vorhanden?
Ja: Fehler ausgeben und fertig.
Nein: Füge Name / Email in Liste dazu.

(Das ist halt etwas, das noch nicht angesprochen wurde, aber da hat Dein Code auch Probleme. In der schleife zur Suche kannst Du im Else noch nicht die andere Aktion machen. Dass der Name nicht vorkommt, weisst Du ja erst, wenn Du komplett Durchgegangen bist und nichts gefunden hast. ==> Das ist also ebenso ein typischen Zeichen, dass Du in etwas Code zu viel auf einmal packen wolltest und Du somit es schlicht zu kompliziert gebaut hast. Code sollte immer einfach bleiben!)