Mengapa semua orang mendorong solusi pengecualian / regex?
Walaupun saya bisa mengerti kebanyakan orang baik-baik saja dengan menggunakan try / catch, jika Anda ingin sering melakukannya ... itu bisa sangat melelahkan.
Apa yang saya lakukan di sini adalah mengambil regex, metode parseNumber (), dan metode pencarian array untuk melihat mana yang paling efisien. Kali ini, saya hanya melihat angka integer.
public static boolean isNumericRegex(String str) {
if (str == null)
return false;
return str.matches("-?\\d+");
}
public static boolean isNumericArray(String str) {
if (str == null)
return false;
char[] data = str.toCharArray();
if (data.length <= 0)
return false;
int index = 0;
if (data[0] == '-' && data.length > 1)
index = 1;
for (; index < data.length; index++) {
if (data[index] < '0' || data[index] > '9') // Character.isDigit() can go here too.
return false;
}
return true;
}
public static boolean isNumericException(String str) {
if (str == null)
return false;
try {
/* int i = */ Integer.parseInt(str);
} catch (NumberFormatException nfe) {
return false;
}
return true;
}
Hasil dalam kecepatan yang saya dapatkan adalah:
Done with: for (int i = 0; i < 10000000; i++)...
With only valid numbers ("59815833" and "-59815833"):
Array numeric took 395.808192 ms [39.5808192 ns each]
Regex took 2609.262595 ms [260.9262595 ns each]
Exception numeric took 428.050207 ms [42.8050207 ns each]
// Negative sign
Array numeric took 355.788273 ms [35.5788273 ns each]
Regex took 2746.278466 ms [274.6278466 ns each]
Exception numeric took 518.989902 ms [51.8989902 ns each]
// Single value ("1")
Array numeric took 317.861267 ms [31.7861267 ns each]
Regex took 2505.313201 ms [250.5313201 ns each]
Exception numeric took 239.956955 ms [23.9956955 ns each]
// With Character.isDigit()
Array numeric took 400.734616 ms [40.0734616 ns each]
Regex took 2663.052417 ms [266.3052417 ns each]
Exception numeric took 401.235906 ms [40.1235906 ns each]
With invalid characters ("5981a5833" and "a"):
Array numeric took 343.205793 ms [34.3205793 ns each]
Regex took 2608.739933 ms [260.8739933 ns each]
Exception numeric took 7317.201775 ms [731.7201775 ns each]
// With a single character ("a")
Array numeric took 291.695519 ms [29.1695519 ns each]
Regex took 2287.25378 ms [228.725378 ns each]
Exception numeric took 7095.969481 ms [709.5969481 ns each]
With null:
Array numeric took 214.663834 ms [21.4663834 ns each]
Regex took 201.395992 ms [20.1395992 ns each]
Exception numeric took 233.049327 ms [23.3049327 ns each]
Exception numeric took 6603.669427 ms [660.3669427 ns each] if there is no if/null check
Penafian: Saya tidak mengklaim metode ini 100% dioptimalkan, mereka hanya untuk demonstrasi data
Pengecualian dimenangkan jika dan hanya jika jumlahnya 4 karakter atau kurang, dan setiap string selalu angka ... dalam hal ini, mengapa bahkan memiliki cek?
Singkatnya, sangat menyakitkan jika Anda sering menemukan nomor yang tidak valid dengan try / catch, yang masuk akal. Aturan penting yang selalu saya ikuti adalah JANGAN PERNAH menggunakan try / catch untuk aliran program . Ini adalah contoh mengapa.
Menariknya, simpel jika char <0 || > 9 sangat sederhana untuk ditulis, mudah diingat (dan harus bekerja dalam berbagai bahasa) dan memenangkan hampir semua skenario pengujian.
Satu-satunya downside adalah bahwa saya menebak Integer.parseInt () mungkin menangani angka non ASCII, sedangkan metode pencarian array tidak.
Bagi mereka yang bertanya-tanya mengapa saya mengatakan mudah untuk mengingat satu array karakter, jika Anda tahu tidak ada tanda-tanda negatif, Anda dapat dengan mudah lolos dengan sesuatu yang kental seperti ini:
public static boolean isNumericArray(String str) {
if (str == null)
return false;
for (char c : str.toCharArray())
if (c < '0' || c > '9')
return false;
return true;
Terakhir sebagai catatan terakhir, saya ingin tahu tentang operator asistensi dalam contoh yang diterima dengan semua suara di atas. Menambahkan penugasan
double d = Double.parseDouble(...)
tidak hanya tidak berguna karena Anda bahkan tidak menggunakan nilai, tetapi juga membuang-buang waktu pemrosesan dan meningkatkan runtime oleh beberapa nanodetik (yang menyebabkan peningkatan 100-200 ms dalam tes). Saya tidak dapat melihat mengapa ada orang yang melakukan itu karena ini adalah pekerjaan tambahan untuk mengurangi kinerja.
Anda akan berpikir bahwa itu akan dioptimalkan ... meskipun mungkin saya harus memeriksa bytecode dan melihat apa yang dilakukan oleh kompiler. Itu tidak menjelaskan mengapa itu selalu tampak lebih panjang bagi saya meskipun jika entah bagaimana dioptimalkan keluar ... karena itu saya bertanya-tanya apa yang terjadi. Sebagai catatan: Secara lebih panjang, maksud saya menjalankan tes untuk 10.000.000 iterasi, dan menjalankan program itu beberapa kali (10x +) selalu menunjukkan itu lebih lambat.
EDIT: Diperbarui tes untuk Character.isDigit ()