Mẫu "tĩnh" không nên tĩnh?

Question

tôi chỉ tìm thấy một lỗi trong một số mã tôi đã không viết và tôi là một chút ngạc nhiên:

Pattern pattern = Pattern.compile("\\d{1,2}.\\d{1,2}.\\d{4}"); 
Matcher matcher = pattern.matcher(s); 

Mặc dù thực tế rằng mã này thất bại nặng nề trên dữ liệu đầu vào chúng tôi nhận (vì nó cố gắng tìm ngày theo định dạng 2011/01/17 và được trở lại những thứ như 10396/2011 và sau đó bị rơi vì nó không thể phân tích cú pháp ngày nhưng điều đó thực sự không phải là điểm của câu hỏi này;) tôi tự hỏi:

• không phải là một trong các điểm của Pa ttern.compile là một tối ưu hóa tốc độ (bằng cách biên dịch trước khi biên dịch)?

• không phải tất cả mẫu "tĩnh" là luôn là được biên dịch thành mẫu tĩnh?

Có rất nhiều ví dụ, tất cả khắp nơi trên web, nơi mà cùng một khuôn mẫu luôn được biên dịch lại bằng Pattern.compile mà tôi bắt đầu tự hỏi, nếu tôi nhìn thấy mọi thứ hay không.

Không phải là (giả định rằng chuỗi là tĩnh và do đó không động xây dựng):

static Pattern pattern = Pattern.compile("\\d{1,2}.\\d{1,2}.\\d{4}"); 

luôn preferrable trên một tài liệu tham khảo mẫu không tĩnh?

Answer 1

1. Có, toàn bộ điểm biên dịch trước là Pattern chỉ thực hiện một lần.
2. Nó thực sự phụ thuộc vào cách bạn sẽ sử dụng nó, nhưng nói chung, các mẫu được biên dịch trước được lưu trữ trong các trường static sẽ ổn. (Không giống như Matcher s, không an toàn cho chủ đề và do đó không thực sự được lưu trữ trong các trường ở tất cả, tĩnh hay không.)

Thông báo duy nhất với các mẫu biên dịch trong bộ khởi tạo tĩnh là nếu mẫu đó không ' t biên dịch và khởi tạo tĩnh ném một ngoại lệ, nguồn gốc của lỗi có thể khá khó chịu để theo dõi. Đó là một vấn đề bảo trì nhỏ nhưng nó có thể đáng nói đến.

Answer 2

trước tiên, lỗi trong mẫu là do dấu chấm (.) Khớp với mọi thứ. (.) Nếu bạn muốn kết hợp chấm bạn phải thoát khỏi nó trong regex:

Pattern pattern = Pattern.compile("\\d{1,2}\\.\\d{1,2}\\.\\d{4}");

Thứ hai, Pattern.compile() là một phương pháp nặng. Nó luôn luôn được khuyến khích để khởi tạo mô hình tĩnh (tôi có nghĩa là các mẫu không được thay đổi hoặc không được tạo ra trên bay) chỉ một lần. Một trong những cách phổ biến để đạt được điều này là đặt Pattern.compile() vào bộ khởi tạo tĩnh.

Bạn có thể sử dụng cách tiếp cận khác. Ví dụ sử dụng mẫu đơn hoặc sử dụng khung làm việc tạo ra các đối tượng đơn lẻ (như Spring).

Answer 3

Có, việc biên dịch Mẫu trên mỗi lần sử dụng là lãng phí và xác định mẫu tĩnh sẽ dẫn đến hiệu suất tốt hơn. Xem this SO thread để có cuộc thảo luận tương tự.

Answer 4

Mẫu tĩnh sẽ vẫn còn trong bộ nhớ miễn là lớp được tải.

Nếu bạn lo lắng về bộ nhớ và muốn dùng một lần ném Pattern mà bạn sử dụng một lần trong một thời gian và có thể thu gom rác khi bạn đã hoàn thành nó, thì bạn có thể sử dụng số Pattern không tĩnh.

Answer 5

Đây là thời điểm cổ điển so với giao dịch bộ nhớ. Nếu bạn chỉ biên dịch Mẫu một lần, đừng dán nó vào trường tĩnh. Nếu bạn đo các mẫu biên dịch chậm, biên dịch trước và đặt nó vào một trường tĩnh.