возвращаясь в середине использования блока - PullRequest
Новая
       33

возвращаясь в середине использования блока

170 голосов
/ 19 марта 2009

Что-то вроде: 

using (IDisposable disposable = GetSomeDisposable())
{
    //.....
    //......
    return Stg();
}

Я считаю, что это не подходящее место для заявления о возврате, не так ли?

Ответы [ 7 ]

179 голосов
/ 19 марта 2009

Как отметили несколько других в целом, это не проблема.

Единственный случай, когда это вызовет у вас проблемы, это если вы вернетесь в середине оператора using и дополнительно вернете переменную using. Но опять же, это также может вызвать проблемы, даже если вы не вернетесь и просто сохраните ссылку на переменную. 

using ( var x = new Something() ) { 
  // not a good idea
  return x;
}

Так же плохо 

Something y;
using ( var x = new Something() ) {
  y = x;
}
121 голосов
/ 19 марта 2009

Это прекрасно.

Вы, очевидно, думаете, что 

using (IDisposable disposable = GetSomeDisposable())
{
    //.....
    //......
    return Stg();
}

слепо переводится на: 

IDisposable disposable = GetSomeDisposable()
//.....
//......
return Stg();
disposable.Dispose();

Что, по общему признанию, было бы проблемой, и сделало бы заявление using довольно бессмысленным - вот почему это не , что он делает.

Компилятор обеспечивает удаление объекта до того, как элемент управления покидает блок - независимо от того, как он покидает блок.

91 голосов
/ 19 марта 2009

Это абсолютно нормально - никаких проблем. Почему вы считаете, что это неправильно?

Оператор using - это просто синтаксический сахар для блока try / finally, и, как говорит Грженио, можно и из блока try вернуться нормально.

Возвращаемое выражение будет оценено, затем будет выполнен блок finally, а затем метод вернется.

25 голосов
/ 19 марта 2009

Это будет прекрасно работать, так же как и в середине try{}finally{}

16 голосов
/ 19 марта 2009

Это вполне приемлемо. Оператор с использованием оператора обеспечивает удаление объекта IDisposable независимо от того, что.

С MSDN :

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

10 голосов
/ 02 августа 2017

Код ниже показывает, как работает using: 

private class TestClass : IDisposable
{
   private readonly string id;

   public TestClass(string id)
   {
      Console.WriteLine("'{0}' is created.", id);
      this.id = id;
   }

   public void Dispose()
   {
      Console.WriteLine("'{0}' is disposed.", id);
   }

   public override string ToString()
   {
      return id;
   }
}

private static TestClass TestUsingClose()
{
   using (var t1 = new TestClass("t1"))
   {
      using (var t2 = new TestClass("t2"))
      {
         using (var t3 = new TestClass("t3"))
         {
            return new TestClass(String.Format("Created from {0}, {1}, {2}", t1, t2, t3));
         }
      }
   }
}

[TestMethod]
public void Test()
{
   Assert.AreEqual("Created from t1, t2, t3", TestUsingClose().ToString());
}

Выход:

't1' создан.
't2' создан.
't3' создан.
«Создано из t1, t2, t3» создано.
't3' расположен.
't2' расположен.
't1' расположен.

Распоряжение вызывается после оператора return, но перед выходом из функции.

0 голосов
/ 23 сентября 2016

Возможно, это не на 100% правда, что это приемлемо ...

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

Взять это в качестве примера: 

using (var memoryStream = new MemoryStream())
{
    using (var textwriter = new StreamWriter(memoryStream))
    {
        using (var csv = new CsvWriter(textwriter))
        {
            //..write some stuff to the stream using the CsvWriter
            return memoryStream.ToArray();
        }
    }
}

Я передавал DataTable для вывода в виде csv. С возвратом в середине он записывал все строки в поток, но в выводимом csv всегда отсутствовала строка (или несколько, в зависимости от размера буфера). Это сказало мне, что что-то не было закрыто должным образом.

Правильный способ - убедиться, что все предыдущие использования расположены правильно: 

using (var memoryStream = new MemoryStream())
{
    using (var textwriter = new StreamWriter(memoryStream))
    {
        using (var csv = new CsvWriter(textwriter))
        {
            //..write some stuff to the stream using the CsvWriter
        }
    }

    return memoryStream.ToArray();
}
...