Зачем мне нужно создать свойство, чтобы проверить, был ли ресурс удален при использовании одноразовой пясти в C#?

Question

Мне нужно написать класс, где я хочу, чтобы потребитель мог утилизировать код, обернув код оператором using(...) в C#.

Для этого я должен реализовать интерфейс Microsoft IDisposable.

на основе Microsoft approach on implementing it, я должен сделать что-то вроде этого

общий интерфейс, который так

public interface ISomeClass : IDisposable 
{ 
    // ... some methods to include 
} 

public class SomeClass : ISomeClass 
{ 
    private readonly TimeTrackerContext _context; 

    private bool disposed = false; 

    protected virtual void Dispose(bool disposing) 
    { 
     if (!this.disposed && disposing && _context != null) 
     { 
      _context.Dispose(); 

     } 
     this.disposed = true; 
    } 

    public void Dispose() 
    { 
     Dispose(true); 
     GC.SuppressFinalize(this); 
    } 
} 

Я пытаюсь научиться C# правильный путь, так что я есть вопрос относительно этой реализации.

Вопрос

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

Другими словами, не могу ли я просто проверить, является ли значение _context нулевым, прежде чем удалять его? Что-то вроде этого

public class SomeClass : ISomeClass 
{ 
    private readonly TimeTrackerContext _context; 

    private void SelfDisposing() 
    { 
     if (_context != null) 
     { 
      _context.Dispose(); 
     } 

    } 

    public void Dispose() 
    { 
     SelfDisposing(); 
     GC.SuppressFinalize(this); 
    } 

    private void SelfDisposing(bool disposing) 
    { 
     if (_context != null && !this.disposed && disposing) 
     { 
      _context.Dispose(); 
     } 

     this.disposed = true; 
    } 
} 

Answer 1

_contextне будет null, если объект уже был удален. Он по-прежнему будет ссылаться на уже расположенный объект.

Теперь, если вы можете определить, расположен ли ваш объект без необходимости хранить логическую переменную, тогда это нормально.

Что вы не должно на самом деле есть GC.SuppressFinalize В вашем объекте нет финализатора, поэтому нечего подавлять.

Answer 2

В поле (или свойство), указывающий, что Dispose уже называются не сильно требуется для реализации Dispose(bool) метода самих, как указано в следующем правиле:

✓ DO позволяют Dispose (BOOL) метод, который будет вызываться более одного раза. Метод может не делать ничего после первого вызова.

Это необходимо, если у вас есть другие методы/свойства, и вы хотите реализовать следующее правило из того же документа:

✓ DO бросить ObjectDisposedException от любого члена, который не может быть использован после того, как объект имеет были уничтожены.

Answer 3

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

public sealed class SomeClass : ISomeClass 
{ 
    private readonly TimeTrackerContext _context; 

    public void Dispose() 
    { 
     _context.Dispose(); 
    } 
} 

Вторая мысль заключается в том, что само поле _context. Если вы передаете контекст через конструктор, вы на самом деле не знаете, действительно ли вам нужно его утилизировать. .NET Stream-based классы (такие как StreamWriter) также страдают от этой проблемы до .NET 4.0. И решение заключалось в том, чтобы написать Stream wrappers (например NonDisposableStreamWrapper), которые не располагают внутренним потоком, когда вызывается dispose. Действительно хорошим решением этой проблемы является добавление дополнительного объекта в конструктор, который указывает, следует ли распоряжаться внутренним классом или нет. Например:

public sealed class SomeClass : ISomeClass 
{ 
    private readonly TimeTrackerContext _context; 
    private bool _dispose; 

    public SomeClass(TimeTrackerContext context, bool dispose = true) 
    { 
     _context = context; 
     _dispose = dispose; 
    } 

    public void Dispose() 
    { 
     if (_dispose) 
     { 
      _context.Dispose(); 
     } 
    } 
}