problème de validation fonction

Question

Bonjour Je ne suis pas en mesure d'obtenir la validation correcte.Je pense qu'il ya une erreur dans ce code, donc quelqu'un peut-il m'aider s'il vous plaît à résoudre ce problème.

public static boolean validateFee(String value) { 

     boolean isvalid = true; 
     try { 
      int fee = 0; 
      if (value != null && !value.isEmpty()) { 
       fee = Integer.parseInt(value); 
      } 
     } catch (NumberFormatException ne) { 
      // ne.printStackTrace(); 
      isvalid = false; 
      return isvalid; 

     } 
     return isvalid; 
    } 
} 

J'utilise actaully ce code pour la validation des droits dans lequel je suis en utilisant une expression régulière comme [0-9] +. Ce code im utiliser dans un appel de validation function.Actually commun est fait dans le servlet comme suit:

private Boolean validateFee(HttpSession session, PropertiesHandler props, String number) { 
    Boolean isvalid = true; 
    HashMap hashMap = new LinkedHashMap(); 
    number = ApplicationConstants.FEE_PATTERN; 
    if (!Validation.validateFee(number)) { 
     isvalid = false; 
     hashMap.put("time", props.getText("error.fee.invalid.type")); 
    } 
    session.setAttribute("errorMessage", hashMap); 
    System.out.println("Map size " + hashMap.size()); 
    logger.info("Exit validateTIme"); return isvalid; 
} 

Je pense qu'il n'y a pas d'erreur dans cela, mais j'ai un doute dans cette function.I suis face à une problème comme si je donne le numéro à la taxe également sa prise de validation.please help me out

Answer 1

Actuellement, il permet la valeur de null ou "" de compter comme étant valide - est-ce délibéré?

Notez que votre code actuel peut être plus simplement réécrite:

public static boolean validateFee(String value) { 
    try { 
     if (value != null && !value.isEmpty()) { 
      Integer.parseInt(value); 
     } 
     return true; 
    } catch (NumberFormatException ne) { 
     return false; 
    } 
} 

Maintenant, si vous voulez null/vide pour être considéré comme invalide, je réécris comme:

public static boolean validateFee(String value) { 
    if (value == null || value.isEmpty()) { 
     return false; 
    } 
    try { 
     Integer.parseInt(value); 
     return true; 
    } catch (NumberFormatException ne) { 
     return false; 
    } 
} 

Answer 2

Je recommande vous utilisez commons-lang StringUtils classe, votre méthode validate réécrites

public static boolean validateFee(String value) { 
    return StringUtils.isNumeric(StringUtils.trimToNull(value)); 
} 

Et vous supprimez complètement ApplicationConstants.FEE_PATTERN. Le problème auquel vous êtes actuellement confronté est que votre servlet écrase sa valeur d'entrée avec ApplicationConstants.FEE_PATTERN. Votre méthode de servlet est réécrite

private Boolean validateFee(HttpSession session, PropertiesHandler props, String number) { 
    final Boolean valid = Validation.validateFee(number); 
    if (!valid) { 
     final HashMap hashMap = new LinkedHashMap(); 
     hashMap.put("time", props.getText("error.fee.invalid.type")); 
     session.setAttribute("errorMessage", hashMap); 
    } 
} 

Answer 3

Coupez votre chaîne, puis transmettez-la à.

StringUtils.isNumeric(StringUtils.trimToNull(fees)); 

Vous pouvez utiliser directement StringUtils.isNumeric()