L'esecuzione differita di For-Loop e LINQ non funziona bene insieme

Question

Il titolo suggerisce che ho già un'idea di cosa sta succedendo, ma non riesco a spiegarlo. Ho cercato di ordinare uno List<string[]> dinamicamente da ogni "colonna", iniziando dal primo e finendo con il minimo Length di tutti gli array.

Quindi, in questo campione è 2, perché l'ultima string[] ha solo due elementi:

List<string[]> someValues = new List<string[]>(); 
someValues.Add(new[] { "c", "3", "b" }); 
someValues.Add(new[] { "a", "1", "d" }); 
someValues.Add(new[] { "d", "4", "a" }); 
someValues.Add(new[] { "b", "2" }); 

Ora ho cercato di ordinare tutto dalla prima e seconda colonna. Potrei farlo in modo statico in questo modo:

someValues = someValues 
    .OrderBy(t => t[0]) 
    .ThenBy(t => t[1]) 
    .ToList(); 

Ma se non so il numero di "colonne" potrei usare questo ciclo (che è quello che ho pensato):

int minDim = someValues.Min(t => t.GetLength(0)); // 2 
IOrderedEnumerable<string[]> orderedValues = someValues.OrderBy(t => t[0]); 
for (int i = 1; i < minDim; i++) 
{ 
    orderedValues = orderedValues.ThenBy(t => t[i]); 
} 
someValues = orderedValues.ToList(); // IndexOutOfRangeException 

Ma questo non funziona, fallisce con uno IndexOutOfRangeException nell'ultima riga. Il debugger mi dice che i è 2 in quel momento, quindi la condizione del ciclo sembra essere ignorata, i è già == minDim.

Perché è così? Qual è il modo corretto per questo?

Answer 1

E 'lo stesso problema come un sacco di persone avevano con foreach loop pre C# 5.

orderedValues = orderedValues.ThenBy(t => t[i]); 

Il valore di i non sarà valutato fino a quando non si chiama .ToList() a quel punto è 2 poiché questa è la condizione di uscita del ciclo for.

è possibile introdurre una nuova variabile locale all'interno del ciclo for per risolvere il problema:

for (int i = 1; i < minDim; i++) 
{ 
    var tmp = i; 
    orderedValues = orderedValues.ThenBy(t => t[tmp]); 
} 

---

Per ulteriori informazioni si potrebbe dare un'occhiata al post sul blog di Eric Lippert su Closing over the loop variable considered harmful.

Answer 2

Questo è probabilmente accade perché il valore della i non è chiuso all'interno del ciclo - quando l'uscita dal ciclo, i avranno un valore di 2 e poit[i] saranno valutati a causa di esecuzione differita.

Una soluzione è quella di creare una variabile chiusura all'interno del ciclo:

int minDim = someValues.Min(t => t.GetLength(0)); // 2 
IOrderedEnumerable<string[]> orderedValues = someValues.OrderBy(t => t[0]); 
for (int i = 1; i < minDim; i++) 
{ 
    var x = i; 
    orderedValues = orderedValues.ThenBy(t => t[x]); 
} 
someValues = orderedValues.ToList();