我有一个简单的函数,看起来像这样:
public static string Merge(string xml1, string xml2)
{
try
{
var doc1 = XDocument.Load(new StringReader(xml1));
var doc2 = XDocument.Load(new StringReader(xml2));
// these "die" by throwing InvalidOperationException
var root1 = GetElementOrDie(doc1, Names.RootElementName);
var root2 = GetElementOrDie(doc2, Names.RootElementName);
foreach (var element in root2.Elements())
root1.Add(element);
return doc1.ToString();
}
catch (Exception e) when (!(e is InvalidOperationException))
{
throw new InvalidOperationException(e.Message, e);
}
}
在Visual Studio 2015下,这会生成代码分析警告CA2202:
对象'root2.Elements()。GetEnumerator()'的处置可以超过
在方法'XmlProcessor.Merge(string,string)'中一次。避免
生成您不应该调用的System.ObjectDisposedException
在一个物体上放置多次。
如果删除
when
子句,警告将消失。这里发生了什么?警告我是对的吗?
以供参考:
private static XElement GetElementOrDie(XContainer container, XName elementName)
{
var element = container.Element(elementName);
if (element == null)
throw new InvalidOperationException();
return element;
}
最佳答案
我认为这是代码分析的问题,我会予以抑制。如果我将循环转换为:
foreach (var element in root2.Elements())
root1.Add(element);
为此公开显式抱怨的枚举器:
var enumerator = root2.Elements().GetEnumerator();
while (enumerator.MoveNext())
{
XElement item = enumerator.Current;
root1.Add(item);
}
这样,即使使用
CA2202
文件管理器,也不会再抛出when
。关于c# - 异常过滤器触发CA2202?,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/35203767/