Функция проверки достоверности

Question

Hello Я не могу получить правильную проверку. Я думаю, что в этом коде есть некоторая ошибка, поэтому кто-нибудь может помочь мне решить эту проблему.

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; 
    } 
} 

Я Actaully используя этот код для подтверждения оплаты услуг, в которых я использую регулярное выражение в [0-9] +. Этот код им использовать его в общий вызов проверки function.Actually делается в сервлет следующим образом:

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; 
} 

Я думаю, что нет никакой ошибки в том, что, но у меня есть сомнения в этом function.I я, сталкивающихся с проблема, как если я дам номер комиссии и его принятие validation.please помочь мне

Answer 1

В настоящее время он позволяет значение null или «» считать действительным - это преднамеренно?

Обратите внимание, что ваш текущий код может быть переписан более просто:

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

Теперь, если вы хотите пустой/пустой считать недействительным, я бы переписать его как:

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

Я рекомендую вы используете Обще-языки StringUtils класс, ваш метод проверки повторно написанный

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

И вы полностью удаляете ApplicationConstants.FEE_PATTERN. Проблема, с которой вы сейчас сталкиваетесь, заключается в том, что ваш сервлет перезаписывает свое входное значение ApplicationConstants.FEE_PATTERN. Ваш метод сервлета переписан

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

обрезать свою строку, а затем передать ее.

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

Вы можете непосредственно использовать StringUtils.isNumeric()