सी # Foreach एक्सएमएल नोड

Question

मैं एक एक्सएमएल फ़ाइल में सभी नोड्स को एक सूची में जोड़ने की कोशिश कर रहा हूं, और मैं कुछ गलत कर रहा हूं लेकिन मैं अपने जीवन के लिए भार को देखने के बाद भी इसे समझ नहीं सकता उदाहरणों के। यहाँ

<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> 

और कोड है:: इस एक्सएमएल टुकड़ा है

 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 });    
     } 

किसी भी मदद की बहुत हमेशा की तरह की सराहना की।

संपादित करें: स्पष्ट करने के लिए, मैं

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

पर एक NullReferenceException हो रही है और मुझे पता है सूचीदृश्य याद आ रही है, मैं अभी तक कि बिट नहीं किया है :)

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(); 

यह दिखता है nicer, और चूंकि आपने यह नहीं कहा कि आपके कोड में वास्तव में क्या गलत है, यह सब कुछ मैं कर सकता हूं।

अद्यतन: बस इसका परीक्षण किया गया, और यह आपके अपवाद को हल करता है।

Answer 2

आपके उदाहरण में एक्सएमएल ठीक काम करता है। NullReferenceException हो रहा है क्योंकि आपके द्वारा उपयोग किए जा रहे वास्तविक XML में स्लॉट में से एक में फ़ाइल नाम तत्व नहीं है। आप

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

उपयोग कर सकते हैं बजाय अगर वहाँ filename के लिए एक संभावित डिफ़ॉल्ट है। लेकिन इस अपवाद को सही ढंग से संभालने की अधिक संभावना बेहतर है।

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); 
    } 
}