Khi nào cần sử dụng đồng bộ trong Java

Question

Tôi hy vọng điều này sẽ là đủ thông tin, vì vậy, thông tin này sẽ xuất hiện. Nếu bạn cần thêm thông tin, hãy để ý trong phần bình luận.

Tôi có một lớp học có hai lớp bên trong. Mỗi lớp bên trong có hai phương thức gọi một phương thức trong lớp ngoài. Vì vậy, nó trông như thế này:

public OuterClass { 
    private boolean outerMethodHasBeenCalled = false; 

    private void outerMethod() { 
     if(!outerMethodHasBeenCalled) { 
      // do stuff 
     } 

     outerMethodHasBeenCalled = true; 
    } 

    private FirstInnerClass { 
     public void someMethod() { 
      outerMethod(); 
     } 
    } 

    private SecondInnerClass { 
     public void someOtherMethod() { 
      outerMethod(); 
     } 
    } 
} 

Điều quan trọng cần lưu ý là:

• này là dành cho một ứng dụng Android. Các trường hợp của FirstInnerClass và SecondInnerClass được chuyển đến WebView dưới dạng giao diện JavaScript, vì vậy, someMethod và someOtherMethod có thể được gọi bất kỳ lúc nào, không theo thứ tự cụ thể.
• Tôi hiện đang gặp sự cố với mã hiện tại (không có từ khóa được đồng bộ hóa) trong đó outerMethod được gọi khá nhiều vào cùng một thời điểm (tôi in một thông điệp tường trình và chúng được đặt thời gian vào 1000 giây) các đối tượng khác nhau. Ứng dụng của tôi sau đó 'thực hiện công cụ' hai lần vì outerMethodHasBeenCalled vẫn sai khi outerMethod được gọi. Điều này không ổn, và đó chính xác là những gì tôi đang cố gắng ngăn chặn. Ứng dụng của tôi chỉ nên 'thực hiện' một lần và chỉ một lần: lần đầu tiên được gọi là outerMethod.
• Có vẻ như tôi có nhiều phiên bản OuterClass nhưng hãy yên tâm rằng chỉ có một phiên bản là OuterClass.

Điều quan trọng là ứng dụng của tôi 'thực hiện' chỉ là lần đầu tiên outerMethod được gọi (tôi hy vọng điều đó hiển nhiên bây giờ). Tất cả các cuộc gọi tiếp theo về cơ bản bị bỏ qua. Bất kỳ lớp bên trong nào cũng gọi số outerMethod trước - không quan trọng.

Vì vậy, việc sử dụng từ khóa được đồng bộ hóa trong trường hợp này có thích hợp không?

Answer 1

Yup, cho những gì bạn đã đặt ra ở trên, tôi muốn đi với:

private synchronized void outerMethod() { 
... 
} 

Lưu ý, điều này sẽ có tác dụng phụ chặn một trong những người gọi đến khi outerMethod() hoàn tất. Nếu đó là chấp nhận được, mát mẻ. Nếu mục đích chỉ đơn thuần là các mã trong outerMethod() được chạy một lần, và nó là OK cho người gọi thứ hai không được trì hoãn nếu người gọi đầu tiên đang chạy outerMethod(), bạn có thể xem xét:

public OuterClass { 
    private AtomicBoolean outerMethodHasBeenCalled = new AtomicBoolean(); 

    private void outerMethod() { 
     if (outerMethodHasBeenCalled.compareAndSet(false, true)) { 
      // do stuff 
     } 
    } 
... 

Xem JavaDoc for AtomicBoolean để bẻ khóa những gì đang xảy ra ở đó (giả sử nó có sẵn trong Java của Android).

Answer 2

Quấn tất cả mọi thứ trong outerMethod mà bạn muốn chạy một lần duy nhất trong một khối đồng bộ:

private void outerMethod() { 
    synchronized (this) { 
     if(!outerMethodHasBeenCalled) { 
      // do stuff 
     } 

     outerMethodHasBeenCalled = true; 
    } 
} 

Bằng cách đó, lần đầu tiên phương pháp này được gọi là, chỉ có một chủ đề sẽ được phép vào khối đồng bộ tại một thời gian. Người đầu tiên sẽ thực thi mã trong câu lệnh if, sau đó đặt outerMethodHasBeenCalled thành true. Các chủ đề khác sẽ thấy rằng nó là đúng, và bỏ qua nếu mã.