Tôi làm cách nào để cải thiện mã phân loại này?

Question

Tôi đang kiểm tra các thông số sắp xếp và xây dựng một loạt các báo cáo if:

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(); 
} 

Làm thế nào để làm điều này tốt hơn?

Answer 1

riêng đặt hàng của bạn và phần còn lại của các truy vấn - những phần giống nhau cho mỗi truy vấn mà bạn không cần phải lặp lại trong codebase của bạn (giữ nó 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

Sử dụng phản xạ :)

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

Có vẻ như bạn chỉ muốn đặt hàng theo tên thuộc tính dưới dạng chuỗi. Trong trường hợp đó, điều này là thực sự đã giải quyết bằng cách sử dụng "Dynamic LINQ":

Dynamic LINQ OrderBy on IEnumerable<T>

Hãy nhìn vào câu trả lời của câu hỏi này và nó sẽ cung cấp cho bạn với mẫu mã để giải quyết vấn đề của bạn.