Manière correcte de passer des objets Entity entre les calques?

Question

Je viens d'apprendre l'Entity Framework et j'ai fait de bons progrès pour l'intégrer à ma structure de code en couches. J'ai 2 couches visuelles, une couche de gestion et une couche d'accès aux données.

Mon problème est de passer l'objet entité entre les couches. Cet exemple de code ne fonctionne pas:

// BLL 
public static void Test1() 
{ 
List<User> users = (from u in GetActiveUsers() 
     where u.ID == 1 
     select u).ToList<User>(); 

// Do something with users 
} 

// DAL 
public static IQueryable<User> GetActiveUsers() 
{ 
using (var context = new CSEntities()) 
{ 
    return from u in context.Users 
     where u.Employee.FirstName == "Tom" 
     select u; 
} 
} 

Je reçois le message d'erreur L'instance ObjectContext a été disposé et ne peut plus être utilisé pour les opérations qui nécessitent une connexion.

Si je supprime le en utilisant de la méthode GetActiveUsers, cela fonctionne correctement.

Je sais que c'est une pratique dangereuse car le GC peut se débarrasser du contexte à tout moment et bousiller mon BLL.

Alors, quelle est la bonne façon de transmettre des informations entre les couches? Dois-je aussi faire circuler le contexte?

Answer 1

Parce que vous retournez IQueryable de votre DAL, vous ne pouvez pas utiliser le using.

La requête est reportée jusqu'à à la requête de cuisson sont - .ToList dans votre BLL.

A ce moment-là, le contexte est éliminé.

Pensez soigneusement sur l'utilisation de IQueryable, car il est risqué de pratiquer sans connaître tous les détails.

Puisque vous apprenez toujours EF, je garderais simple:

// BLL 
public static void Test1() 
{ 
List<User> users = GetActiveUsers(); 
var singleUser = users.SingleOrDefault(u => u.ID == 1); 

// Do something with users 
} 

// DAL 
public static ICollection<User> GetActiveUsers() 
{ 
using (var context = new CSEntities()) 
{ 
    var users = from u in context.Users 
     where u.Employee.FirstName == "Tom" 
     select u; 
    return users.ToList(); 
} 
} 

Si vous souhaitez obtenir un seul utilisateur, créez une autre méthode:

// DAL 
public static User GetSingleActiveUser(int userId) 
{ 
using (var context = new CSEntities()) 
{ 
    var users = from u in context.Users 
     where u.Employee.UserId == userId 
     select u; 
    return users.SingleOrDefault(); 
} 
}