OrderBy/ThenBy looping - Liste annidate in C#

Question

ho una lista annidata,

List<List<String>> intable; 

dove vorrei ordinare tutte le colonne. Il problema è che il numero di colonne dipende dall'input dell'utente.

Ordinamento della lista come questo funziona bene (assumendo 4 colonne per questo esempio)

var tmp = intable.OrderBy(x => x[0]); 
tmp = tmp.ThenBy(x => x[1]); 
tmp = tmp.ThenBy(x => x[2]); 
tmp = tmp.ThenBy(x => x[3]); 
intable = tmp.ToList(); 

Ma, quando ho messo in un ciclo, come questo:

var tmp = intable.OrderBy(x => x[0]); 
for (int i = 1; i <= 3; i++) 
{ 
     tmp = tmp.ThenBy(x => x[i]); 
} 
intable = tmp.ToList(); 

esso non funziona più correttamente e ordina solo la quarta colonna.

Answer 1

Questo è un caso di accesso a una chiusura modificata. Modificare il codice per questo e funzionerà:

var tmp = intable.OrderBy(x => x[0]); 
for (int i = 1; i <= 3; i++) { 
    var thisI = i; 
    tmp = tmp.ThenBy(x => x[thisI]); 
} 
intable = tmp.ToList(); 

Eric Lippert ha scritto un two-part article che descrive il problema. Il motivo per cui non funziona come previsto è, in breve, perché LINQ utilizza solo l'ultimo valore di i quando viene valutato quando si chiama ToList(). E 'lo stesso come se si fosse scritto:

var tmp = intable.OrderBy(x => x[0]); 
tmp = tmp.ThenBy(x => x[3]); 
tmp = tmp.ThenBy(x => x[3]); 
tmp = tmp.ThenBy(x => x[3]); 
intable = tmp.ToList(); 

Answer 2

Creare un operatore di confronto

class StringListComparer : IComparer<List<string>> 
{ 
    public int Compare(List<string> x, List<string> y) 
    { 
     int minLength = Math.Min(x.Count, y.Count); 
     for (int i = 0; i < minLength; i++) { 
      int comp = x[i].CompareTo(y[i]); 
      if (comp != 0) { 
       return comp; 
      } 
     } 
     return x.Count.CompareTo(y.Count); 
    } 
} 

quindi ordinare l'elenco come questo

intable.Sort(new StringListComparer());