Jak mogę poprawić ten kod sortowania?

Question

mam sprawdzanie parametr sortowania i budowania kilka if oświadczenia:

if (sortDirection == "ASC") 
{ 
    if (sortBy == "Id") 
     return customerList.OrderBy(x => x.Id).Skip(startIndex).Take(pageSize).ToList(); 
    if (sortBy == "FirstName") 
     return customerList.OrderBy(x => x.FirstName).Skip(startIndex).Take(pageSize).ToList(); 
    if (sortBy == "City") 
     return customerList.OrderBy(x => x.City).Skip(startIndex).Take(pageSize).ToList(); 
} 
else 
{ 
    if (sortBy == "Id") 
     return customerList.OrderByDescending(x => x.Id).Skip(startIndex).Take(pageSize).ToList(); 
    if (sortBy == "FirstName") 
     return customerList.OrderByDescending(x => x.FirstName).Skip(startIndex).Take(pageSize).ToList(); 
    if (sortBy == "City") 
     return customerList.OrderByDescending(x => x.City).Skip(startIndex).Take(pageSize).ToList(); 
} 

Jak zrobić to lepiej?

Answer 1

oddzielić zamawiania i reszta zapytania - części, które są takie same dla każdego zapytania nie trzeba powielać w swoim kodzie (zachować to DRY):

var query = customerList; 

if (sortDirection == "ASC") 
{ 
    if (sortBy == "Id") 
     query = query.OrderBy(x => x.Id); 
    ///and so on 
} 

query = query.Skip(startIndex).Take(pageSize).ToList(); 

Answer 2

użyć refleksji :)

customerList = (sortDirection == "ASC") 
    ? customerList 
     .OrderBy(x => x.GetType().GetProperty(sortBy).GetValue(x, null)) 
     .Skip(startIndex) 
     .Take(pageSize) 
     .ToList() 
    : customerList 
     .OrderByDescending(x => x.GetType().GetProperty(sortBy).GetValue(x, null)) 
     .Skip(startIndex) 
     .Take(pageSize) 
     .ToList(); 

Answer 3

Wygląda na to, że chcesz po prostu zamówić przez nazwę właściwości jako ciąg znaków. W tym przypadku jest to rzeczywiście już rozwiązany za pomocą „Dynamic LINQ”:

Dynamic LINQ OrderBy on IEnumerable<T>

Spójrz na to pytanie odpowiedzieć i powinien dostarczyć próbki kodu, aby rozwiązać problem.