One if statement taking precedence over another if, despite it being lower?

Question

I'm trying to make an if statement that would allow the user to exit the program by typing quit. My problem lies in that another if statement I have in the same block, albeit lower down, catching to ensure that the string entered is no longer than 3 characters long (required for the program) and that if statement takes precedence and cancels out the "quit" if statement. What am I doing wrong that is causing this?

        System.out.print("Guess #"+numGuess+": ");
            guess = input.nextLine();
            guess.toUpperCase();
        if(guess == "quit" || guess == "QUIT"){
            done = true;
            System.out.println("You lose.");
            display();
        }
        if(guess.length() != 3){
            while(guess.length() < 3){
                System.out.println("Guess too short");
                System.out.print("Guess #"+numGuess+": ");
                guess = input.nextLine();
                guess.toUpperCase();
            }
            while(guess.length() > 3){
                System.out.print("Guess too long");
                System.out.print("Guess #"+numGuess+": ");
                guess = input.nextLine();
                guess.toUpperCase();
            }
        }

Use `else if(guess.length() != 3){` ... use `else if` to make sure that both the quit and 3 length conditions are not both evaluated. — Tim Biegeleisen, Nov 15 '16 at 02:03

score -1 · Accepted Answer · answered Nov 15 '16 at 02:06

-1

Keep things simple:

Make your quitting if block return; to exit your program (after disply();)
Don't use == to compare strings. Use .equals() or better .equalsIgnoreCase()

answered Nov 15 '16 at 02:06

Bohemian

412,405
93
575
722

score -1 · Answer 2 · edited Nov 15 '16 at 02:09

-1

String comparison if(guess == "quit" || guess == "QUIT") in your code is wrong. If you want to compare the contents of a string you have to use .equals() method. == compares the references not the contents.

Change if(guess == "quit" || guess == "QUIT") to if(guess.equalsIgnoreCase("QUIT"))

edited Nov 15 '16 at 02:09

OneCricketeer

179,855
19
132
245

answered Nov 15 '16 at 02:08

v3nM

952
1
12
19

Who ever down-voted an answer should justify a reason behind it. If the answer is wrong, then a comment along with that is a very reasonable and acceptable action. I simply don't like my answer down-voted just because the the question is a duplicate. That does not serve the purpose of down-voting. Very misleading. Whoever did this should atleast justify – v3nM Nov 16 '16 at 16:49

score -1 · Answer 3 · answered Nov 15 '16 at 02:14

Try simplifying the code a bit.

 while (guess.length() != 3) {
    System.out.print("Guess #"+numGuess +": ");
    guess = input.nextLine();
    guess = guess.toUpperCase();

    if(guess.equalsIgnoreCase("quit")){
        done = true;
        System.out.println("You lose.");
        display();
        break;
    }

    if (guess.length() < 3){
        System.out.println("Guess too short");
    }
    else if (guess.length() > 3){
        System.out.print("Guess too long");
    }
   numGuess++;
}

One if statement taking precedence over another if, despite it being lower?

3 Answers3