7
Powiedziano mi, że muszę dispose of instances of my Entity Framework repository classes i stworzyłem klasę bazową, aby wymusić tę implementację.Czy muszę wdrożyć IDisposable na wszystkich klasach lub czy klasa podstawowa jest wystarczająca?
Muszę skontaktować się z ekspertami: czy dopuszczalne jest wdrożenie IDisposable przez klasę podstawową?
Należy zauważyć, że klasa repozytorium nie ma zmiennych członków klasy.
/// 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);
}
}
W moim przypadku nie mam nic lokalnego w mojej dziedziczonej klasie repozytorium. Jeśli tak, to czy moja implementacja CompanyRepository jest kompletna? – LamonteCristo
Jeśli w klasie pochodnej nie ma niczego, co wymaga jawnego usunięcia, prawdopodobnie wystarczająca jest implementacja klasy podstawowej. –
Tylko klasa podstawowa powinna faktycznie * implementować * 'IDisposable'. Klasy pochodne powinny na ogół dodawać swój kod czyszczący, zastępując 'Dispose (bool)', zamiast dodawać nową implementację 'IDisposable.Dispose()'. – supercat