mejora de funcion.

[abulon81]

Hi all, You take a look at this friends, I guess this function it's can make improvements.

Código PHP:

   
System.out.println(testNum("10.50")); 
........ 
   public static int testNum(String numero) { 
      int bol=0; 
      boolean bole=false; 
       int number ; 
       double num ; 
       try { 
          if(bole==false) 
          { 
               Integer.parseInt(numero); 
              bol=1; bole=true; 
          } 
           //System.out.println("it's integer"); 
       }        
       catch (NumberFormatException e) { 
           //e.printStackTrace(); 
       } 
       finally { 
          try { 
              if (bole==false) 
              { 
                   Double.parseDouble(numero); 
                   bol=3; bole=true; 
              } 
          } 
           
          catch(NumberFormatException e) { 
           e.printStackTrace();    
          } 
           
          finally { 
              return bol; 
          } 
       } 
   }  
   

Hope this helps all of you.

[Heimish2000]

Excuse me for my english level, I know could be better

The first

if(bole==false)

have no sense because in this point bole ALWAYS will be false.

bol will be 0 if it's not a number, 1 if it's a integer and 3 if it's a double, is this what you want?

A better way to do the same with less code line is:

Código JAVA:

System.out.println(testNum("10.50")); 
........ 
   public static int testNum(String numero) { 
      int bol=0;  
       int number ; 
       double num ; 
       try { 
            Integer.parseInt(numero); 
            bol=1; 
       }        
       catch (NumberFormatException e) { 
          try { 
            Double.parseDouble(numero); 
            bol=3; 
          } 
          catch(NumberFormatException e) { 
                e.printStackTrace();    
          } 
       } 
       finally { 
        return bol; 
          } 
       } 
   }
 

[abulon81]

Iniciado por Heimish2000

Excuse me for my english level, I know could be better

The first

if(bole==false)

have no sense because in this point bole ALWAYS will be false.

bol will be 0 if it's not a number, 1 if it's a integer and 3 if it's a double, is this what you want?

A better way to do the same with less code line is:

Código JAVA:

System.out.println(testNum("10.50")); 
........ 
   public static int testNum(String numero) { 
      int bol=0;  
       int number ; 
       double num ; 
       try { 
            Integer.parseInt(numero); 
            bol=1; 
       }        
       catch (NumberFormatException e) { 
          try { 
            Double.parseDouble(numero); 
            bol=3; 
          } 
          catch(NumberFormatException e) { 
                e.printStackTrace();    
          } 
       } 
       finally { 
        return bol; 
          } 
       } 
   }
 

no worries mate, youre right is less code and the results are the same. Thanks a lot my friend.
Cheers.