在try/catch語句C＃Monitor.Exit

Question

我疑難解答老的源代碼，並遇到了如下語句：

if (Monitor.TryEnter(lockObj)) 
{ 
    try 
    { 
     //does something 
     if (failing_condition) 
     { 
      Monitor.Exit(lockObj); 
      throw new Exception("Oops!"); 
     } 
     catch (Exception ex) 
     { 
      throw ex; 
     } 
     finally 
     { 
      Monitor.Exit(lockObj); 
     } 
    } 
} 

的代碼與System.Threading.SynchronizationLockException: Object synchronization method was called from an unsynchronized block of code.

崩潰是否有一個安全的方式來撥打Monitor.Exit()在失敗的條件和正常執行完成？

Answer 1

只要在finally塊中做到這一點，即使上面拋出異常，它也會始終運行。捕捉部分也是不必要的。

if(Monitor.TryEnter(lockObj)) { 
    try { 
    //does something 
    if(failing_condition) {  
     throw new Exception("Oops!"); 
    } 
    } 
    finally { 
    Monitor.Exit(lockObj); 
    } 
}