¿Cómo podría imponer IDisposible en este contexto?

Question

Estoy usando Entity Framework 4 junto con MSSQL para almacenar y acceder a los datos en mi aplicación Windows Forms.

Aquí es una clase de ejemplo que utilizo para acceder a los datos:

public class StudentRepository : IDisposable 
{ 
    ColegioDBEntities db = new ColegioDBEntities(); 

    public IQueryable<Student> FindAllStudents() 
    { 
     return db.Students; 
    } 

    public Student FindStudent(int id) 
    { 
     return db.Students.SingleOrDefault(c => c.StudentId == id); 
    } 

    public void Add(Student Student) 
    { 
     db.AddToStudents(Student); 
    } 

    public void Save() 
    { 
     db.SaveChanges(); 
    } 

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

Y aquí es un ejemplo de cómo lo uso.

private void btnLogin_Click(object sender, EventArgs e) 
    { 
     UserRepository repo = new UserRepository(); 
     var result = repo.FindAllUsers().Where(u => u.Username == txtUsername.Text && u.Password == txtPassword.Text); 
     if (result.Count() > 0) 
     { 
      MainForm form = new MainForm(txtUsername.Text); 
      form.Show(); 
      this.Hide(); 
     } 
     else 
     { 
      MessageBox.Show("Usuario y/o contraseña incorrecta.", 
      "Acceso Denegado", 
      MessageBoxButtons.OK, 
      MessageBoxIcon.Stop, 
      MessageBoxDefaultButton.Button1); 
      txtUsername.Focus(); 
      txtPassword.Focus(); 
     } 
    } 

Alguien sugirió que yo uso IDisposable para "limpiar" correctamente la conexión, pero no sé cómo implementar esto.

¿Alguna sugerencia? Gracias por tu tiempo.

Answer 1

No estoy seguro si realmente tiene el punto, pero pareces a aplicar IDisposable, pero hay que llamar Dispose o utilizar using:

using(UserRepository repo = new UserRepository()) 
    { 
    // ... 
    } 

Para esto se necesitan botar al salir del bloque usando y se limpia el UserRepository .

hay algo más de información:

• using Statement (C# Reference)
• Implementing a Dispose Method

Answer 2

Es StudentRepository o UserRepository? Y si uno deriva del otro, entonces tiene un problema.

Sin herencia, su implementación de StudentRepository es aceptable. Para ser totalmente correcta debe asegurarse declarándolo sellada:

public sealed class StudentRepository : IDisposable 
{ 
    .... 
    public void Dispose() 
    { 
     db.Dispose(); 
    } 
} 

Y, como ya se ha señalado @Stefan, usted tiene que hacer uso de él cada vez que se ejemplariza un StudentRepository, con using() { }.