Saya memiliki fungsi yang menggunakan Pattern#compiledan Matcheruntuk mencari daftar string untuk suatu pola.
Fungsi ini digunakan di banyak utas. Setiap utas akan memiliki pola unik yang diteruskan ke Pattern#compilesaat utas dibuat. Jumlah utas dan pola dinamis, artinya saya dapat menambahkan lebih banyak Patterns dan utas selama konfigurasi.
Apakah saya perlu synchronizemenggunakan fungsi ini jika menggunakan regex? Apakah regex di thread java aman?
Jawaban:
Ya , dari dokumentasi Java API untuk kelas Pattern
Instance dari kelas (Pola) ini tidak dapat diubah dan aman untuk digunakan oleh beberapa utas bersamaan. Instance dari kelas Matcher tidak aman untuk penggunaan seperti itu.
Jika Anda melihat kode yang berpusat pada kinerja, coba setel ulang instance Matcher menggunakan metode reset (), alih-alih membuat instance baru. Ini akan mengatur ulang status instance Matcher, membuatnya dapat digunakan untuk operasi regex berikutnya. Faktanya, itu adalah negara yang dipertahankan dalam contoh Matcher yang bertanggung jawab untuk menjadi tidak aman untuk akses bersamaan.
Keamanan thread dengan ekspresi reguler di Java
RINGKASAN:
API ekspresi reguler Java telah dirancang untuk memungkinkan satu pola terkompilasi untuk dibagikan ke beberapa operasi pencocokan.
Anda bisa dengan aman memanggil Pattern.matcher () pada pola yang sama dari utas berbeda dan dengan aman menggunakan matcher secara bersamaan. Pattern.matcher () aman untuk membuat matcher tanpa sinkronisasi. Meskipun metode ini tidak disinkronkan, internal ke kelas Pola, variabel volatil yang disebut dikompilasi selalu disetel setelah membuat pola dan dibaca di awal panggilan ke matcher (). Ini memaksa utas apa pun yang merujuk ke Pola untuk "melihat" konten objek itu dengan benar.
Di sisi lain, Anda tidak boleh membagikan Matcher di antara utas yang berbeda. Atau setidaknya, jika pernah melakukannya, Anda harus menggunakan sinkronisasi eksplisit.
Meskipun Anda perlu mengingat bahwa keamanan utas juga harus mempertimbangkan kode di sekitarnya, Anda tampaknya beruntung. Fakta bahwa Pencocokan dibuat menggunakan metode pabrik pencocokkan Pola dan kurangnya konstruktor publik adalah tanda positif. Demikian juga, Anda menggunakan metode statis kompilasi untuk membuat Pola yang mencakup .
Jadi, singkatnya, jika Anda melakukan sesuatu seperti contoh:
Pattern p = Pattern.compile("a*b");
Matcher m = p.matcher("aaaaab");
boolean b = m.matches();kamu harus melakukannya dengan cukup baik.
Tindak lanjut dari contoh kode untuk kejelasan: perhatikan bahwa contoh ini sangat menyiratkan bahwa Matcher yang dibuat adalah thread-local dengan Pola dan pengujian. Yaitu, Anda tidak boleh mengekspos Matcher yang dibuat ke utas lainnya.
Terus terang, itulah risiko dari pertanyaan keamanan utas apa pun. Kenyataannya adalah bahwa kode apa pun dapat dibuat thread-unsafe jika Anda berusaha cukup keras. Untungnya, ada buku bagus yang mengajari kita banyak cara untuk merusak kode kita. Jika kita menjauh dari kesalahan itu, kita sangat mengurangi kemungkinan masalah threading kita sendiri.
Sekilas kode untuk Matcher.javamenunjukkan sekelompok variabel anggota termasuk teks yang cocok, array untuk grup, beberapa indeks untuk memelihara lokasi dan beberapa booleanuntuk negara lain. Ini semua mengarah ke stateful Matcheryang tidak akan berperilaku baik jika diakses oleh banyak orang Threads. Begitu juga dengan JavaDoc :
Instance dari kelas ini tidak aman untuk digunakan oleh beberapa thread bersamaan.
Ini hanya masalah jika, seperti yang dikatakan @Bob Cross, Anda berusaha keras untuk mengizinkan penggunaan Anda Matcherdi Threads yang terpisah . Jika Anda perlu melakukan ini, dan menurut Anda sinkronisasi akan menjadi masalah bagi kode Anda, opsi yang Anda miliki adalah menggunakan ThreadLocalobjek penyimpanan untuk mempertahankan Matcherper thread yang berfungsi.
Singkatnya, Anda dapat menggunakan kembali (tetap dalam variabel statis) Pola yang telah dikompilasi dan memberi tahu mereka untuk memberi Anda Matcher baru ketika diperlukan untuk memvalidasi pola regex tersebut terhadap beberapa string
import java.util.regex.Matcher;
import java.util.regex.Pattern;
/**
* Validation helpers
*/
public final class Validators {
private static final String EMAIL_PATTERN = "^[_A-Za-z0-9-]+(\\.[_A-Za-z0-9-]+)*@[A-Za-z0-9-]+(\\.[A-Za-z0-9-]+)*(\\.[A-Za-z]{2,})$";
private static Pattern email_pattern;
static {
email_pattern = Pattern.compile(EMAIL_PATTERN);
}
/**
* Check if e-mail is valid
*/
public static boolean isValidEmail(String email) {
Matcher matcher = email_pattern.matcher(email);
return matcher.matches();
}
}lihat http://zoomicon.wordpress.com/2012/06/01/validating-e-mails-using-regular-expressions-in-java/ (di bagian akhir) mengenai pola RegEx yang digunakan di atas untuk memvalidasi email ( jika tidak sesuai dengan kebutuhan untuk validasi email seperti yang diposting di sini)
static {}? Anda dapat memasukkan inisialisasi variabel itu dan membuat Pattern finaljuga.
private static final Pattern emailPattern = Pattern.compile(EMAIL_PATTERN);lebih baik.
compile()metodenya mungkin tidak. Ada dua atau tiga bug selama bertahun-tahun yang menyebabkan kompilasi gagal di lingkungan multithread. Saya akan merekomendasikan melakukan kompilasi dalam blok tersinkronisasi.