Prawidłowe sposobem na uniknięcie parseInt rzucanie NumberFormatException do ciągu wejściowego: ""

Question

Kiedy biegnę parseInt:

Integer.parseInt(myString); 

rzuca:

NumberFormatException: For input string: "" 

Czy to znaczy, że muszę zrobić coś takiego?

if(StringUtils.isNotBlank(myString)) 
    return Integer.parseInt(myString); 
else 
return 0; 

Answer 1

Cóż, można użyć operatora warunkowego Zamiast:

return StringUtils.isNotBlank(myString) ? Integer.parseInt(myString) : 0; 

Jeśli trzeba to zrobić w wielu miejscach, to prawdopodobnie chcesz umieścić to w osobnej metody. Zauważ, że powinieneś również wziąć pod uwagę sytuacje, w których myString ma wartość null lub zawiera tekst nieliczbowy.

Answer 2

Tak. (Zatwierdź swoje dane wejściowe zanim podejmiesz założenia dotyczące tego, co w nich jest. :-)

+1 za odnalezienie już popularnego Apache'a w/StringUtils.

Answer 3

Integer.parseInt(String) nie przyjmuje nieliczbowych danych wejściowych, w tym pustych znaków i pustych ciągów znaków.

Zabezpiecz się przed tym, jak sugerujesz, lub złap NFE.

Answer 4

Tak, ale: Owiń ją w cienkiej metodą (i wyeliminować zbędne else) lub użyć istniejącego wdrożenia, jak Commons Lang's NumberUtils.toInt(str, defaultValue):

NumberUtils.toInt(myString, 0); 

Metoda ta obsługuje null wartości i niepowodzeń konwersji.

Answer 5

Co masz jest w porządku, ale jako styl kodowania Wolę zrobić testy „pozytywne” (isBlank), zamiast „negatywne” (isNotBlank), tj

if (StringUtils.isBlank(myString)) { 
    return 0; 
} 
return Integer.parseInt(myString); // Note: No need for else when the if returns 

lub bardziej zwięźle:

return StringUtils.isBlank(myString) ? 0 : Integer.parseInt(myString); 

Answer 6

nie wiem dlaczego ja szukając tego, ale oto prosty sposób:

int test=str.isEmpty()?0:Integer.parseInt(str); 

Answer 7

Jeśli łańcuch może być pusta zrobić to w ten sposób:

Integer.parseInt("0" + inputString) 

Kiedy nie jestem pewien, że zawiera tylko cyfry:

Integer.parseInt(0 + inputString.replaceAll("\\D+",""))