Erweiterungsmethode auf Klasse - gute oder schlechte Idee?

Question

Ich habe bereits Entity Framework zusammen mit einem Repository und einigen statischen Klassen/Methoden zur Manipulation der Daten. Hier ist ein typisches Beispiel:

public static IEnumerable<Supplier> Contains(IEnumerable<int> idList) 
{ 
    return SupplierView.Select().Where(x => idList.Contains(x.ID)); 
} 

Diese Methoden abfragen meiner EF-Repository und manchmal brauche ich eine Reihe von Variablen zu übergeben Sie die Daten, die ich wieder brauchen.

Da mein Lieferant Unternehmen bereits vorhanden ist, ich bin Betrachtung meiner Fragen Erweiterungsmethoden mit Hilfe der Klasse machen, so etwas wie diese:

public static IEnumerable<Supplier> GetSimilar(this Supplier s) 
    { 
     return SupplierView.Select().Where(/* the criteria matches */)); 
    } 

Es wäre nur für die Abfrage von Daten verwendet werden - aber wie ich bin Ich bin nicht sicher, ob dies eine großartige Design-Idee ist - aber es ist sicherlich bequemer, Params zu übergeben/zu validieren etc.

Ich habe bereits eine Teilklasse für meine Haupt-Klasse eingerichtet Entitäten, aber ich neige dazu, Eigenschaften hinzuzufügen, wenig Auswirkungen.

Irgendwelche Gedanken?

Answer 1

Ist Supplier eine Klasse für sich?

Wenn ja, dann würde ich nur die Erweiterung dieser Klasse empfehlen - anstatt auf Erweiterungsmethoden anzuheften.

Wenn es eine Klasse aus der Datenbank von EF erzeugt wird, ist es ein public partial class so können Sie leicht für Supplier in einer zweiten Datei zusätzliche Methoden schreiben, auch die public partial class mit - diese werden dann in fusionierten eine einzige .NET-Klasse bei Kompilierzeit

Ich mag Erweiterungsmethoden und sie machen eine Menge Sinn - wenn Sie Methoden zu Klassen hinzufügen müssen, die Sie nicht steuern - z. .NET-Framework-Klassen oder Drittanbieterklassen.