Korzystanie z ObjectDisposedException

Question

Podczas implementacji wzoru Dispose, właściwości obiektu powinny generować ObjectDisposedException po usunięciu obiektu.

Wydaje się, że wiele powtarzających się kodów, jeśli każda właściwość i metoda ma linię kodu na początku settera, a program pobierający wyrzuci ten wyjątek, nawet jeśli jest to tylko ThrowIfDisposed(). Jeśli masz 40 właściwości, to jest 80 powtarzających się linii. W dużej klasie może się bardzo szybko sumować.

Nie byłem w stanie znaleźć lepszego sposobu, więc wygląda na to, że tak właśnie jest, ale myślałem, że zapytam. Czy istnieje lepszy sposób?

Answer 1

Jeśli masz 40 właściwości w jednej klasie, prawdopodobnie masz problem z projektowaniem, ale dzieje się dalej ... To naprawdę zależy od konkretnego przypadku użycia, ale powinieneś rzucić tylko ObjectDisposedException, jeśli właściwość nie może faktycznie zostanie rozwiązany, ponieważ klasa została usunięta. Dać konkretny przykład:

public class MyStream : IDisposable 
{ 
    private string name; 
    private Stream stream; 

    public MyStream(string name) 
    { 
     this.Name = name; 
     this.Stream = new FileStream(...); 
    } 

    public string Name 
    { 
     get 
     { 
      return this.name; 
     } 
    } 

    public Stream Stream 
    { 
     get 
     { 
      if (this.IsDisposed) 
      { 
       throw new ObjectDisposedException(); 
      } 

      return this.stream; 
     } 
    } 

    private bool IsDisposed { get; set; } 

    public void Dispose() 
    { 
     if (!this.IsDisposed) 
     { 
      this.IsDisposed = true; 
      this.Stream.Dispose(); 
      this.Stream = null; 
     } 
    } 
} 

tutaj właściwość Stream zgłasza wyjątek, ponieważ podstawowa strumienia zostały usunięte, tak, że nie można ich stosować. Ale Name nie ma powodu do rzucania. Stamtąd, w wielu przypadkach można na czynniki swój kod tak, że dostęp do rzeczywistej własności problematycznej rzuca wyjątek:

public int Length 
{ 
    get 
    { 
     // Since we access the Stream property, the ObjectDisposedException 
     // will be thrown when the class is disposed 
     return this.Stream.Length; 
    } 
} 

---

Jeśli nadal masz zbyt wiele właściwości do dekoracji nawet po zastosowaniu tego wzoru, możesz używać bibliotek takich jak Fody, które pozwalają wstrzykiwać zachowanie w czasie kompilacji. Używając go, możesz utworzyć atrybut, który chcesz zastosować w zależności od potrzeb.

Answer 2

Sprawdź inny wątek: Handling ObjectDisposedException correctly in an IDisposable class hierarchy

Nie trzeba rzucać wyjątek każdym razem, gdy ktoś własności dostępu klasy implementującej :IDisposable. Jest to konieczne tylko wtedy, gdy uzyskujesz dostęp do obiektu, który musi zostać usunięty (np. Zapis/odczyt do plików, dostęp do gniazd sieciowych ...). We wszystkich innych przypadkach możesz po prostu przejść z pomysłem do normalnej klasy.

Zwykle obiekty/klasy nie mają tak dużej liczby właściwości, ponieważ istnieje opcja podziału ich na kilka klas -> które pozwolą Ci wykonać tylko część tego :IDisposable, a reszta będzie niezależna od IDisposable klasa.