7
Me dijeron que necesito dispose of instances of my Entity Framework repository classes y creé una clase base para hacer cumplir esta implementación.¿Debo implementar IDisposable en todas las clases, o es una clase base suficiente?
Necesito consultar con los expertos: ¿es aceptable implementar IDisposable a través de una clase base?
Tenga en cuenta que la clase de repositorio no tiene variables de miembro de clase.
/// Sample repository. Note that I return List<T> as IEnumerable,
/// and I use IDisposable
///
public class CompanyRepository : DisposableBase, ICompanyRepository
{
public IEnumerable<CompanyDetail> GetOneCompany(int? CompanyID)
{
var t = from c in _entities.CompanyDetail
where c.CompanyID == CompanyID.Value
select c;
return t.ToList();
}
}
/// <summary>
/// Disposable implementation based on advice from this link:
/// from Http://www.asp.net/entity-framework/tutorials/implementing-the-repository-and-unit-of-work-patterns-in-an-asp-net-mvc-application
/// </summary>
public class DisposableBase : IDisposable
{
protected TLSAdminEntities1 _entities;
public DisposableBase()
{
_entities = new TLSAdminEntities1();
disposed = false;
}
private bool disposed ;
protected virtual void Dispose(bool disposing)
{
if (!this.disposed)
{
if (disposing)
{
_entities.Dispose();
}
}
this.disposed = true;
}
public void Dispose()
{
Dispose(true);
GC.SuppressFinalize(this);
}
}
En mi caso no tengo nada local en mi clase de repositorio heredado. Si este es el caso, ¿mi implementación de CompanyRepository está completa? – LamonteCristo
Si no hay nada en la clase derivada que requiera eliminación explícita, la implementación de la clase base es probablemente suficiente. –
Solo la clase base debería * implementar * 'IDisposable'. Las clases derivadas generalmente deberían agregar su código de limpieza anulando 'Dispose (bool)', en lugar de agregar una nueva implementación de 'IDisposable.Dispose()'. – supercat