Лучшая практика возврата от использования блоков

Question

Какой способ лучше использовать: вернуть значение из метода внутри оператора using или объявить переменную раньше, установить ее и вернуть ее после?

public int Foo() 
{ 
    using(..) 
    { 
    return bar; 
    } 
} 

или

public int Foo() 
{ 
    var b = null; 
    using(..) 
    { 
    b = bar; 
    } 
    return b; 
} 

Answer 1

Я предпочитаю первый пример. Меньшее число переменных, меньшее количество строк кода, легче следовать, проще в обслуживании ...

public int Foo() 
{ 
    using(..) 
    { 
    return bar; 
    } 
} 

Answer 2

Нет причин не так, что using оператор переводит в try...finally блок и finally часть гарантированно будет выполнена (даже через возвращение или необработанное исключение).

Answer 3

Это действительно подходит для личных предпочтений. Вы найдете аргументы по обеим сторонам этого конкретного забора. Сам я предпочитаю вариант 1: возвращение, как только сможете. Я считаю, что это лучше отражает смысл кода; нет оснований держаться дольше, чем нужно. Если вы завершили всю свою работу, вернитесь.

Иногда у вас будет несколько возможных точек возврата, а работа «end-of-method» (регистрация, очистка) может привести вас к одному оператору возврата. Ничего страшного в этом, но вы часто можете справиться с этими ситуациями в блоках finally или с аспектами в аспектно-ориентированном программировании.

Answer 4

Следуя принципу «меньше больше» (на самом деле только вариант KISS), первый. Существует меньше строк кода для поддержки, без изменений в семантике и без потери удобочитаемости (возможно, этот стиль легче читать).

Answer 5

От using Statement - MSDN

используя заявление гарантирует, что Dispose вызывается даже если исключение происходит в то время как вы вызываете методы на объекте. Вы можете достичь результата , поставив объект внутри блока try, а затем вызывая Утилизируйте в блок finally; на самом деле, так используется оператор using , переведенный компилятором.

От try-finally (C# Reference)

наконец используется, чтобы гарантировать операторный блок кода выполняет независимо от того, как вышел из предыдущего попытка блока.

Чтобы ответить на ваш вопрос, да, его нормально возвращать из используемого заявления.

Answer 6

Второй, безусловно, лучше, и вы можете убедиться, что он отлично работает, написав тестовую программу.

Сам оператор using не может иметь значение, которое является ограничением.Предположим, у вас есть метод, называемый Open, который возвращает открытый FileStream, и вы хотите, чтобы получить длину файла:

Console.WriteLine(Open().Length); 

Исправлена ​​ошибка есть, что вы не располагая FileStream. Таким образом, вы должны написать (аналогично ваш пример):

long length; 

using (FileStream file = Open()) 
    length = file.Length; 

Console.WriteLine(length); 

Но с simple extension method, вы можете написать вместо этого:

Console.WriteLine(Open().Use(file => file.Length)); 

Приятных и аккуратные, и FileStream получают должным образом утилизировать.

Answer 7

Я чувствую второй один лучше

public int Foo() 
{ 
    using(..) 
    { 
    return bar; 
    } 
} 

Одна вещь, которая возникает в виду при использовании этого пути является то, что мы возвращаемся в между использованием так будет объект (который мы завернутые в использовании) получит расположенный , ответ yes , потому что оператор using - это просто смесь блока try/finally, также хорошо вернуться из блока try. Возвращаемое выражение будет оценено, тогда будет выполнен блок finally, и этот метод будет return then.So go Ahead :)