之前的一个重要基础工程每次进入单步调试,总会在装载XML之后的解析工作时异常缓慢,往往需要7秒,而直接运行则只需要1秒,无论Debug还是Release.
后来一点点把代码注释,单步,注释再单步,终于确定了问题所在:
首先在一个重复运算的循环内部有如下语句:
var bPreDay = GetAttributeText("PreDay", "0").ToInt()>0;
此函数是一个扩展函数,实现如下:
public static string GetAttributeText(this XmlNode arg_node, string arg_strAttribute, string arg_strDefault = "")
{
var strRet = arg_strDefault;
try
{
return (arg_node.Attributes[arg_strAttribute] as XmlAttribute).InnerText;
}
catch (Exception ex)
{
}
return strRet;
}
这是一个老旧的历史代码,无疑其实现非常糟糕。竟然吞咽了异常!
此函数在初期书写时也许觉得可以不用判断,偷懒,结果不停重复调用时,由于arg_node.Attributes[arg_strAttribute]
返回null,造成此函数不停进入exception,消耗了大量时间资源。
我们现在将函数改为如下形式:
public static string GetAttributeText(this XmlNode arg_node, string arg_strAttribute, string arg_strDefault = "")
{
var attr = arg_node.Attributes[arg_strAttribute];
if (null == attr)
return arg_strDefault;
if (attr is XmlAttribute)
return (attr as XmlAttribute).InnerText;
else
throw new NotSupportedException();
}