Solved

Function for finding out if a varible holds a number i.e., IsNumber()?

Posted on 2004-08-25
20
847 Views
Last Modified: 2007-12-19
I need to know if a variable contains a number or not.
I have some conditional formatiing that is taking place, and the variable could either be set to nothing, null, or a number.  Is there a JSP function like this?

String myvar = null
myvar = request.getParameter("myvar");
if(myvar.IsNumber()){
     //Do Something
}else{
     //Do Something else...
}

Thanks in advance,
-MD
0
Comment
Question by:mderbin
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 6
  • 5
  • 4
  • +1
20 Comments
 
LVL 27

Expert Comment

by:rrz
ID: 11898940
Did you try that code ? won't it throw a null pointer ?  

String myvar = null
myvar = request.getParameter("myvar");
if(myvar != null && myvar.IsNumber()){
     //Do Something
}else{
     //Do Something else...
}  

Maybe  you  should look at JSTL if you are looking for functions.   rrz
0
 
LVL 27

Expert Comment

by:rrz
ID: 11898966
Here is old article ( we are using JSTL 1.0 ).
http://java.sun.com/developer/technicalArticles/javaserverpages/faster/
0
 
LVL 27

Expert Comment

by:rrz
ID: 11898970
>we are using JSTL 1.0 ).
No, we are using  1.1  now  !
0
Technology Partners: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 
LVL 27

Accepted Solution

by:
rrz earned 32 total points
ID: 11899226
You could use something like the following.  
<%
      try{ Integer.parseInt(myvar);
                                 out.print("myvar is a number");
      }
         catch(NumberFormatException nfe){
                                out.print("myvar is not a number");
         }
%>

But hopefully someone  else has something prettier.                 rrz
0
 
LVL 27

Expert Comment

by:rrz
ID: 11899265
Or we could use
Float.parseFloat(myvar);
in same way as the above code.
0
 
LVL 35

Assisted Solution

by:TimYates
TimYates earned 31 total points
ID: 11900266
I think rrz's way of catching the parse exception is the only way...
0
 
LVL 14

Expert Comment

by:boonleng
ID: 11900782
You can use regex to check whther the value is a number:
JDK 1.3 needs to use org.apache.regexp.RE, JDK 1.4 can do regex in String object.

RE re = new RE("^[0-9\\.]+$");
if (re.match(value)) {
}
0
 
LVL 35

Expert Comment

by:TimYates
ID: 11900824
That doesn't work, as

     System.out.println( java.util.regex.Pattern.matches( "^[0-9\\.]+$", "0.3.2") ) ;

prints "true"
0
 
LVL 14

Assisted Solution

by:boonleng
boonleng earned 31 total points
ID: 11900947
opps...
modify a bit, try to use pattern {n} but cant seem to get it work, so check base on index.
any better suggestion?

      RE re = new RE("^[0-9\\.]+$");
      if (re.match(value)) {
            int idx = value.indexOf(".");
            if (idx > -1) {
                  return !(value.indexOf(".", idx + 1) > -1);
            }
            return true;
      }
0
 
LVL 35

Expert Comment

by:TimYates
ID: 11900956
> any better suggestion?

yes, the way rrz@871311 does it ;-)
0
 
LVL 14

Expert Comment

by:boonleng
ID: 11900990
hmm... true also :)
0
 
LVL 3

Expert Comment

by:pat5star
ID: 11904934
How about this?

    public static boolean isNumber(String in) {
        if (null == in || in.trim().length() == 0) return false;
        return in.matches("\\d+");
    }

0
 
LVL 35

Expert Comment

by:TimYates
ID: 11905096
>  How about this?

Will that work for 5.3?
0
 
LVL 3

Expert Comment

by:pat5star
ID: 11905220
No, but this will:

    public static boolean isNumber(String in) {
        if (null == in || in.trim().length() == 0) return false;
        return in.matches("\\d*(?:\\.?\\d+)?");
    }

This allows any number. If the variable contains a decimal it must have at least 1 number following it. Here are some example results:

.321 = true
2.45 = true
23.1 = true
23.   = false
230  = true
0
 
LVL 35

Expert Comment

by:TimYates
ID: 11906811
what about

9.0E-3

?

;-)

My point here, is that the best way of doing it is rrz's way that he posted first ;-)
0
 
LVL 3

Assisted Solution

