Закрытие потока после исключения - PullRequest
12 голосов
/ 22 января 2010

Я открываю поток и затем десериализирую данные в потоке. Однако я добавил элемент в объект SavedEventSet, поэтому теперь, когда я пытаюсь открыть старый файл, он вызывает исключение в строке десериализации.

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

Как закрыть поток после этого исключения? Если я помещаю stream.Close () в catch или finally, он жалуется на попытку доступа к неназначенной локальной переменной. Кажется плохой практикой открывать случайный файл, который, я знаю, там есть. Есть ли способ открыть поток так, чтобы он был похож на пустой конструктор, чтобы он выглядел так, как будто он назначен?

Спасибо

SavedEventSet sES;
OpenFileDialog oFD = new OpenFileDialog();
Stream stream;
BinaryFormatter bF;

try
{
    oFD.InitialDirectory = this.path;
    oFD.Title = "Open Event Saved File.";
    oFD.ShowDialog();

    if(oFD.FileName.Contains(".sav"))
    {
        stream = File.Open(oFD.FileName, FileMode.Open);
        bF = new BinaryFormatter();

        sES = (SavedEventSet)bF.Deserialize(stream);
        stream.Close();

    }
}
catch (Exception ex)
{
    stream.Close();
    /*handle Exception*/
}

Ответы [ 4 ]

27 голосов
/ 22 января 2010

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

using(Stream stream = File.Open(oFD.FileName, FileMode.Open))
{
    bF = new BinaryFormatter();

    sES = (SavedEventSet)bF.Deserialize(stream);
}
6 голосов
/ 22 января 2010

Установите для потока значение null перед блоком try.

В вашей проверке перехвата, если поток не равен нулю, если нет, то закройте поток.

  SavedEventSet sES;
  OpenFileDialog oFD = new OpenFileDialog();
  Stream stream = null;
  BinaryFormatter bF;

  try
  {
    oFD.InitialDirectory = this.path;
    oFD.Title = "Open Event Saved File.";
    oFD.ShowDialog();

    if (oFD.FileName.Contains(".sav"))
    {
      stream = File.Open(oFD.FileName, FileMode.Open);
      bF = new BinaryFormatter();

      sES = (SavedEventSet)bF.Deserialize(stream);
      stream.Close();

    }
  }
  catch (Exception ex)
  {
    if (stream != null)
      stream.Close();
    /*handle Exception*/
  }
4 голосов
/ 22 января 2010

Использовать блок finally, он будет выполняться независимо от того, произошло исключение или нет:

try
{
  oFD.InitialDirectory = this.path;
  oFD.Title = "Open Event Saved File.";
  oFD.ShowDialog();

  if(oFD.FileName.Contains(".sav"))
  {
    stream = File.Open(oFD.FileName, FileMode.Open);
    bF = new BinaryFormatter();

    sES = (SavedEventSet)bF.Deserialize(stream);
  }
}
catch (Exception ex)
{
  /*handle Exception*/
}
finally
{
  if (stream != null)
    stream.Close();
}
0 голосов
/ 22 января 2010
SavedEventSet sES; 
OpenFileDialog oFD = new OpenFileDialog(); 
BinaryFormatter bF; 

try 
{ 
    oFD.InitialDirectory = this.path; 
    oFD.Title = "Open Event Saved File."; 
    oFD.ShowDialog(); 

    if(oFD.FileName.Contains(".sav")) 
    { 
        using(Stream stream = File.Open(oFD.FileName, FileMode.Open))
        {
           bF = new BinaryFormatter(); 

           sES = (SavedEventSet)bF.Deserialize(stream); 
           stream.Close(); 
        }
    } 
} 
catch (Exception ex) 
{ 
    /*handle Exception*/ 
} 
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...