C# Monitor.Exit in Try/Catch

Question

Ich Fehlerbehebung alten Quellcode und stieß auf eine Anweisung wie folgt:

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); 
     } 
    } 
} 

Der Code wird mit einem Gibt es einen sicheren Weg System.Threading.SynchronizationLockException: Object synchronization method was called from an unsynchronized block of code.

Absturz zu Rufen Sie Monitor.Exit() auf der fehlerhaften Bedingung und wenn die normale Ausführung beendet ist?

Answer 1

Tun Sie es einfach im finally-Block, es läuft immer, auch wenn oben eine Ausnahme ausgelöst wird. Auch der Fangabschnitt ist nicht notwendig.

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