无法将类型为“System.Xml.XmlComment”的对象强制转换为类型“System.Xml.XmlElement”。---C#

最近自己负责维护的一个软件出现了如下错误:

Exception: 无法将类型为“System.Xml.XmlComment”的对象强制转换为类型“System.Xml.XmlElement”。

收到产品线发来的bug通知,我很快就意识到了在什么地方出错。之前公司软件升级时,有一个xml文件需要我们plc这边负责升级(由原来的lua文件格式转为xml格式)。解析xml的代码部分如下:

 var doc = new XmlDocument();
 doc.Load(Path)         
 foreach (XmlElement childNode in doc.DocumentElement.ChildNodes)
    {
         if(childNode.Name == "I")
           {
               .........
           }  
          .........
    }

待解析的xml文件的内容格式为:



   
   
  
   

按理来说,待解析的xml文件只要符合上述的格式,都是可以解析成功的。之前也是做了非常“充分”的测试,所以组件的alpha(不能保证没有bug的)版本就发出去了,奈何这个alpha版本组件在测试部门的测试下也没有找到这个隐藏的bug。那到底是怎么样才会出现上述的bug呢。

非常简单的重现该bug就是在解析的xml文件中进行注释内容的添加:



   
   
   

这个添加注释的操作是合理的,但是在代码中解析时却是“不合理的”,因为doc去解析xml获得的ChildNodes集合中并非所有元素的类型都是XmlElement。就拿注释的内容来说,它就是XmlComment类型。于是乎,用户输入的xml文件内容包含非XmlElement类型的子节点时发生了报错。

那么怎么改呢,超级简单,只要用foreach去遍历ChildNodes子节点时,我先用object类型变量node去获取,然后转为XmlElement类型变量childNode,转换成功则继续(childNode不为null),否则跳过当前循环。

 var doc = new XmlDocument();
 doc.Load(Path)         
 foreach (object node in doc.DocumentElement.ChildNodes)
    {
         XmlElement childNode = childNode as XmlElement;
         if(childNode == null)
             continue;

         if(childNode.Name == "I")
           {
               .........
           }  
          .........
    }

总结一下:遇到遍历某个集合的逻辑时,先判断自己要处理的是什么类型,以及集合里面大概都有什么类型。上述修改的做法是最可靠的,不管你集合里面有什么类型,我只关心我要处理的类型,否则pass掉。

你可能感兴趣的:(重拾c#,工作)