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.