Question

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;
}
Was it helpful?

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;

OTHER TIPS

This line:

int range = Integer.parseInt(input);

appears before your while loop. So, you know how to convert the input to an int. The next step is to realize that you should do it each time the user gives you an input. You are almost there.

If input string does not contain a valid number format in string then this line will throw NumberFormatException

int range = Integer.parseInt(input)

You need to put it try catch block

try {
            Integer.parseInt("test");
        } catch (java.lang.NumberFormatException e) {
                count++; //allow user for next attempt.
              //    showInputDialog HERE
              if(count==3) {
                // show your msg here in JDIalog.
              System.exit(0);
           }
}

For 3 chances to input the correct information, you need to use loop, inside loop call your showInputDialog method

As of JDK 1.8, you can use the temporal package to solve this, along-with the try-catch as shown by rest of the answers from this thread:

import java.time.temporal.ValueRange;
...
public static void main (String[] args){
ValueRange range = ValueRange.of(1,9);

    int num, counter = 0;

    do {
    System.out.print("Enter num: ");
    num = Integer.parseInt(scanner.next());
    if (range.isValidIntValue(num))
        System.out.println("InRange");
    else 
        System.out.println("OutOfRange");

    counter ++;
    } while (!range.isValidIntValue(num) && counter < 3);
...
}
Licensed under: CC-BY-SA with attribution
Not affiliated with StackOverflow
scroll top