Исключение сбоя при удалении результата?

Question

Рассмотрим следующий фрагмент кода:

public static Stream OpenWavStream(string path) 
{ 
    var stream = new FileStream(path, FileMode.Open); 

    try 
    { 
     stream.Seek(44, SeekOrigin.Current); 
    } 
    catch (Exception) 
    { 
     stream.Dispose(); 
     throw; 
    } 

    return stream; 
} 

Я открывающий поток WAV, данные которого всегда начинается с 44. Если смещение стремится к тому, что смещение не может, поток расположен, в противном случае он возвращается. Учитывая, что catch (Exception) is considered bad practice, уместно ли в этом случае?

Следует ли скорее исследовать конкретные исключения (даже если поток должен быть удален, если в вызове Stream.Seek происходит какое-либо исключение) или переместить его в блок finally?

Answer 1

catch (Exception) плохая практика, если вы используете его, чтобы проглотить исключения и их не обрабатываете. Вы немедленно перебрасываете исключение и делаете это правильно (например, вы не делаете throw ex;). Вам нужно будет удалить поток для любого исключения, поэтому вы не должны улавливать определенные исключения здесь.

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

//In your callee code 
using (var stream = new FileStream(path, FileMode.Open)) 
{ 
    ConfigureStream(steam); 
    //Other stuff.. 
} 

public static void ConfigureStream(Stream stream) 
{ 
    stream.Seek(44, SeekOrigin.Current); 
} 

Или вы можете сначала проверить длину потока, чтобы полностью исключить исключение.

Answer 2

Только в том случае, если Stream не загружается. Что-то я использую:

string fileName = "C:\\PCM.wav"; 

if (!System.IO.File.Exists(fileName)) 
{ 
LogStatus("Wave file not found."); 
return; 
} 
else 
{ 
WaveFileByteArray = File.ReadAllBytes(fileName); 
LogStatus("Wave file Loaded!" + Environment.NewLine); 
} 

Это работает нормально.

затем играть/Использование:

System.Media.SoundPlayer soundPlayer; 
soundPlayer.Stream.Seek(0, SeekOrigin.Begin); 
soundPlayer.Stream.Write(WaveFileByteArray, 0, WaveFileByteArray.Length); 
soundPlayer.Play(); 

Опираясь на Exception, чтобы поймать возможных ошибок, как представляется, не лучшая практика, если ошибка не является неожиданным. Лучше всего справляться с возможными ошибками до их появления.

Answer 3

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

public static Stream OpenWavStream(string path) 
    { 
     var stream = new FileStream(path, FileMode.Open); 
     stream.Seek(44, SeekOrigin.Current); 
     return stream; 
    } 

public static void UseWaveStream() 
    { 
     try 
     { 
      using(Stream thisStream = OpenWavStream("C:\\myfile.txt")) 
      { 
       // do whatever 
      } 
     } 
     catch(Exception ex) 
     { 
      Console.WriteLine(ex.ToString()); 
     } 
    }