Zagnieżdżona pętla foreach odczytywania .xml i pisania obiektu do listy

Question

Mam metodę, która wykonuje się na zdarzeniu form_load, które wydaje się działać poprawnie, pomijając jedną linię.

private int ReadInPeople() 
{ 
    XmlNodeList nodeList = m_xmlDoc.DocumentElement.ChildNodes; 
    foreach (XmlNode PersonNode in nodeList) 
    { 
     Employee ccontact = new Employee(); 
     foreach (XmlNode PersonTag in PersonNode.ChildNodes) 
     { 
      switch (PersonTag.Name) 
      { 
       case "Employee": 
        ccontact.EmployeeNumber = PersonTag.FirstChild.Value; 
        break; 
       case "FirstName": 
        ccontact.FirstName = PersonTag.FirstChild.Value; 
        break; 
       case "LastName": 
        ccontact.LastName = PersonTag.FirstChild.Value; 
        break; 
       default: 
        break; 
      } 
     } 
     this.AddContact(ccontact); 
    } 
    return nodeList.Count; 
} 

Metoda AddContact dodaje obiekt pracownika do listy statycznej; jednak linia:

this.AddContact(ccontact); 

nie została wykonana.

Próbkę pliku XML:

<?xml version="1.0" encoding="utf-8"?> 
<people> 
    <person> 
    <Employee>123456789</Employee> 
    <FirstName>John</FirstName> 
    <LastName>Smith</LastName> 
    </person> 
    <person> 
    <Employee>987654321</Employee> 
    <FirstName>Ellen</FirstName> 
    <LastName>Wayne</LastName> 
    </person> 
</people> 

Próbowałem ustawienie przerwania i debugowania, i na pewno wystarczy, linia została całkowicie pomijane, jakby to było w ogóle nie było.

Zgodnie z radą Alana, zmieniłem wartość PersonTag.FirstChild.Value, gdy próbowałem odwołać się do węzła ChildNode, który nie istniał.

Zaktualizowana metoda pracy:

private int ReadInPeople() 
{ 
    XmlNodeList nodeList = m_xmlDoc.DocumentElement.ChildNodes; 
    foreach (XmlNode PersonNode in nodeList) 
    { 
     Employee ccontact = new Employee(); 
     foreach (XmlNode PersonTag in PersonNode.ChildNodes) 
     { 
      switch (PersonTag.Name) 
      { 
       case "Employee": 
        ccontact.EmployeeNumber = PersonTag.InnerText; 
        break; 
       case "FirstName": 
        ccontact.FirstName = PersonTag.InnerText; 
        break; 
       case "LastName": 
        ccontact.LastName = PersonTag.InnerText; 
        break; 
       default: 
        break; 
      } 
     } 
     this.AddContact(ccontact); 
    } 
    return nodeList.Count; 
} 

Answer 1

można łatwo przeanalizować swoją xml z LINQ do XML: rozwiązania

XDocument xdoc = XDocument.Load(path_to_xml); 
var employees = xdoc.Descendants("person") 
        .Select(p => new Employee() 
        { 
         FirstName = (string)p.Element("FirstName"), 
         LastName = (string)p.Element("LastName"), 
         EmployeeNumber = (long)p.Element("Employee") 
        }); 

foreach (var ccontact in employees) 
    this.AddContact(ccontact); 

---

XmlDocument:

XmlNodeList nodeList = m_xmlDoc.DocumentElement.SelectNodes("person"); 
foreach (XmlNode PersonNode in nodeList) 
{ 
    Employee ccontact = new Employee(); 
    ccontact.LastName = PersonNode["LastName"].InnerText; 
    ccontact.FirstName = PersonNode["FirstName"].InnerText; 
    ccontact.EmployeeNumber = PersonNode["Employee"].InnerText; 
    this.AddContact(ccontact); 
} 

Answer 2

Czy jesteś pewien, że nie tylko uderzanie wyjątek? Zawiń logikę w try/catch i zobacz, czy trafi.

linie jak może to spowodować zerowe wyjątki odniesienia i może być powodem nie jesteś uderzając tej linii kodu:

PersonTag.FirstChild.Value 

Mogę sobie wyobrazić, nazywając ją na firstChild na pracownika, imię, Nazwisko lub wróci null, ponieważ nie zawierają węzła potomnego.

Może spróbuj:

PersonTag.InnerText 

Answer 3

Build -> Rebuild Solution?

Czasami wiadomo, że pomaga.

Albo: Debug -> Exceptions... -> Thrown