Располагая базовый класс, прежде чем унаследовал класс

Question

У меня есть два класса:

public abstract class UnitOfWorkBase: IDisposable { } 
public sealed class UnitOfWorkSql: UnitOfWorkBase { } 

метод производного класса Dispose обычно выглядит следующим образом:

protected override void Dispose (bool disposing) 
{ 
    if (!this.Disposed) 
    { 
     if (disposing) 
     { 
      // Managed. 
     } 

     // Unmanaged. 

     this.Disposed = true; 
    } 

    base.Dispose(disposing); 
} 

Я всегда видел вызов base.Dispose(disposing) в конец метода. Однако, в недавнем случае, существует необходимость располагать объекты базового класса перед тем объектов производного класса следующим образом:

protected override void Dispose (bool disposing) 
{ 
    base.Dispose(disposing); 

    if (!this.Disposed) 
    { 
     if (disposing) 
     { 
      // Managed. 
     } 

     // Unmanaged. 

     this.Disposed = true; 
    } 
} 

Я не уверен, если это считается плохой практикой. Есть ли что-то, на что можно обратить внимание?

Answer 1

Думаю, вы должны сломать обычный образец здесь. Вместо того, чтобы:

class BaseClass 
{ 
    protected virtual void Dispose(bool disposing) 
    { 
     if (disposing) 
     { 
      //dispose my resources 
     } 
    } 
} 

написать что-то вроде:

class BaseClass 
{ 
    private void Dispose(bool disposing) 
    { 
     if (disposing) 
     { 
      //dispose my resources 
      DisposeManagedOverride(); 
     } 

     CloseUnmanagedOverride(); 
    } 

    protected virtual void DisposeManagedOverride() {} 

    protected virtual void CloseUnmanagedOverride() {} 
} 

Таким образом, вы обеспечите надлежащий порядок освобождения ресурсов.