Чи хороший підхід для виклику 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однак буде викликаний незалежно від того, що він знаходиться всередині останнього блоку.

Ваш код по суті такий самий, як цей (якщо це полегшує розуміння):

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речення під капотом) завжди виконуються, коли область залишається, незалежно від того, як.

Однак це справедливо лише для тверджень, які знаходяться в блоці нарешті (які не можна явно встановити при використанні using). Тому у вашому прикладі scope.Complete()ніколи не викликатимуть (я сподіваюсь, компілятор попередить вас про недосяжний код).

Загалом, це хороший підхід. Але у вашому випадку, якщо ви повернетесь перед тим scope.Complete(), як зателефонувати , він просто смітить TransactionScope. Залежить від вашого дизайну.

Отже, у цьому прикладі Complete () не викликається, а область дії розподілена, припускаючи, що вона успадковує інтерфейс IDisposable.

scope.Complete слід однозначно викликати раніше return. Компілятор відобразить попередження, і цей код ніколи не буде викликаний.

Що стосується returnсамого себе - так, можна з упевненістю назвати це внутрішньою usingзаявою. Використання перекладено для спроби остаточно блокувати за кадром, і нарешті блок, безумовно, повинен бути виконаний.

У наведеному вами прикладі є проблема; scope.Complete()ніколи не називається. По-друге, не є доброю практикою використовувати returnтвердження всередині usingвисловлювань. Зверніться до наступного:

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

У цьому простому прикладі справа в тому, що; Значення scopeбуде нульовим, коли використання оператора закінчено.

Тому краще не повертатися всередину, використовуючи оператори.

Щоб переконатися, що 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 подбає про все, що не є посиланням. Отже, якщо немає об’єкта, який не може бути підхоплений ГК, немає ніяких проблем.