CODEWITHSUNDEEP
java
fdasf
fdasf

Reputation: 31

java user validation for string input until correct input is given

How would I use a while loop to keep asking the user to type a valid answer repeatedly until a valid input is given and end the program when a valid input is given? I only know how to use int and numbers. I am confused by letters. How should I apply the NOT operator or other logical operators || &&.

Scanner myScan = new Scanner(System.in);                                        
System.out.println("Enter food");                                       
String food = myScan.nextLine();

if (food.equalsIgnoreCase("b"))
{
    System.out.println("beans");
}
else if (food.equalsIgnoreCase("e"))                                        
{
    System.out.println("eggs");
}
else
{
    System.out.println("error");
}

Upvotes: 2

Views: 5522

Answers (4)

Bishoy Kamel
Bishoy Kamel

Reputation: 2355

you also can use Map and feel no worries about adding more food

    Scanner myScan = new Scanner(System.in);
    System.out.println("Enter food");
    String food;

    Map<String, String> foodMap = new HashMap<>();
    foodMap.put("e", "Eggs");
    foodMap.put("b", "Beans");
    // add more food
    while (true) {
        food = myScan.nextLine();
        String value = foodMap.get(food);
        if (value == null) {
            System.out.println("Error!");
            break;
        }
        System.out.println(value);
    }

Upvotes: 0

MadProgrammer
MadProgrammer

Reputation: 347314

At a very simple level, I'd use a do-while loop, as you want to enter the loop at least once. I'd then determine the validity of the input, using a boolean flag and make further determinations based on that, for example...

Scanner myScan = new Scanner(System.in);
boolean userInputCorrect = false;
String food = null;
do {
    System.out.println("Enter food");
    food = myScan.nextLine();
    userInputCorrect = food.equalsIgnoreCase("b") || food.equalsIgnoreCase("e") || food.equalsIgnoreCase("exit");
    if (!userInputCorrect) {
        System.out.println("Error");
    }
} while (!userInputCorrect);
System.out.println("You selected " + food);

An expanded solution might use some kind of valid method, into which I can pass the String and have it validate the input based on known values, but that's probably a little beyond the scope of the question

As has been, correctly, pointed out but others, it would be more efficient to convert the input to lowercase once and compare all the values, in this case, it might be better to use a switch statement...

Scanner myScan = new Scanner(System.in);
boolean userInputCorrect = false;
String food = null;
do {
    System.out.println("Enter food");
    food = myScan.nextLine();
    switch (food.toLowerCase()) {
        case "b":
        case "e":
        case "exit":
            userInputCorrect = true;
            break;
        default:
            System.out.println("Error");
    }
} while (!userInputCorrect);
System.out.println("You selected " + food);

But you could also do...

food = myScan.nextLine().toLowerCase();
userInputCorrect = food.equals("b") || food.equals("e") || food.equals("exit");

Upvotes: 2

Yusof Bandar
Yusof Bandar

Reputation: 181

First of all its bad practice to use flags, what you should use is something called a Do while Loop.

do { 
//block of code to be executed
} while(booleanExpression);

A do while loop will excute the code within the do block then checks the while expression.

For your case you could do something like this:

Scanner myScan = new Scanner(System.in);  
String food;
do{
  System.out.println("Enter food");                                       
  food = (myScan.nextLine()).toLoweCase();
}while(!food.equals("b") || !food.equals("c"))

if (food.equals("b"))                                        
{                                       
 System.out.println("beans");                                      
}                                       
else if (food.equals("e"))                                        
{                                       
 System.out.println("eggs");                                       
}

For future proofing you might have a problem if the number of foods continues to grow. Consider using an Array or Array List

Upvotes: 0

paulsm4
paulsm4

Reputation: 121799

One approach would be to define a "flag" variable, then loop until it's "true". For example:

  Scanner myScan = new Scanner(System.in);
  boolean done = false;
  while (!done) {                                        
    System.out.println("Enter food");                                       
    String food = myScan.nextLine().toLowerCase();                                        
    switch (food) {
       case "b" :
         System.out.println("beans");                                      
         done = true;
         break;
       case "e" :
         System.out.println("eggs");                                       
         done = true;
         break;
       ...
       default :
         System.out.println("error");                                      
     }    
  }

Upvotes: 1

Related Questions