Question

I don’t understand what is going on with this switch statement. I’m trying to have the user input l for less and m for more. If they put less then it would do the calculations under the ‘l’ case only. But this seems to be asking the user for input from the ‘l’ case AND going to the ‘m’ case and asking again so the user has to input l l for less and m m for more instead of just once.

Is this a problem with trying to use switch or a problem with the way I wrote the code?

        while (choice == console.next().charAt(0));

EDIT: Figured out this was asking for input instead so it kept asking twice each time.

Answer 1

You’re asking for choice twice, once in the case statements:

        case 'l':   //If it is lower then 499 becomes the upperBound
            upperBound = nextGuess;
            tries++;
            nextGuess = ( (lowerBound + upperBound)/2 );
            System.out.println("Is it " + nextGuess + 
                 "? \n\t Enter y if it is, l if it is less, or m if it is more.");

            // **** here ****
            choice = console.next().charAt(0);

            System.out.println("LOWER");
            break;

and again in the while condition, i.e.,

    while (choice == console.next().charAt(0));

Solution: Get the input only once.