È corretto utilizzare i metodi di estensione nel mio caso?

Question

Io uso MVC4 e l'entità framework 5 nel mio progetto e ho molte tabelle. Come regola del nostro progetto, non cancelliamo alcun record dal database, ogni record ha un campo isActive e se quel campo è falso, allora lo consideriamo cancellato. Volevo scrivere un metodo di estensione per ottenere record attivi e dopo un po 'googling ho scritto questo:

public static IQueryable<Company> GetAll(this IQueryable<Company> source) 
{ 
    return source.Where(p => p.isActive); 
} 

ora posso usare il mio metodo di estensione per ottenere solo i record attivi come

Context db = new Context(); 
db.Company.GetAll(); 

Ma diciamo che ho Ho più di 50 tabelle nel mio database, è un buon approccio per scrivere lo stesso metodo di estensione per ciascuna delle mie tabelle. C'è un modo migliore per scrivere un solo metodo di estensione GetAll() per tutte le nostre tabelle? In realtà non sono nemmeno sicuro se sia giusto usare i metodi di estensione per questa istanza?

Qualcuno potrebbe aiutarmi e mostrarmi nel modo giusto? Apprezzo se mi aiuti con esempi di codice.

Answer 1

Dipende da come si utilizza Entity Framework, se si dipende dal generatore normale (e penso che lo si faccia), il caso diventa più difficile e non ho mai avuto un case study simile. Ma, se usi un normale generatore di classi POCO, puoi usare una classe base chiamiamola CEntity che è una classe base per ciascuna delle tue altre classi (tabelle).

Siamo già arrivati? No, per continuare con questo, io preferisco usare il Pattern Repository, e si può fare quel repository generico (CEntity), ad esempio:

public class Repository<CEntity> where CEntity : class 
{ 
    public IQueryable<CEntity> GetAll() 
    { 
     return source.Where(p => p.isActive); 
    } 

} 

E questo è come usarlo:

Repository<Company> com = new Repository<Company>(); 
Repository<Employee> emp = new Repository<Employee>(); 

var coms = com.GetAll(); // will get all ACTIVE companies 
var emps = emp.GetAll(); // will get all ACTIVE employees 

Questo è fuori di testa, se hai avuto altri problemi, mettili come commenti, lieti di aiutarti.

Answer 2

Proprio come un punto di interesse, questo è esattamente il modo a implementare i miei livelli di dati, e credo che la sua impressionante :)

Ho anche marmellata di un repository in mezzo pure ma il concetto generale dovrebbe funzionare con o senza.

Ecco alcuni esempi di codice funzionante su come utilizzo questo metodo nel mio blog per alcuni casi d'uso simili.

https://github.com/lukemcgregor/StaticVoid.Blog/blob/master/Blog/Data/Entities/Post/PostRepositoryExtensions.cs

ho scoperto che fa un po 'di codice piuttosto elegante, pur non limitando quello che si può fare troppo. Come ho detto, penso che questo metodo sia fantastico e raccomandi davvero il suo utilizzo.