Loop error

[Spyderpig]

im trying to get this loop to work. if i dont enter an integer into floor it will go to the else statement and display: Error, not an integer and keep displaying that. How do i make it go back and ask for int floor again?

import java.util.Scanner;

public class ElevatorSimulation2 {

	public static void main (String[] args)
	{
		Scanner in = new Scanner(System.in);
		
		boolean done = false;
		while(!done)
		{
			System.out.println("Floor: ");
		
			if (in.hasNextInt())
			{
				int floor = in.nextInt();
				
				if (floor == 13)
				{
					System.out.println("Error: There is no 13th floor.");
				}
				else if (floor <= 0 || floor > 20)
				{
					System.out.println("Error: The floor must be between 1 and 20.");
				}
				else
				{
					int actualFloor = floor;
					if (floor > 13)
					{
						actualFloor = floor - 1;
					}
					System.out.println("The elevator will travel to the actual floor " + actualFloor);
					done = true;
				}
			}
			else
			{
				System.out.println("Error: Not an integer.");
							
			}
		}
	}
}

[pbrockway2]

If hasNextInt() returns false you had better print the message and consume the bad input that the scanner is looking at before you call hasNextInt() again. Otherwise that method will keep looking at the bad input and will keep returning false (as you have seen).

----------------

In general with Scanner methods, it pays to read the API documentation carefully taking note of BOTH what is returned AND where the scanner ends up looking after the method has returned.

[Spyderpig]

i see what you're saying but i still cant get it to work. i tried in.reset()

[al_Marshy_1981]

try reading in the bad data in your last else statement into a String.

String badData=in.next();

[pbrockway2]

It's not clear why you would use reset() as its documentation clearly says that what it resets are the radix/locale/delimiter settings, none of which you are using.

The first thing to be done is to have the user actually enter something. Clearly your program must stop and wait for their input before it can do anything else. Sorry, I hadn't noticed that your code wasn't doing this - it is racing ahead and calling hasNextInt() before the user has even had a chance to provide any input.

Try scrapping the existing while loop and proceeding along the following lines:

loop while not done

prompt and get user input (check the nextXXX() methods of scanner)

if input is not an integer give error message

else get int value that was entered and...

if it is outside the range give error message

else set actualFloor and make done true