Одноразовые, использующие & Try / Catch Blocks

Question

Имея сегодня ментальный блок, нужна рука, проверяющая мою логику, не fubar'ed.

Традиционно я бы сделал ввод / вывод файла подобным этому:

FileStream fs = null; // So it's visible in the finally block
try
{
   fs = File.Open("Foo.txt", FileMode.Open);

   /// Do Stuff
}
catch(IOException)
{
   /// Handle Stuff
}
finally
{
   if (fs != null)
      fs.Close();
}

Однако это не очень элегантно.

В идеале я бы хотел использовать блок using, чтобы избавиться от потока файлов, когда я закончу, однако я не уверен насчет синергии между использованием и try / catch.

Вот как я хотел бы реализовать вышеупомянутое:

try
{
   using(FileStream fs = File.Open("Foo.txt", FileMode.Open))
   {
      /// Do Stuff
   }
}
catch(Exception)
{
   /// Handle Stuff
}

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

Answer 1

Вы просто параноик и это сработает так, как вы намереваетесь:)

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

Ваш код похож на:

try
{
   FileStream fs = null;
   try
   {
       fs = File.Open("Foo.txt", FileMode.Open);
       // Do stuff
   }
   finally
   {
       if (fs != null)
       {
           fs.Dispose();
       }
   }
}
catch(Exception)
{
   /// Handle Stuff
}

Answer 2

Вам не нужно try..finally, если у вас есть using(). Они выполняют одну и ту же операцию.

Если вы не уверены, укажите Reflector на вашу сборку и сравните сгенерированный код.

Answer 3

Блок использования будет работать точно так же, как вы переводите, блок использования на самом деле просто

try
{
   FileStream fs = null;
   try
   {
        fs = File.Open("Foo.txt", FileMode.Open))
        //Do Stuff
   }
   finally
   {
      if(fs != null)
          fs.Dispose();
   }
}
catch(Exception)
{
   /// Handle Stuff
}

Answer 4

    try
    {
        FileStream fs = null;
        try
        {
           fs = File.Open("Foo.txt", FileMode.Open);

        }
        finally
        {
           fs.Dispose();
        }
    }
    catch(Exception)
    {
       /// Handle Stuff
    }

второй кусок кода переводится в

Answer 5

Это будет работать - внутренне оператор using компилируется так же, как и блок try-finally

Answer 6

Не волнуйтесь, он очистится, как и ожидалось, и чище, чем ваш оригинал.

На самом деле гораздо чаще встречаются операторы try / finally aka using в бизнес-логике и try / catch в обработчике верхнего уровня на уровне пользовательского интерфейса или на границе физического уровня. Что-то вроде:

try
{
    DoStuffWithFile("foo.txt");
}
catch(Exception ex)
{
   ...
}

и

public void DoStuffWithFile(string fileName)
{
    using(FileStream fs = File.Open(fileName,...))
    {
        // Do Stuff
    }
}