Po prostu zwróć IQueryable<T>.

Zanim napiszesz kolejny fragment "kodu repozytorium", znacząco skorzystasz z przeczytania artykułu Ayende Architecting in The Pit of Doom - The Evils of the repozytorium Abstraction Layer

Twoje podejście jest bez wątpienia dodawaniem znacznej niepotrzebnej złożoności.

Cały kod z drugiego pytania na ogólnej listy OrderBy Lambda nie robi nic innego niż maskowanie istniejącego skutecznego API z niepotrzebną i nieznaną abstrakcją.

Jeśli chodzi o Wasze obawy,

1. Dostawcy LINQ zachowują się inaczej, ale dopóki przekazywane predykaty mogą być przetwarzane przez dostawcę LINQ, nie ma to znaczenia. W przeciwnym razie nadal napotkasz ten sam problem, ponieważ przekazujesz Expression, Który i tak przechodzi do IQueryable. Jeśli implementacja IQueryProvider nie może obsłużyć twojego predykatu, to nie może zajmij się swoim orzeczeniem. (Zawsze możesz wywołać ToList(), jeśli musisz ocenić przed dalszym filtrowaniem, którego nie można przetłumaczyć).

2. Modyfikacja zapytania może powodować problemy z wydajnością, ale jest bardziej prawdopodobne, że ujawni bardzo potrzebne funkcje. Co więcej, problemy z wydajnością poniesione przez nieoptymalne zapytanie LINQ mogą być znacznie mniej szkodliwe niż problemy z wydajnością poniesione przez pobranie o wiele większej liczby rekordów niż potrzeba, aby uniknąć ujawnienia IQueryable lub poprzez systematyczne filtrowanie logiki dostępu do danych przez nadęte poziomy abstrakcji, które tak naprawdę nic nie robią (pierwsze zagrożenie jest bardziej znaczące). Ogólnie rzecz biorąc, nie będzie to problemem, ponieważ większość wiodących dostawców LINQ zoptymalizuje logikę zapytań w procesie tłumaczenia.

Jeśli chcesz ukryć logikę zapytań z przodu, nie próbuj tworzyć ogólnego repozytorium. Enkapsulate zapytań z rzeczywistych metod specyficznych dla biznesu. Teraz Ja może się mylę, ale zakładam, że użycie wzorca repozytorium jest inspirowane przez Domain Driven Design. Jeśli tak jest, to powodem korzystania z repozytorium jest umożliwienie utworzenia domeny z głównym naciskiem na model domeny. Jednak używanie tego rodzaju repozytorium generycznego nie robi nic więcej niż zmiana semantyki z Create Read Update Delete na Find Add Remove Save. Nie ma tam żadnej prawdziwej wiedzy biznesowej.

Rozważ sensowność (i usability) z

interface IPersonRepository 
{ 
     Person GetById(int id);
     IEnumerable<Person> FindByName(string firstName, string lastName);
}

W przeciwieństwie do

interface IRepository<T> { 
     IEnumerable<T> FindBy(Query<T> query);
}

Ponadto, czy rzeczywiście można wskazać na korzyść korzystania z IRepository<T> w ogóle (w przeciwieństwie do IQueryable<T>)?

Należy również wziąć pod uwagę, że przy podejściu ogólnym, w rzeczywistości nie enkapsulujesz logiki zapytań w ogóle. W końcu budujesz go zewnętrznie, co doprowadzi do dodatkowych niepotrzebnych kodów.

* Jeszcze jedna uwaga na temat zasobów, które odradzają używanie IQueryable<T>, jest taka, że warto przyjrzeć się Data ich publikacji. Był czas, kiedy dostępność dostawców LINQ była dość ograniczona (do wczesnych EF i LINQ-to-SQL). W tym czasie ujawnienie IQueryable<T> wiązałoby się z niekompatybilnością z niektórymi popularniejszymi zamiennikami Microsoft ORM (LINQ-to-NHibernate już dawno zostało zaimplementowane). W tym momencie Wsparcie LINQ jest praktycznie wszechobecne w poważnych bibliotekach ORM. NET