Between 1 & 100?

[joe kane]

Hy everyone,
Im try to valadate a input dialog: so that the value can only be between 1 & 100.

 
choice1 = JOptionPane.showInputDialog("Enter Grade 1");
 
     while (choice1 == null){ System.exit(0);}
     while ( !choice1.matches("[0-9]+" ) ){
  JOptionPane.showMessageDialog(null,"Error, Numbers only");
choice1 = JOptionPane.showInputDialog("Enter Grade 1");
} 

As you can see its valadated so that only numbers can be entered....Could you please show me how this is done....
Thanks in advance joe kane

[joe kane]

choice1 is a String....

[Billybob Marshall]

int value = Integer.parseInt(choice1);
boolean between1And100 = (value >= 1) && (value <= 100);
You'll need to catch the possible NumberFormatException.

[Sadanand Murthy]

Originally posted by joe kane:
Hy everyone,
Im try to valadate a input dialog: so that the value can only be between 1 & 100.

 
choice1 = JOptionPane.showInputDialog("Enter Grade 1");
 
     while (choice1 == null){ System.exit(0);}
     while ( !choice1.matches("[0-9]+" ) ){
  JOptionPane.showMessageDialog(null,"Error, Numbers only");
choice1 = JOptionPane.showInputDialog("Enter Grade 1");
} 

As you can see its valadated so that only numbers can be entered....Could you please show me how this is done....
Thanks in advance joe kane

Billybob Marshall has already explained that easiest is to convert the string to an int & check that the int is between 1 & 100.
I want to point out that you don't need a while loop to test if choice is null. You can do this with an if statement.

[joe kane]

Thats got it sorted thanks loads for the help...
Thanks joe Kane

[joe kane]

 
choice1 = JOptionPane.showInputDialog("Enter Grade 1");
 
if (choice1 == null){ System.exit(0);}
while ( !choice1.matches("[0-9]+" ) ){
JOptionPane.showMessageDialog(null,"Error, Numbers only");
choice1 = JOptionPane.showInputDialog("Enter Grade 1");
}
int value = Integer.parseInt(choice1);
boolean between1And100 = (value >= 1) || (value <= 100);
    while (between1And100 = false){
        
        JOptionPane.showMessageDialog(null,"Grade is between 1 & 100");
        
    }
    
    while (between1And100 = true){
        
        JOptionPane.showMessageDialog(null,"sucesses");
        choice1 = JOptionPane.showInputDialog("Enter Grade 1");
    } 

Sorry im still having a problem...
Thanks joe kane

[Dirk Schreckmann]

while (between1And100 = false)
This is no good. between1And100 = false is an assignment statement. Here between1And100 is assigned a value of false, and this assigned value is then used to evaluate the condition of the loop, which would always be false. Perhaps you want to use the comparison operator ==.