Problem with Java Scanner - Please Help!

[Kevin Behr]

Hey all - so I am creating a non-graphical version of the game Mastermind (which entails guessing a secret color code within the fewest amount of tries possible - here is a link containing an example [http://www.irt.org/games/js/mind/]).

With my game, you first choose for either a computer-generated secret code or a human entered code, and then the player is prompted to enter the first guess of the color sequence. It should go like this:

Color 1: ____ [ENTER]
Color 2: ____ [ENTER]
Color 3: ____ [ENTER]
Color 4: ____ [ENTER]

But insted, it goes like this...

Color 1: Color 2: ____ [ENTER]
Color 3: ____ [ENTER]
Color 4: ____ [ENTER]

The problem lies within my ConfirmYes() and my ConfirmNo() methods. Could you please try giving my code a run and maybe seeing where the problem lies? Thanks!

  
 
   import java.util.*;
   import java.util.Scanner;
 
    public class mastermind { 
    
      static final String line1[] = new String[4];
      static final String line2[] = new String[4];
      static final String line3[] = new String[4];
      static final String line4[] = new String[4];
      static final String line5[] = new String[4];
      static final String line6[] = new String[4];
      static final String line7[] = new String[4];
      static final String line8[] = new String[4];
      
      static final String clue1[] = new String[4];
      static final String clue2[] = new String[4];
      static final String clue3[] = new String[4];
      static final String clue4[] = new String[4];
      static final String clue5[] = new String[4];
      static final String clue6[] = new String[4];
      static final String clue7[] = new String[4];
      static final String clue8[] = new String[4];
         
      static final String code[] = new String[4];
      static final Scanner scan = new Scanner(System.in);
   
       public static void ComputerCode()
      {
         String colors[] = new String[6];
         colors[0] = "Red";
         colors[1] = "Orange";
         colors[2] = "Yellow";
         colors[3] = "Green";
         colors[4] = "Blue";
         colors[5] = "Purple";
      
         Random generator = new Random();
      
         for (int i = 0; i < 4; i++)
         {
            int ran = generator.nextInt(6);
            code[i] = colors[ran];
         }
      
      }
      
       public static void PrintBoard()
      {
         System.out.println();
         System.out.println("     B O A R D     --      C L U E S     ");
         System.out.println("     _________             _________     ");
         System.out.println();
         
         for (int i = 0; i < 4; i++)
         {
            line8[i] = "0";
            System.out.print("   " + line8[i]);
         }
      
         System.out.print("   -- ");
      
         for (int i = 0; i < 4; i++)
         {
            clue8[i] = "0";
            System.out.print("   " + clue8[i]);
         }
         
         System.out.println("");
         
         for (int i = 0; i < 4; i++)
         {
            line7[i] = "0";
            System.out.print("   " + line7[i]);
         }
      
         System.out.print("   -- ");
      
         for (int i = 0; i < 4; i++)
         {
            clue7[i] = "0";
            System.out.print("   " + clue7[i]);
         }
         
         System.out.println("");
         
         for (int i = 0; i < 4; i++)
         {
            line6[i] = "0";
            System.out.print("   " + line6[i]);
         }
      
         System.out.print("   -- ");
      
         for (int i = 0; i < 4; i++)
         {
            clue6[i] = "0";
            System.out.print("   " + clue6[i]);
         }
         
         System.out.println("");
         
         for (int i = 0; i < 4; i++)
         {
            line5[i] = "0";
            System.out.print("   " + line5[i]);
         }
      
         System.out.print("   -- ");
      
         for (int i = 0; i < 4; i++)
         {
            clue5[i] = "0";
            System.out.print("   " + clue5[i]);
         }
         
         System.out.println("");
         
         for (int i = 0; i < 4; i++)
         {
            line4[i] = "0";
            System.out.print("   " + line4[i]);
         }
      
         System.out.print("   -- ");
      
         for (int i = 0; i < 4; i++)
         {
            clue4[i] = "0";
            System.out.print("   " + clue4[i]);
         }
         
         System.out.println("");
         
         for (int i = 0; i < 4; i++)
         {
            line3[i] = "0";
            System.out.print("   " + line3[i]);
         }
      
         System.out.print("   -- ");
      
         for (int i = 0; i < 4; i++)
         {
            clue3[i] = "0";
            System.out.print("   " + clue3[i]);
         }
         
         System.out.println("");
         
         for (int i = 0; i < 4; i++)
         {
            line2[i] = "0";
            System.out.print("   " + line2[i]);
         }
         System.out.print("   -- ");
      
         for (int i = 0; i < 4; i++)
         {
            clue2[i] = "0";
            System.out.print("   " + clue2[i]);
         }
         
         System.out.println("");
         
         for (int i = 0; i < 4; i++)
         {
            line1[i] = "0";
            System.out.print("   " + line1[i]);
         }
      
         System.out.print("   -- ");
      
         for (int i = 0; i < 4; i++)
         {
            clue1[i] = "0";
            System.out.print("   " + clue1[i]);
         }
      
      }
   
       public static void ConfirmYes()
      {
         PrintBoard();     
         
         System.out.println();
         System.out.println();
         
         System.out.println("Please enter your first guess. You can choose out of five");
         System.out.println("colors: Red, Orange, Yellow, Green, Blue, or Purple");
         System.out.println();
         System.out.println("Note: Color choices are entered from left to right (on the board");
         
         System.out.println();
         
         System.out.print("Color #1: ");
         line1[0] = scan.nextLine();
         
         System.out.print("Color #2: ");
         line1[1] = scan.nextLine();
         
         System.out.print("Color #3: ");
         line1[2] = scan.nextLine();
         
         System.out.print("Color #4: ");
         line1[3] = scan.nextLine();
      }
   
       public static void ConfirmNo()
      {
      
         System.out.println();
      
         System.out.println("Please enter the secret code. You can choose out of five");
         System.out.println("colors: Red, Orange, Yellow, Green, Blue, or Purple.");
         
         System.out.println();
         
         System.out.print("Color #1: ");
         code[0] = scan.nextLine();
         
         System.out.print("Color #2: ");
         code[1] = scan.nextLine();
         
         System.out.print("Color #3: ");
         code[2] = scan.nextLine();
         
         System.out.print("Color #4: ");
         code[3] = scan.nextLine();
         
         System.out.println();
         System.out.print("Confirm:");
         
         for(int i = 0; i < 4; i++)
         {
            
            System.out.print(" " + code[i]);
         }
         
         System.out.print(" " + "(Yes = 1 / No = 2}? ");
         int confirm = scan.nextInt();
      
         if (confirm == 1)
         {         
            ConfirmYes();
         }
         else{
         
            ConfirmNo();
         
         }
      }
   
       public static void main (String[] args) {
       
         System.out.println("Type '1' for the computer to generate a secret code.");
         System.out.println("Type '2' for a human player to enter a secret code.");
      
         System.out.println();
         System.out.print("(1/2): ");
      
         int codeChoice = scan.nextInt();
      
         if (codeChoice == 1)
         {         
            ComputerCode();
            ConfirmYes();
         }
         else{
         
            System.out.println();
         
            System.out.println("Please enter the secret code. You can choose out of five");
            System.out.println("colors: Red, Orange, Yellow, Green, Blue, or Purple.");
         
            System.out.println();
         
            System.out.print("Color #1: ");
            code[0] = scan.nextLine();
         
            System.out.print("Color #2: ");
            code[1] = scan.nextLine();
         
            System.out.print("Color #3: ");
            code[2] = scan.nextLine();
         
            System.out.print("Color #4: ");
            code[3] = scan.nextLine();
         
            System.out.println();   
            System.out.print("Confirm:");
         
            for(int i = 0; i < 4; i++)
            {
               System.out.print(" " + code[i]);
            }
         
            System.out.print(" " + "(Yes = 1 / No = 2}? ");
            int confirm = scan.nextInt();
         
            if (confirm == 1)
            {         
               ConfirmYes();
            }
            else{
            
               ConfirmNo();
            
            };
         }
      }
   }
 

[Keith Lynn]

Note that according to the Javadocs, nextLine() doesn't block. But the next() method may.

[Kevin Behr]

That did the trick, thanks alot!