Linq Abfrage ToList() ist leer, aber foreach funktioniert

Question

Ich habe folgende Linq query:

IEnumerable<Network> net = from file in Directory.GetFiles(folder+ @"\network") 
          from lines in File.ReadLines(file).Skip(1) 
          let row = lines.Split(',')   
          select new Network 
          { 
           networkname = getnetwork(row), 
           ... 
           networkdate = getnetworkdate(row) 
          }; 

wenn ich ToList() darauf verwenden, die Liste ist leer, aber wenn ich eine foreach-Schleife, wo ich eine leere Liste und fügen Sie jedes Element hinzu, ist es nicht. Habe ich bei dieser Abfrage einen Fehler gemacht oder könnte es eine andere Quelle für dieses seltsame Verhalten geben?

UPDATE: ich es in einer Verlängerung Methode wie folgt bin mit: Dies funktioniert nicht:

public static void FillFromCsv(this List<Network> network) 
    { 
     [QUERY HERE] 

     network = net.toList(); 
    } 

Dies funktioniert:

public static void FillFromCsv(this List<Network> network) 
    { 
     [QUERY HERE] 

     network.Clear(); 
     Foreach (Network n in net) 
     { 
      network.add(n); 
     } 
    } 

Answer 1

Ihr Problem nichts mit foreach zu tun hat vs ToList. Das Problem ist, dass in dem ersten Verfahren Sie die Liste nicht ändern, die in übergeben werden Sie überschreiben die lokale Referenz mit einer neuen Referenz:.

public static void FillFromCsv(this List<Network> network) 
{ 
    [QUERY HERE] 

    // this only affects the _local_ "network" reference, not the reference passed in 
    network = net.toList(); 
} 

Sie könntest die Parameter ändern ein ref Parameter, aber warum überhaupt eine Erweiterungsmethode verwenden? Warum hat der Anrufer eine Liste übergeben, wenn Sie ihn einfach wegblasen wollen? Ich würde

public static IEnumerable<Network> FillFromCsv() 
{ 
    [QUERY HERE] 

    return net.ToList(); 
} 

Bei der zweiten Methode nur tun, werden Sie die Liste Instanz Clearing, die in und durch Hinzufügen der Ergebnisse an sie übergeben wird. Was ist in Ordnung, aber es ist verschiedene, was Sie in der ersten Methode tun.

Sie konnte die foreach vermeiden, indem sie gerade tun:

public static void FillFromCsv(this List<Network> network) 
{ 
    [QUERY HERE] 

    network.Clear(); 
    network.AddRange(net); 

} 

Aber es scheint seltsam den Anrufer haben Sie eine Liste übergeben und Sie klar, und es zu füllen. Eine bessere Methode wäre, nur eine eine Liste (wie der erste Vorschlag).