Thread: I'm wanting this code to jump out of the loop at case 2 of the switch statement

For some reason it won't jump out of it.

  System.out.println("\nWould you like to try to guess the number again?\n" +
                               "1.Yes\n" +
                               "2.No\n" +
                               "3.Exit the System");
            guessAgainPrompt=keyboard.nextInt();
 
            switch (guessAgainPrompt)
            {
               case 1:
               guessNumber();
               guessCount=+1;
               if (numberGuess==bonusNumber)
               {
                  guessCorrect();
                  condition=false;
               }
               else
               {
                  guessFail();
               }
               break;
 
               case 2:
               System.out.println("Thanks for playing the Game!");
               condition=false;
               break;
 
               case 3:
               System.out.println("Goodbye");
               condition=false;
               break;
            }
 
 
       }while (guessCount<=5 || condition==true);

Originally Posted by Norm

Can you explain what statement in the code is executing and not doing what you want it to do?

What variables control the looping? Do they have the correct values for the loop to end?

Here's an update of the code:

switch (guessAgainPrompt)
            {
               case 1:
               guessNumber();
               guessCount=+1;
               if (numberGuess==bonusNumber)
               {
                  condition=false;
                  guessCorrect();
 
               }
               else
               {
                  guessFail();
 
               }
               break;
 
               case 2:
               System.out.println("Thanks for playing the Game!");
               condition=false;
               break;
 
               case 3:
               System.out.println("Goodbye");
               condition=false;
               break;
            }
 
 
       }while (guessCount<=5 || condition==true);
      }

The loop will not end here, therefore, I am prompted with the prompt that is at the beginning of the loop.

if (numberGuess==bonusNumber)
               {
                  condition=false;
                  guessCorrect();
 
               }

The variables that are controlling the loop is the 'condition variable, I have it initialised at the top as static boolean condition=true;
The loop is set to continue while the condition is true, but because the above condition is false, shouldn't it jump out?

I want either of the variables to be true, but for now I just want to get the 'condition' one working. I can't understand why it won't jump out when the condition is false. Here is the full code:

 
 
import java.util.Scanner;
 
public class GuessNumber
 
{
 
  static String Name;
  static  int guesses=0,bonusNumber = (int) (Math.random()*10) ,numberGuess, Score=0, guessAgainPrompt, guessCount;
  static  boolean condition=true;
 
  static  Scanner keyboard = new Scanner(System.in);
 
   public static void guessCorrect()
   {
      Score =+10;
      System.out.println("CONGRATULATIONS & WELL DONE!!!!\nYou have won the game.");
 
   }
 
   public static void guessFail()
   {
      System.out.println("Sorry, wrong number,");
 
   }
 
   public static void guessNumber()
   {
      System.out.println("Please guess a number between 1-10");
      numberGuess=keyboard.nextInt();
 
   }
 
 
   public static void main(String[] args)
   {                                                                                        //prompt for name to personalise
      System.out.println("What is your name?");
      Name=keyboard.nextLine();
 
      guessNumber();            //prompt for user to guess the bonus number
 
 
      if (numberGuess==bonusNumber)
      {
         guessCorrect();
         condition=false;
 
      }
      else
      {
         guessFail();
 
       do
       {
 
            System.out.println("\nWould you like to try to guess the number again?\n" +
                               "1.Yes\n" +
                               "2.No\n" +
                               "3.Exit the System");
            guessAgainPrompt=keyboard.nextInt();
 
            switch (guessAgainPrompt)
            {
               case 1:
               guessNumber();
               if (numberGuess==bonusNumber)
               {
 
                  guessCorrect();
                  condition=false;
               }
               else
               {
                  guessFail();
 
               }
               break;
 
               case 2:
               System.out.println("Thanks for playing the Game!");
               condition=false;
               break;
 
               case 3:
               System.out.println("Goodbye");
               condition=false;
               break;
            }
 
 
       }while (guessCount<=5 || condition==true);
      }
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
   }//main
}//class

Originally Posted by Norm

Did you read the end of my last post?

OR means if EITHER are true
AND means BOTH must be true

Yeah, but I want OR because i just need either one to be true, || is the correct operator for OR right?

Its already set to true, if either are false, I want it to stop looping.