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.
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;
}
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;