Co jest lepsze - Integer.parseInt() i złap wyjątek lub zweryfikuj ciąg ze wzorcem przed konwersją?

Question

Widziałem dwa style sprawdzania, czy zmienna jest poprawną liczbą całkowitą w Javie. Jeden wykonując Integer.parseInt i przechwytując wynikowy wyjątek. Innym jest użycie Pattern. Które z poniższych rozwiązań jest lepsze?

String countStr; 
int count; 
try { 
    count = Integer.parseInt(countStr); 
} catch (Exception e) { 
    //return as the variable is not a proper integer. 
    return; 
} 

lub

String integerRegex = "([0-9]{0,9})"; 
if (countStr.isEmpty() || !Pattern.matches(integerRegex, countStr)) { 
    //return as the variable is not a proper integer. 
    return; 
} 

Edycja: My tu pytanie brzmi, czy robi Integer.parseInt() i połowu wyjątek dla walidacji standardowy sposób sprawdzania poprawności int? Przyznaję, że moje regex nie jest doskonałe. Ale czy istnieją jakieś wbudowane metody dostępne w Javie do sprawdzania poprawności int? Właściwie nie lepiej jest przeprowadzić jakąś walidację zamiast po prostu uchwycić wyjątek?

Answer 1

Korzystanie z powyższego podejścia jest lepsze, ponieważ uwzględnia wszystkie rodzaje możliwych błędów i obsługuje wszystkie przypadki. Na przykład to, co napisałeś, nie będzie poprawnie analizować liczb ujemnych.

To ma sens tylko napisać własnego weryfikatora, jeśli chcesz sprawdzić poprawność danego podzbioru wszystkich liczb całkowitych.

Ogólna porada: nie wynajduj ponownie koła, chyba że masz ku temu ważne powody.

Answer 2

Te dwie funkcje służą różnym celom. Jeśli chcesz tylko upewnić się, że łańcuch zawiera określony wzór, użyj drugiego podejścia. Jeśli chcesz go przekonwertować, powinieneś móc parseInt() W takim przypadku nie ma sensu go sprawdzać i konwertować. Jeśli jednak masz określone wymagania dotyczące liczby, być może będziesz musiał najpierw ją sprawdzić, ponieważ parseInt() nie zawsze może wyrzucić wyjątek, jeśli potrafi on parsować coś, co wciąż nie spełnia Twoich wymagań.

Answer 3

Jeśli po prostu walidujesz liczbę całkowitą, myślę, że druga droga jest lepsza.

Obie te metody będą działały dobrze, ale oczywiście będą miały inną ostrość. Ta pierwsza koncentruje się na samej transformacji, podczas gdy ta druga jest wyraźnie bardziej uważnie sprawdzana. I chcesz sprawdzić numer, więc myślę, że druga metoda jest lepsza. Sądzę też, że część drugiej metody jest bardziej czytelna, pozwalając na łatwe utrzymanie kodu, gdzie logika polega na sprawdzaniu poprawności liczby zamiast łańcucha na liczbę.

Answer 4

Istnieje naprawdę dobry powód, aby nie iść z drugim podejściem, jeśli rzeczywiście chcesz sprawdzić, czy dany ciąg może być reprezentowany jako 32-bitowa liczba całkowita, a nie tylko, że reprezentuje liczbę całkowitą w sensie matematycznym.

Jestem pewien, że wszyscy zgadzamy się, że 2147483648 (2 ** 31 dla osób zwracających uwagę) jest idealnie cienką liczbą całkowitą, ale jest to tylko jedna z nieskończenie wielu liczb, dla których dwie opcje dadzą różne wyniki. Jeśli więc chcesz sprawdzić, czy możesz reprezentować ciąg jako 32-bitową liczbę całkowitą, użyj metody parseInt, jeśli chcesz tylko sprawdzić, czy jest to liczba całkowita, skorzystaj z wyrażenia regularnego.

PS: Mimo to nie złapać wyjątek, ale poprawna wyjątek numberFormat zamiast ..