Search code examples
javavalidationcounter

Java Input Validation for Number Range and Numeric values only with counter


I am trying to complete a task and I am unsure what route to take. I have tried while, if, and a combination of statements and cannot get the input validation I need.

  • I am trying to validate user input and ensure their input is a number between 0 and 10 (excluding 0 and 10).
  • Also I need to make sure that what they do enter is a number and not some symbol or letter.
  • Finally I need a counter that will allow them 3 chances to input the correct information.

The code below is my method I am trying to setup to accomplish this.

private static int getNumericInput(String quantity) {
    int count = 0;
    String input;
    input = JOptionPane.showInputDialog(quantity);
    int range = Integer.parseInt(input);

    while ((range > 9 || range < 1) && (count < 2)) {
        JOptionPane.showMessageDialog(null, 
                "Sorry that input is not valid, please choose a quantity from 1-9");
        input = JOptionPane.showInputDialog(quantity);
        count++;
    }
    if (count == 2) {
        JOptionPane.showMessageDialog(null, 
                "Sorry you failed to input a valid response, terminating.");
        System.exit(0);
    }
    return range;
}

Solution

  • As others have said to see if a String is a valid integer you catch a NumberFormatException.

    try {
        int number = Integer.parseInt(input);
        // no exception thrown, that means its a valid Integer
    } catch(NumberFormatException e) {
        // invalid Integer
    }
    

    However I would also like to point out a code change, this is a prefect example of a do while loop. Do while loops are great when you want to use a loop but run the condition at the end of the first iteration.

    In your case you always want to take the user input. By evaluating the while loops condition after the first loop you can reduce some of that duplicate code you have to do prior to the loop. Consider the following code change.

    int count = 0;
    String input;
    int range;
    do {
        input = JOptionPane.showInputDialog(quantity);
        try {
            range = Integer.parseInt(input);
        } catch(NumberFormatException e) {
            JOptionPane.showMessageDialog(null, "Sorry that input is not valid, please choose a quantity from 1-9");
            count++;
            // set the range outside the range so we go through the loop again.
            range = -1;
        }
    } while((range > 9 || range < 1) && (count < 2));
    
    if (count == 2) {
        JOptionPane.showMessageDialog(null, 
                "Sorry you failed to input a valid response, terminating.");
        System.exit(0);
    }
    return range;