我有一个简单的函数,看起来像这样:

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/

10-08 22:21