1

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

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

или так

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

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

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

1 ответ 1

-1

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

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

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

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

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

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.