我有一个简单的遗留代码块,它位于一个循环内,循环遍历一些可能不好的xml节点,并且需要重构,因为它没有按预期工作: Try xmlFrag.LoadXml("temproot" strXMLfragment "/temproot") writer.WriteRaw(strXMLf
Try xmlFrag.LoadXml("<temproot>" & strXMLfragment & "</temproot>") writer.WriteRaw(strXMLfragment) Catch ex As Exception InvalidXML = True End Try
这个块的目的是检查有效的xml,然后写出xml.它实际上做的是检查无效的xml,然后只有在它有效时才写出xml.因此需要将其修复为按预期工作.
我第一次尝试修复:
Try xmlFrag.LoadXml("<temproot>" & strXMLfragment & "</temproot>") 'writer.WriteRaw(strXMLfragment) Catch ex As Exception InvalidXML = True Finally writer.WriteRaw(strXMLfragment) End Try
这适用于我的测试数据,但我担心WriteRaw可能会对其他数据抛出异常.我还没有找到关于什么会导致WriteRaw抛出异常的结论性陈述,以及当Finally block中的代码抛出异常时会发生什么.
所以我试着像这样重写它:
Try xmlFrag.LoadXml("<temproot>" & strXMLfragment & "</temproot>") Catch ex As Exception InvalidXML = True End Try Try writer.WriteRaw(strXMLfragment) Catch End Try
坦率地说它看起来很难看.有没有更优雅的方式重构这个或第一次尝试是否合适?
当在Finally块中引发一个excpetion时,没有什么特别的事情发生:异常会像任何其他异常一样向外传播,并且不会执行此finally块中的异常之后的代码.如果strXMLfragment为null或空字符串(或由于already running asynchronous operation),则您的第一次尝试将失败.
因此,如果您真的想要处理/吞下所有异常,则必须使用两个Try块.