如何正确处理此代码？

Question

我有一个相当大的项目，我尽量保持干净整洁。当我在 Visual Studio 中运行代码分析器时，我遇到了一个我觉得很烦人的可靠性错误。我真的很想学习如何解决它。这是我正在做的一个简化示例。

这是警告。

警告 1 CA2000：Microsoft.Reliability：在方法“MyExampleClassForStackOverflow.AddFeed(string)”中，在对对象“new FeedClassExamle()”的所有引用超出范围之前调用 System.IDisposable.Dispose。

这是我的示例代码：

class MyExampleClassForStackOverflow : IDisposable
{
    public ConcurrentDictionary<string, FeedClassExamle> Feeds { get; set; }

    public void AddFeed(string id)
    {
        //The warning is coming from this code block.
        //In the full code, the feed classes collects data on a specific 
        //interval and feeds them back using events.
        //I have a bunch of them and they need to be accessible so I 
        //store them in dictionaries using keys to effeciently find them.
        Feeds.TryAdd(id, new FeedClassExamle());
        Feeds[id].Start();
    }
    public void Dispose()
    {
        foreach (var item in Feeds)
            item.Value.Dispose();
    }
}

class FeedClassExamle : IDisposable
{
    public void Start()
    {

    }
    public void Dispose()
    {

    }
}

为了测试代码，使用：

using (var example = new MyExampleClassForStackOverflow())
{

}

欢迎提出任何建议。

Answer 1

存在警告是因为代码分析工具无法确定对象是否会被正确处置。根据您的代码编写方式，对象实际上不会被正确处理，但修复代码可能不会消除警告。

从根本上说，需要为每个AddFeed 方法确保在它创建的每个FeedClassExample 实例上调用Dispose。最好的方法是避免创建 FeedClassExample 实例，如果一个实例已经存在于当前 ID 下的字典中。如果做不到这一点，AddFeed 方法应该处理它创建的任何FeedClassExample，然后决定不存储在字典中，或者与字典中的那个交换（我不确定ConcurrentDictionary支持这样做）然后Dispose旧的。基本要求是在AddFeed 的实际执行之外的任何时候，字典都将保存所有已创建但未销毁的FeedClassExample 实例。

在你的FeedClassExample 类中添加一个析构函数可能会提供有用的信息，除了记录一条消息之外什么都不做。如果您在该类上正确调用Dispose，则析构函数将永远不会执行。如果你没有打电话给Dispose，它会的。因此，如果析构函数曾经执行过，您就可以知道自己做错了什么。

Answer 2

如果 TryAdd 失败，对象不会被 Disposed，因此请尝试显式执行此操作：

public void AddFeed(string id)
{
    FeedClassExample fce = new FeedClassExamle();
    if (!Feeds.TryAdd(id, fce))
    {
        fce.Dispose();
    } 
    Feeds[id].Start();
}

Answer 3

仅在需要添加时才创建实例：

if (!Feeds.ContainsKey(id)) {
  Feeds.GetOrAdd(id, new FeedClassExamle());
}