by:pat5star
pat5star earned 31 total points
ID: 11907292
If the OP specifies his exact requirements (like whether he has to handle cases like your last example) then the proper regex can be built to handle that. I disagree that rrz's method is the best way to do it. Checking whether something is true or not by determining whether an exception is thrown is a bad design pattern. I was also going to back this argument up by mentioning that it would be more expensive to test this way but I was suprised by the results.

Running this code:

import java.util.regex.Pattern;
import java.util.regex.Matcher;

public class FindNumber {
   
    private static final Pattern p = Pattern.compile("\\d*(?:\\.?\\d+)?");
   
    public static void main(String[] args) {
        double start, end;
       
        // Use regex, isNumber1()
        start = System.currentTimeMillis();
        for (int i=0; i<1000000; i++) {
            isNumber1("55.55");
        }
        end = System.currentTimeMillis();
        System.out.println("Method using regex completed in " + (end - start)/1000 + " seconds.");
       
        // Use exception check, isNumber2()
        start = System.currentTimeMillis();
        for (int i=0; i<1000000; i++) {
            isNumber2("55.55");
        }
        end = System.currentTimeMillis();
        System.out.println("Method using exception check completed in " + (end - start)/1000 + " seconds.");
       
        // Use regex, with regex pattern precompiled, isNumber3()
        start = System.currentTimeMillis();
        for (int i=0; i<1000000; i++) {
            isNumber3("55.55");
        }
        end = System.currentTimeMillis();
        System.out.println("Method using regex with regex pattern precompiled completed in " + (end - start)/1000 + " seconds.");
    }
   
    public static boolean isNumber1(String in) {
        if (null == in || in.trim().length() == 0) return false;
        return in.matches("\\d*(?:\\.?\\d+)?");
    }
   
    public static boolean isNumber2(String in) {
        try {
            float value = Float.parseFloat(in);
            return true;
        } catch (NumberFormatException ignored) {
            return false;
        }
    }
   
    public static boolean isNumber3(String in) {
        if (null == in || in.trim().length() == 0) return false;
        Matcher m = p.matcher(in);
        return m.matches();
    }
}

Resulted in these benchmarks:

Method using regex completed in 5.579 seconds.
Method using exception check completed in 0.593 seconds.
Method using regex with regex pattern precompiled completed in 1.641 seconds.

So my last argument is out the window :P

Whatever the OP chooses to use is up to him. It seems wrong to me to use isNumber2() but the benchmarks don't lie! I really expected the regex method to be much faster. Either way, I found this topic interesting and learned something new today :)

-Pat
0
 
LVL 3

Expert Comment

by:pat5star
ID: 11908794
Those results were bugging me today so I thought about it more and realized what made the exception check method so much faster. The test was with a valid float so the isNumber2() method never once threw an exception. If you use a value for the test that isn't a valid float, the benchmarks show a whole different picture:

Method using regex completed in 5.796 seconds.
Method using exception check completed in 9.125 seconds.
Method using regex with regex pattern precompiled completed in 1.891 seconds.

That makes isNumber3() look a whole lot better :)

-Pat

0
 
LVL 27

Expert Comment

by:rrz
ID: 11909478
>Is there a JSP function like this?
There probably should be one.  
Javascript has isNaN() .
Why not validate before  submission ?
Here is another way.

    public static boolean isNumber(String var){
                          int pointCount = 0;
                          for(int i=0;i<var.length();i++){
                                   char c = var.charAt(i);
                                   if(var.charAt(i) == '.')pointCount++;
                                   if(!Character.isDigit(c)|| pointCount > 1) return false;
                          }
                          return true;
    }
0

Featured Post

On Demand Webinar: Networking for the Cloud Era

Did you know SD-WANs can improve network connectivity? Check out this webinar to learn how an SD-WAN simplified, one-click tool can help you migrate and manage data in the cloud.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Here's how to start interacting with our community through Post.
Article by: Justin
In light of the WannaCry ransomware attack that affected millions of Windows machines, you might wonder if your Mac needs protecting. Yes, it does and here is how to do it.
Michael from AdRem Software explains how to view the most utilized and worst performing nodes in your network, by accessing the Top Charts view in NetCrunch network monitor (https://www.adremsoft.com/). Top Charts is a view in which you can set seve…
If you’ve ever visited a web page and noticed a cool font that you really liked the look of, but couldn’t figure out which font it was so that you could use it for your own work, then this video is for you! In this Micro Tutorial, you'll learn yo…

724 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question