Ist meine Schleife korrekt in Java eingerichtet?

Question

Ich muss codieren, um zwei verschiedene Fehlermeldungen für verschiedene Ausgaben zurückzugeben. eine für dann die Benutzereingabe ist leer und eine andere, wenn die Benutzereingabe nicht y oder n ist. Ich laufe auf das Problem meines Codes ein, der nur die Fehlermeldung für das nicht y oder n zurückgibt, weil ich diese Nachricht zurückgeben muss, wenn der Benutzer irgendetwas anders als y oder n eingibt. Der Code wird die richtige Fehlermeldung für den leeren Fehler einmal zurückgeben, aber danach wird nur die Fehlermeldung für die nicht y oder n zurückgegeben. irgendwelche Vorschläge, wie man das behebt?

while (choice.isEmpty()) 
{ 
    System.out.println("Error! This entery is required. Try again."); 
    choice = sc.nextLine(); 
} 

while (!(choice.equalsIgnoreCase ("y") || choice.equalsIgnoreCase ("n"))) 
{ 
    System.out.println ("Error! Please enter y, Y, n, or N. Try again "); 
    choice = sc.nextLine(); 
} 

Answer 1

Du bist wahrscheinlich mit einer Schleife besser dran:

while (!choice.equalsIgnoreCase("y") && !choice.equalsIgnoreCase("n")) { 
    if (choice.isEmpty()) { 
     System.out.println("Error! This entry is required. Try again."); 
    } else { 
     System.out.println("Error! Please enter y, Y, n, or N. Try again."); 
    } 

    choice = sc.nextLine(); 
} 

Answer 2

Ich glaube, Sie nur eine einzige Schleife brauchen hier:

String choice = ""; 
do { 
    choice = sc.nextLine(); 
    if (choice.equalsIgnoreCase("y") || choice.equalsIgnoreCase("n")) { 
     break; 
    } 
    else { 
     System.out.println ("Error! Please enter y, Y, n, or N. Try again "); 
    } 
} while (true); 

Beachten Sie, dass dieser Ansatz richtig alle Eingaben behandelt, einschließlich des ersten, wenn der Eingang noch nicht vorher definiert wurde.

Answer 3

Du hast nicht beschreiben, was Sie erreichen wollen, wenn es sich um eine Endlosschleife ist (solange sc.nextLine() kehrt etwas), wo die Benutzer nur eingeben ‚y‘ oder ‚n‘:

while((choice = sc.nextLine()) != null) { 
    if(choice.isEmpty()) { 
     System.out.println("Error! This entry is required. Try again."); 
    } else if(!choice.equalsIgnoreCase("y") && !choice.equalsIgnoreCase("n")) { 
     System.out.println("Error! Please enter y, Y, n, or N. Try again."); 
    } else { 
     // do whatever you need 
    } 
}