IQueryable <T> zurückgeben oder IQueryable <T> nicht zurückgeben

Ich habe eine Repository-Klasse, die meinen LINQ in SQL Data Context umschließt. Die Repository-Klasse ist eine Geschäftsbereichsklasse, die die gesamte Datenebenenlogik (und das Caching usw.) enthält.

Hier ist meine Version 1 meiner Repo-Oberfläche.

public interface ILocationRepository
{
    IList<Location> FindAll();
    IList<Location> FindForState(State state);
    IList<Location> FindForPostCode(string postCode);
}

Um das Paging für FindAll zu handhaben, überlege ich, ob IQueryable <ILocation> anstelle von IList verfügbar gemacht werden soll, um die Benutzeroberfläche für Umstände wie Paging zu vereinfachen.

Was sind die Vor- und Nachteile, um IQueryable aus dem Daten-Repo herauszustellen?

Jede Hilfe wird sehr geschätzt.

Die Profis; Zusammensetzbarkeit:

• Anrufer können Filter hinzufügen
• Anrufer können Paging hinzufügen
• Anrufer können Sortierung hinzufügen
• etc

Die Nachteile; Nichtprüfbarkeit:

• Ihr Repository kann nicht mehr ordnungsgemäß auf Einheiten getestet werden. Sie können sich nicht darauf verlassen, dass a: es funktioniert, b: was es tut;
  • Der Aufrufer könnte eine nicht übersetzbare Funktion hinzufügen (dh keine TSQL-Zuordnung; Unterbrechungen zur Laufzeit).
  • Der Anrufer könnte einen Filter / eine Sortierung hinzufügen, mit der er sich wie ein Hund verhält
• Da Anrufer erwarten IQueryable<T>, dass sie zusammensetzbar sind, werden nicht zusammensetzbare Implementierungen ausgeschlossen - oder Sie werden gezwungen, einen eigenen Abfrageanbieter für sie zu schreiben
• Dies bedeutet, dass Sie die DAL nicht optimieren / profilieren können

Für Stabilität, habe ich genommen nicht auszusetzen IQueryable<T>oder Expression<...>auf meine Repositories. Dies bedeutet, dass ich weiß, wie sich das Repository verhält, und dass meine oberen Ebenen Mocks verwenden können, ohne sich Sorgen machen zu müssen, ob das eigentliche Repository dies unterstützt. (Erzwingen von Integrationstests).

Ich benutze immer noch IQueryable<T>etc im Repository - aber nicht über die Grenze. Ich habe hier einige weitere Gedanken zu diesem Thema veröffentlicht . Es ist genauso einfach, Paging-Parameter in die Repository-Oberfläche einzufügen. Sie können sogar Erweiterungsmethoden (auf der Schnittstelle) verwenden, um optionale Paging-Parameter hinzuzufügen , sodass die konkreten Klassen nur eine Methode implementieren müssen, dem Aufrufer jedoch möglicherweise 2 oder 3 Überladungen zur Verfügung stehen.

Wie in der vorherigen Antwort erwähnt, können Anrufer durch das Offenlegen von IQueryable mit IQueryable selbst spielen, was gefährlich ist oder werden kann.

Die erste Aufgabe der Geschäftslogik besteht darin, die Integrität Ihrer Datenbank aufrechtzuerhalten.

Sie können IList weiterhin verfügbar machen und Ihre Parameter wie folgt ändern: So machen wir ...

public interface ILocationRepository
{
    IList<Location> FindAll(int start, int size);
    IList<Location> FindForState(State state, int start, int size);
    IList<Location> FindForPostCode(string postCode, int start, int size);
}

Wenn Größe == -1, dann geben Sie alle ...

Alternativer Weg...

Wenn Sie IQueryable weiterhin zurückgeben möchten, können Sie IQueryable of List in Ihren Funktionen zurückgeben. Zum Beispiel ...

public class MyRepository
{
    IQueryable<Location> FindAll()
    {
        List<Location> myLocations = ....;
        return myLocations.AsQueryable<Location>;
        // here Query can only be applied on this
        // subset, not directly to the database
    }
}

Die erste Methode hat einen Vorteil gegenüber dem Speicher, da Sie weniger Daten als alle zurückgeben.

Ich empfehle IEnumerablestattdessen zu verwenden IList, damit haben Sie mehr Flexibilität.

Auf diese Weise können Sie von Db nur den Teil der Daten abrufen, den Sie wirklich verwenden werden, ohne dass zusätzliche Arbeit in Ihrem Repository geleistet wird.

Stichprobe:

// Repository
public interface IRepository
{
    IEnumerable<Location> GetLocations();
}

// Controller
public ActionResult Locations(int? page)
{
    return View(repository.GetLocations().AsPagination(page ?? 1, 10);
}

Welches ist super sauber und einfach.