Phát hành một Mutex

Question

Tôi có một ứng dụng web cần sử dụng bộ nhớ cache của ứng dụng để lưu trữ dữ liệu (do chi phí cao khi nhận được yêu cầu ona dữ liệu theo yêu cầu). Xem bài trước ở https://stackoverflow.com/a/16961962/236860

Cách tiếp cận này dường như làm việc tốt, nhưng tôi nhìn thấy các lỗi thỉnh thoảng sau do lỗi trang web của:

System.ApplicationException: Object synchronization method was called from an 
unsynchronized block of code. 
at System.Threading.Mutex.ReleaseMutex() 
at InboxInsight.Web.Web_Controls.Twitter.TwitterFeed.GetTwitterData(HttpContext context) 
at InboxInsight.Web.Web_Controls.Twitter.TwitterFeed.ProcessRequest(HttpContext context) 
at System.Web.HttpApplication.CallHandlerExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute() 
at System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously) 

Để tham khảo, đây là khối mã:

public string GetData(HttpContext context) 
{ 
    var cache  = context.Cache; 
    Mutex mutex = null; 
    string data = (string)cache[CacheKey]; 

    // Start check to see if available on cache 
    if (data == null) 
    { 
     try 
     { 
      // Lock base on resource key 
      // (note that not all chars are valid for name) 
      mutex = new Mutex(true, CacheKey); 

      // Wait until it is safe to enter (someone else might already be 
      // doing this), but also add 30 seconds max. 
      mutex.WaitOne(30000); 

      // Now let's see if some one else has added it... 
      data = (string)cache[CacheKey]; 

      // They did, so send it... 
      if (data != null) 
      { 
       return data; 
      } 


      // Still not there, so now is the time to look for it! 
      data = GetSlowFeed(context); 

      cache.Remove(CacheKey); 
      cache.Add(CacheKey, data, null, GetExpiryDate(), 
       TimeSpan.Zero, CacheItemPriority.Normal, null); 
     } 
     finally 
     { 
      // Release the Mutex. 
      if (mutex != null) 
      { 
       mutex.ReleaseMutex(); 
      } 
     } 
    } 

    return data; 
} 

Từ những gì tôi đã nghiên cứu, nó cho thấy vấn đề này được gây ra bởi một luồng quá trình cố gắng để phát hành một Mutex mà nó không tạo ra, nhưng tôi không hiểu làm thế nào điều này có thể xảy ra.

Có ai có thể đề xuất cách tôi có thể cấu trúc lại mã để tránh sự cố này không?

Answer 1

Bạn không xử lý trường hợp theo đó mutex.WaitOne trả về số lần xuất hiện sai. Nếu WaitOne trả về false bạn không sở hữu mutex, do đó bạn không cần phải giải phóng nó.

bool iOwnTheMutex; 

try { 
    // set up mutex here... 
    iOwnTheMutex = mutex.WaitOne(2000); 
    if (iOwnTheMutex) { 
     // do what you need to do 
    } 
} 
finally { 
    if (mutex != null && iOwnTheMutex) { 
     mutex.ReleaseMutex(); 
    } 
}