Non-numeric data

That doesn't work properly. First of all If i enter 2 players I want to be able to do the maths then once i press input 4 i get my score. Once i press my score it goes to the next Player, he does the maths presses input 4 and it shows his score, followed by all users scores. This is incorrect.

Also the alpha numeric checks don't work at all. Instead of looping the same input box it skips and goes to the next player.

I had userChoice = Integer.parseInt(userChoiceStr); in my original code. I have no idea why it wasn't here when i posted.

Got most of it working except one input.


import javax.swing.JOptionPane;
import java.util.Random;
import java.util.Vector;

public class Calc
  {
  //Initialise result & count to 0

  public static int result = 0;
  public static int count = 0;
 
 
  public static void main(String[] args)
    {
       boolean inputOK = false;
       int  i=0;
       int userChoice = 0;
       double playerChoice = 0;
       String userChoiceStr ;
       String playerChoiceStr;
       
       
    //   Vector v = new Vector ();
        double v[] = new double[50];

         while (!inputOK)
        {    
            try
            {
         playerChoiceStr = JOptionPane.showInputDialog(null,
                        "How many players?");
        playerChoice = Double.parseDouble (playerChoiceStr);
         inputOK = true; // code will never reach this spot if input is not correct
        }
        catch (NumberFormatException e)
        {
           
                   
            }
        }
 
   
           
    for (i =0; i < (int)playerChoice ; i++)
    {

          do
          {          
       
              // user input prompt
              userChoiceStr =  JOptionPane.showInputDialog (null,
                             
                        "Player" + " " + (i+1)
                        + "\n" +                                  
                          "1. Practice addition\n"
                        + "2. Practice subtraction\n"
                        + "3. Practice multiplication\n"
                        + "4. Quit the program\n"
                        + "\n"
                        + "Enter your choice",
                     
                        "Enter your choice",
                        JOptionPane.QUESTION_MESSAGE);
                        // keeps track of wrong entry
       
            userChoice = Integer.parseInt(userChoiceStr);
           
                       
       // switch statement to count user score                  
       switch (userChoice)
       {
       case 1:  
          ++count;
          addition ();
          break;
       case 2:
           ++count;
           subtraction ();
           break;
       case 3:
           ++count;
           multiplication ();
            break;
       case 4:      
            break;
        default:
         // prompt if user doesn't enter a number between 1-4
        }
    }            

    while ( userChoice != 4 );  // returns your percentage if input not equal to 4


 
String str = "Got " + (result / count) + "% right!! \n";
v[i]= (result/count);

JOptionPane.showMessageDialog ( null , "You " + str );


}

String showAll = "";
    i=0;

for (i =0; i < (int)playerChoice ; i++)
    {
    showAll = showAll + "Player "+ (i+1) + " " + v[i] +" % right!! \n";
    }

JOptionPane.showMessageDialog ( null , showAll );
                               
// Ends program
  System.exit(0);
 }


 
  public static void addition()
    {      
        boolean inputOK = false;
        int additionData = 0;
        int genNumOne, genNumTwo;
        String selectAddition;
 
       // generates 2 random numbers between 0 and 9
        genNumOne = (int) (Math.random() * 9) + 0;
        genNumTwo = (int) (Math.random() * 9) + 0;
   
        while (!inputOK)
        {    
            try
            {
        selectAddition = JOptionPane.showInputDialog ("What is" + " " + genNumOne +
                                            " " + "plus" + " " + genNumTwo + "?");  
 
        additionData = Integer.parseInt(selectAddition);
        inputOK = true; // code will never reach this spot if input is not correct
        }
        catch (NumberFormatException e)
        {
                             
            }
        }
     
       if (additionData == (genNumOne + genNumTwo) )
       {
           correctAnswer() ;
           result = result + 100;
           
        }
        else
            {
        incorrectAnswer() ;
     
                                 
     }
}
 
 
    public static void subtraction()
    {    
        boolean inputOK = false;
        int subtractionData = 0;
        int genNumOne, genNumTwo;
        String selectSubtraction;
 
     
        genNumOne = (int) (Math.random() * 9) + 0;
        genNumTwo = (int) ( Math.random() * genNumOne) + 0;
        // makes sure first random number is greater than second
 
        while (!inputOK)
        {    
            try
            {
        selectSubtraction = JOptionPane.showInputDialog("What is" + " " + genNumOne +
                            " " + "minus" + " " + genNumTwo + "?");  
                                             
        subtractionData = Integer.parseInt(selectSubtraction);
       
         inputOK = true; // code will never reach this spot if input is not correct
        }
        catch (NumberFormatException e)
        {
                             
            }
        }
   
 
       if (subtractionData == (genNumOne - genNumTwo) )
       {
           correctAnswer() ;
           result = result + 100;
           
        }
        else
            {
        incorrectAnswer() ;
     
                                 
     }
}
     
 
 
    public static void multiplication()
    {
        boolean inputOK = false;
        int multiplicationData = 0;
        int genNumOne, genNumTwo;
        String selectMultiplication;
       
        genNumOne = (int) (Math.random() * 9) + 0;
        genNumTwo = (int) (Math.random() * 9) + 0;
 
        while (!inputOK)
        {    
            try
            {
        selectMultiplication = JOptionPane.showInputDialog("What is" + " " + genNumOne +
                                                 " " + "times" + " " + genNumTwo + "?");  
     
                                               
        multiplicationData = Integer.parseInt(selectMultiplication);
        inputOK = true; // code will never reach this spot if input is not correct
        }
        catch (NumberFormatException e)
        {
                           
            }
        }
   
 
       if ( multiplicationData == (genNumOne * genNumTwo) )
       {
           correctAnswer() ;
           result = result + 100;
           
        }
        else
            {
        incorrectAnswer() ;
     
                                 
     }
}
 
 
 
    public static void correctAnswer()
{
 
    String[] arrGood = { "Very good!",  "Well done", "Great!" };
    Random randGood = new Random();
 
    int new_rand_index = randGood.nextInt( arrGood.length );
   String message = arrGood[ new_rand_index ];
   JOptionPane.showMessageDialog(null, message,
                    "Good Answer",
                    JOptionPane.INFORMATION_MESSAGE);
                 
                 
 
    }
 
 
 
    public static void incorrectAnswer()
{
 
    String[] arrBad = {  "Sorry that was incorrect.", "Better luck next time",  "Bad luck" };
    Random randBad = new Random();
 
 
    int new_rand_index = randBad.nextInt( arrBad.length );
   String message = arrBad[ new_rand_index ];
   JOptionPane.showMessageDialog(null, message,
                    "Bad Answer",
                    JOptionPane.INFORMATION_MESSAGE);
 
   
 
      }
}

Missed out userChoice=0;

Totally working now