Đây có phải là thiết kế đồng bộ hóa chính xác không?

Question

Tôi có một hệ thống lấy mẫu. Tôi có nhiều chủ đề khách hàng trong ứng dụng quan tâm đến các Mẫu này, nhưng quá trình thực hiện lấy mẫu chỉ có thể xảy ra trong một ngữ cảnh. Đó là đủ nhanh mà nó okay cho nó để chặn quá trình gọi điện thoại cho đến khi lấy mẫu được thực hiện, nhưng đủ chậm mà tôi không muốn nhiều chủ đề chồng chất lên yêu cầu. Tôi đã đưa ra thiết kế này (tước xuống chi tiết tối thiểu):

public class Sample 
{ 
    private static Sample _lastSample; 
    private static int _isSampling; 

    public static Sample TakeSample(AutomationManager automation) 
    { 
     //Only start sampling if not already sampling in some other context 
     if (Interlocked.CompareExchange(ref _isSampling, 0, 1) == 0) 
     { 
      try 
      { 
       Sample sample = new Sample(); 
       sample.PerformSampling(automation); 
       _lastSample = sample; 
      } 
      finally 
      { 
       //We're done sampling 
       _isSampling = 0; 
      } 
     } 

     return _lastSample; 
    } 

    private void PerformSampling(AutomationManager automation) 
    { 
     //Lots of stuff going on that shouldn't be run in more than one context at the same time 
    } 
} 

Điều này có an toàn để sử dụng trong trường hợp tôi mô tả không?

Answer 1

Có, có vẻ như an toàn vì int là một loại nguyên tử ở đây. Nhưng tôi vẫn sẽ tư vấn thay thế

private static int _isSampling; 

với

private static object _samplingLock = new object(); 

và sử dụng:

lock(_samplingLock) 
{ 
    Sample sample = new Sample(); 
    sample.PerformSampling(automation); 
    _lastSample = sample; 
} 

Đơn giản bởi vì nó là mô hình đề nghị và cũng đảm bảo tất cả các truy cập vào _lastSample được xử lý một cách chính xác.

NB: Tôi mong đợi tốc độ tương đương, khóa sử dụng lớp Monitor được quản lý sử dụng Interlocked nội bộ.

Edit:

tôi đã bỏ lỡ các khía cạnh back-off, đây là một phiên bản khác:

if (System.Threading.Monitor.TryEnter(_samplingLock)) 
    { 
    try 
    { 
     .... // sample stuff 
    } 
    finally 
    { 
      System.Threading.Monitor.Exit(_samplingLock); 
    } 
    } 

Answer 2

tôi thường tuyên bố một bool bay hơi và làm điều gì đó như:

private volatile bool _isBusy; 
private static Sample _lastSample; 

private Sample DoSomething() 
{ 
    lock(_lastSample) 
    { 
     if(_isBusy) 
      return _lastSample; 
     _isBusy = true; 
    } 

    try 
    { 
     _lastSample = new sameple//do something 
    } 
    finally 
    { 
     lock(_lastSample) 
     { 
      _isBusy = false; 
     } 
    } 
    return _lastSample; 
}