lock и try-catch

Question

Как правильно написать когда операция требует одновременно и то и другое?

try
{
 lock(locker)
 {
 ....
 }
}
catch(...)
{
 ...
}

или так

lock(locker)
{
 try
 {
 ...
 }
 catch(...)
 {
 ...
 }
 }

И как быть в случае возникновения исключения, если оно будет залочено?

• user176262: и так, и так можно. "исключения ... будет залочено" - это как?
• Bulson: @Igor то есть если в операции возникла ошибка, то ведь как-то нужно будет снять лок, он же из-за ошибки останется активным, как я понимаю.
• Pavel Mayorov: @Bulson с чего бы?
• Pavel Mayorov: Почитайте в любом учебнике что конструкция lock делает...
• user176262: Нет, не останется. lock - это сокращенная запись Monitor.Enter/Exit с try-finaly. msdn.microsoft.com/en-us/library/c5kehkcz.aspx

Answer 1

На SO настоятельно не рекомендуют использовать вместе c lock конструкцию try-catch. А советуют поступать так:

lock(whatever)
{
 try
 {
 MakeAMess();
 }
 finally
 {
 CleanItUp();
 // Либо доводим операцию до конца, либо откатываем в прежнее состояние
 }
}

Там пишут, что выбрасывание исключения в lock нарушает саму суть ради чего существует этот самый lock. Поэтому если возникло исключение есть два варианта:

1. Прибить процесс совсем с помощью Environment.FailFast()
2. Либо, если это возможно, откатить в прежнее состояние, до lock.

В оригинале можно ознакомиться здесь.