Come posso migliorare questo codice di classificazione?

Sto controllando il parametro ordinamento e la costruzione di una serie di if dichiarazioni:Come posso migliorare questo codice di classificazione?

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

Come faccio questo meglio?

fonte

2012-05-11 Rod

Definire "migliore". Meglio per cosa? – Oded

In che modo vuoi "migliorarlo"? Non funziona come previsto? È troppo lento? Non ti piace come è strutturato il codice? Abbiamo bisogno di maggiori informazioni qui. –

Vorrei raccomandare l'uso della composizione LINQ. Vedi http://stackoverflow.com/questions/5881107/how-can-i-build-entity-framework-queries-dynamically/5882243#5882243 – Euphoric

separata vostra ordinazione e il resto della query - le parti che sono gli stessi per ogni query non devi duplicare nel codebase (tenerlo 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();

fonte

2012-05-11 19:50:04 BrokenGlass

Questo è ancora un gran numero di piatti per tutti i campi, la riflessione non sarebbe molto più pulita? Immagina se avesse 50 campi! :) – mattytommo

@NiklasB. Questo sarebbe un problema, perché OrderBy è generico e il lambda ha un tipo diverso per ogni proprietà. A meno che tu non voglia convertirlo in oggetto ovunque. – Euphoric

@mattytommo: Questo è certamente possibile e ordinato se si tratta di una raccolta in memoria – BrokenGlass

Usa riflessione :)

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

fonte

2012-05-11 19:50:35 mattytommo

Sembra che tu voglia semplicemente ordinare il nome della proprietà come una stringa. Nel qual caso, questo è in realtà già risolto utilizzando "Dynamic LINQ":

Dynamic LINQ OrderBy on IEnumerable<T>

Date un'occhiata a risposta di questa domanda e si dovrebbe fornire con il codice di esempio per risolvere il problema.

fonte

2012-05-11 19:52:34 Tejs

Come posso migliorare questo codice di classificazione?

risposta

Problemi correlati