Это хороший подход - вызвать return внутри с помощью оператора {}?

Я просто хочу знать, безопасен ли / хороший подход для вызова returnвнутри usingблока.

Например,

using(var scope = new TransactionScope())
{
  // my core logic
  return true; // if condition met else
  return false;
  scope.Complete();
}

Мы знаем, что последняя самая фигурная скобка dispose()будет снята. Но что будет в приведенном выше случае, поскольку returnэлемент управления переходит из заданной области (AFAIK) ...

1. Мне scope.Complete()позвонят?
2. То же самое и с dispose()методом прицела .

Совершенно безопасно вызывать returnвнутри вашего usingблока, поскольку используемый блок - это просто try/finallyблок.

В приведенном выше примере после возврата trueобласть видимости будет удалена и значение будет возвращено. return falseИ scope.Complete()будет не дозвонились. Disposeоднако будет вызываться независимо от того, что находится внутри блока finally.

Ваш код по сути такой же (если это упрощает понимание):

var scope = new TransactionScope())
try
{
  // my core logic
  return true; // if condition met else
  return false;
  scope.Complete();
}
finally
{
  if( scope != null) 
    ((IDisposable)scope).Dispose();
}

Имейте в виду, что ваша транзакция никогда не будет зафиксирована, поскольку нет возможности scope.Complete()выполнить транзакцию.

Это нормально - finallyпредложения (что и делает закрывающая фигурная скобка usingпредложения под капотом) всегда выполняются при выходе из области видимости, независимо от того, как.

Однако это верно только для операторов, которые находятся в блоке finally (который не может быть явно установлен при использовании using). Следовательно, в вашем примере scope.Complete()он никогда не будет вызван (хотя я ожидаю, что компилятор предупредит вас о недоступности кода).

В общем, подход хороший. Но в вашем случае, если вы вернетесь до вызова scope.Complete(), он просто уничтожит TransactionScope. Зависит от вашего дизайна.

Итак, в этом примере Complete () не вызывается, а область видимости удаляется, предполагая, что она наследует интерфейс IDisposable.

scope.Complete обязательно должен вызываться раньше return. Компилятор выдаст предупреждение, и этот код никогда не будет вызван.

Что касается returnсамого себя - да, это смело можно назвать внутренним usingутверждением. Использование переводится как блок try-finally за сценой, и блок finally обязательно должен быть выполнен.

В приведенном вами примере есть проблема; scope.Complete()никогда не называется. Во-вторых, использование returnоператоров внутри usingоператоров - не лучшая практика . См. Следующее:

using(var scope = new TransactionScope())
{
    //have some logic here
    return scope;      
}

В этом простом примере суть в том, что; значение scopeбудет нулевым, когда оператор using будет завершен.

Так что лучше не возвращаться внутрь с помощью операторов.

Чтобы убедиться, что scope.Complete()функция будет вызываться, оберните ее try/finally. disposeНазываются , потому что вы должны обернуть его с , usingчто является альтернативой try/finallyблока.

using(var scope = new TransactionScope())
{
  try
  {
  // my core logic
  return true; // if condition met else
  return false;
  }
  finally
  {
   scope.Complete();
  }
}

В этом примере scope.Complete () никогда не будет выполняться. Однако команда return очистит все, что назначено в стеке. GC позаботится обо всем, на что нет ссылок. Итак, если нет объекта, который не может быть захвачен сборщиком мусора, проблем нет.