BinaryReader throws NullReferenceException при удалении

Question

У меня есть класс MyClass, которому нужны данные из файла, который будет использоваться на протяжении всего запуска программы. Для чтения в данных у меня есть еще один класс OpenFileService, производный от IDisposable и использует BinaryReader читать все данные:

internal class OpenFileService : IDisposable 
{ 
    #region disposing data 
    bool disposed = false; 

    public void Dispose() 
    { 
     if (!disposed) 
     { 
      Dispose(true); 
      GC.SuppressFinalize(this); 
      disposed = true; 
     } 
    } 

    protected virtual void Dispose(bool disposing) 
    { 
     if (disposing) 
     { 
      br.Dispose(); 
     } 
    } 

    ~OpenFileService() 
    { 
     Dispose(false); 
    } 
    #endregion 

    internal event EventHandler ErrorInFileReadEventHandler; 
    internal event EventHandler SuccessfulFileReadEventHandler; 

    private BinaryReader br; 

    internal void OpenFile(object obj) 
    { 
     MyClass sender = obj as MyClass; 
     bool isWrongFormat = false; 

     try 
     { 
      br = new BinaryReader(File.Open((sender).fileName, FileMode.Open)); 
      //read in header from file. 
      if (//header shows wrong file format) 
      {  
       isWrongFormat = true; 
       throw new System.Exception(); 
      } 
      //read rest of file. 
      SuccessfulFileReadEventHandler(this,null); 
     } 
     catch 
     { 
      if (isWrongFormat) 
       MessageBox.Show("Wrong format."); 
      else 
       MessageBox.Show("Couldn't access."); 
      ErrorInFileReadEventHandler(this, null); 
      return; 
     } 
     finally 
     { 
      this.Dispose(); 
     } 
    } 
} 

И он используется как таковой:

class MyClass 
{ 
    internal filePath; //assuming it has already been determined 

    internal ImageData(string filePath) 
    { 
     this.filePath = filePath; 
     OpenFileService ofs = new OpenFileService(); 
     ofs.ErrorInFileReadEventHandler += new EventHandler(UnsuccessfulFileRead); 
     ofs.SuccessfulFileReadEventHandler += new EventHandler(SuccessfulFileRead); 
     Thread thread = new Thread(new ParameterizedThreadStart(ofs.OpenFile)); 
     thread.IsBackground = true; 
     thread.Start(this); 
    } 
} 

Теперь, когда файл формат является неправильным, и я сам создаю исключение в блоке try, все работает без проблем, но когда файл фактически не может быть доступен (например, с защитой от записи), br.Dispose(); создает NullReferenceException, и я не могу понять, почему. Я действительно урезал код до его простых вещей, надеюсь, он не слишком длинный.

Редактировать: Возможный ответ можно найти из принятого ответа here в качестве рекомендуемого ответа Microsoft.

Answer 1

вопрос может стать понятнее, если вы разделяете ваш файл открытой логики через две строки:

try 
{ 
    var fs = File.Open((sender).fileName, FileMode.Open); 
    br = new BinaryReader(fs); 
} 
finally 
{ 
    br.Dispose(); 
} 

При сбое File.Open вызова, исключение без ничего быть назначено br поля. Когда вы попытаетесь утилизировать его в блоке finally, все равно будет null, таким образом, ваше исключение.

Edit: Как я хотел бы предложить исправить эту проблему:

try 
{ 
    using (FileStream fs = File.Open(sender.fileName, FileMode.Open)) 
    using (BinaryReader br = new BinaryReader(fs)) 
    {   
     //read in header from file. 
     if (/*header shows wrong file format*/) 
     {  
      isWrongFormat = true; 
      MessageBox.Show("Wrong format."); 
      ErrorInFileReadEventHandler(this, null); 
     } 
     else 
     { 
      //read rest of file. 
      SuccessfulFileReadEventHandler(this, null); 
     } 
    } 
} 
catch 
{ 
    MessageBox.Show("Couldn't access."); 
    ErrorInFileReadEventHandler(this, null); 
} 

В процессе, я понижен ваш BinaryReader от поля к локальной переменной. Поскольку вы обращаетесь к нему только в рамках метода OpenFile (и удаляя его перед возвратом), нет необходимости в том, чтобы его ссылка сохранялась вне метода.