需要帮助清理我的XML解析器代码

问题描述:

所以,我想让自己熟悉XML。为此,我创建了一个简单的XML文件,然后创建了一个C#脚本来读取该XML文件。现在一切都按预期工作,但代码感觉很糟糕。需要帮助清理我的XML解析器代码

我的XML看起来是这样的:

<buildings> 
    <building> 
     <name>House</name> 
     <buildTime>5</buildTime> 
     <treeCost>100</treeCost> 
     <stoneCost>10</stoneCost> 
     <goldCost>10</goldCost> 
    </building> 

    <building> 
     <name>Tower</name> 
     <buildTime>5</buildTime> 
     <treeCost>10</treeCost> 
     <stoneCost>300</stoneCost> 
     <goldCost>50</goldCost> 
    </building> 
</buildings> 

而且我的C#代码如下所示:

XDocument xmlDoc = XDocument.Load ("BuildingGame_Data/Buildings.xml"); 

    var names = xmlDoc.Descendants("building").Descendants("name"); 
    var buildTimes = xmlDoc.Descendants("building").Descendants("buildTime"); 
    var treeCosts = xmlDoc.Descendants("building").Descendants("treeCost"); 
    var stoneCosts = xmlDoc.Descendants("building").Descendants("stoneCost"); 
    var goldCosts = xmlDoc.Descendants("building").Descendants("goldCost"); 

    List<string> namesList = new List<string>(); 
    List<int> buildTimesList = new List<int>(); 
    List<int> treeCostsList = new List<int>(); 
    List<int> stoneCostsList = new List<int>(); 
    List<int> goldCostsList = new List<int>(); 

    foreach (string n in names) 
    { 
     namesList.Add(n); 
    } 

    foreach (string b in buildTimes) 
    { 
     buildTimesList.Add(System.Convert.ToInt32(b)); 
    } 

    foreach (string t in treeCosts) 
    { 
     treeCostsList.Add(System.Convert.ToInt32(t)); 
    } 

    foreach (string s in stoneCosts) 
    { 
     stoneCostsList.Add(System.Convert.ToInt32(s)); 
    } 

    foreach (string g in goldCosts) 
    { 
     goldCostsList.Add(System.Convert.ToInt32(g)); 
    } 


    List<Building> buildings = new List<Building>(); 
    for (int i = 0; i < namesList.Count; i++) 
    { 
     buildings.Add(new Building(namesList[i], buildTimesList[i], treeCostsList[i], stoneCostsList[i], goldCostsList[i])); 
    } 

现在,我想办法跳过,我将我所有的XML中间部分数据并将其存储在不同的列表中。这似乎是一个不必要的步骤,但我无法以任何其他方式使其工作。

一旦你有你的后代只是做一个选择映射到您的类的XML。您可以使用Element来获取“建筑物”的子元素并将其转换为正确的值。

var buildings = xmlDoc.Descendants("building") 
         .Select(be => new Building (
          (string)be.Element("name"), 
          (int)be.Element("buildTime"), 
          (int)be.Element("treeCost"), 
          (int)be.Element("stoneCost"), 
          (int)be.Element("goldCost"))) 
         .ToList(); 
+0

当我实现这个我得到一个错误说“类型'System.Collections.Generic.IEnumerable ‘不包含'选择的定义’,并没有扩展方法选择' 'System.Collections.Generic.IEnumerable '类型可以找到(你是否缺少使用指令或程序集引用?)“ – 2014-10-06 12:44:27

+1

@JesperEvertsson您需要添加'使用System.Linq ;' – juharr 2014-10-06 12:47:37

+0

谢谢,现在它按预期工作。现在唯一的问题是我不太了解代码。你有什么建议,我可以阅读这个主题,并了解发生了什么? – 2014-10-06 12:50:17