C# Nodo XML de Foreach

Question

Estoy tratando de agregar todos los nodos en un archivo XML a ListView, y estoy haciendo algo mal, pero no puedo resolverlo incluso después de mirar una carga de ejemplos. Este es el fragmento de XML:

<queue> 
<slots> 
<slot> 
<status>Downloading</status> 
<filename>file1</filename> 
<size>1 GB</size> 
</slot> 
<slot> 
<status>Downloading</status> 
<filename>file2</filename> 
<size>2 GB</size> 
</slot> 
</slots> 
</queue> 

Y aquí está el código:

 XDocument xDoc = XDocument.Load(xmlFilePath); 

     List<Download> list = new List<Download>(); 

     foreach (var download in xDoc.Descendants("slots")) 
     { 
      string filename = download.Element("filename").Value; 
      string size = download.Element("size").Value; 
      string status = download.Element("status").Value; 
      list.Add(new Download { Filename = filename, Size = size, Status = status });    
     } 

Cualquier ayuda muy apreciada como siempre.

EDIT: Para aclarar, yo estoy poniendo un NullReferenceException en

string filename = download.Element("filename").Value; 

y sé la vista de lista no está, yo no he hecho eso todavía poco :)

Answer 1

var list = (from download in xDoc.Descendats("slot") 
      select new Download 
        { 
         Filename = (string) download.Element("filename"), 
         Size = (string) download.Element("size"), 
         Status = (string) download.Element("status") 
        }).ToList(); 

esto parece más agradable, y ya que no dijiste qué es exactamente lo que está mal con tu código, es todo lo que puedo hacer.

Actualización: acaba de probar esto y soluciona su excepción.

Answer 2

El XML en su ejemplo funciona bien. NullReferenceException está sucediendo porque el XML real que está utilizando no tiene un elemento de nombre de archivo en uno de los espacios. Puede utilizar

string filename = download.Element("filename") == null ? 
    String.Empty : download.Element("filename").Value; 

lugar si existe un posible defecto para el filename. Pero lo más probable es que el manejo correcto de esta excepción sea mejor.

Answer 3

void LoadSlots() 
{ 
    XmlDocument doc = new XmlDocument(); 
    doc.Load(Environment.CurrentDirectory + "\\queue.xml"); 

    XmlNodeList nodes = doc.SelectNodes("//queue/slots/slot"); 

    foreach (XmlNode node in nodes) 
    { 
    string filename = node.Attributes["filename"].InnerText; 
    string size = node.Attributes["size"].InnerText; 
    string status = node.Attributes["status"].InnerText; 
    _slots.Add(filename, size, status); 
    } 
}