我有一个像这样的简单类,它实现了IDisposable
,所以它被CA1816: Call GC.SuppressFinalize correctly命中:
public class A : IDisposable
{
// ...
int foo
public A()
{
SomeObj.SomeEvent += DoSomething;
}
public void Dispose()
{
SomeObj.SomeEvent -= DoSomething;
}
}
字符串
更改A.Dispose()以调用GC.SuppressFinalize(object)。这将防止引入终结器的派生类型需要重新实现“IDisposable”来调用它。
现在我已经读了this,this和this,但我仍然不明白到底是什么(下面有更多的解释).所以我的问题是:
- 如果我不理解
GC.SuppressFinalize(object)
是什么,我应该总是把GC.SuppressFinalize(object)
添加到我的Dispose
方法中吗? - 什么时候我应该不这样做,只是抑制警告?
要添加更多上下文,请执行以下操作:我仍然不明白SuppressFinalize
在做什么。我知道我已经在Dispose
中处理了一些资源(示例中的事件处理程序,或IDisposable
资源等),但是像int foo
这样的变量仍然需要清理吗?还有警告部分“这将防止引入终结器的派生类型需要重新实现”呢?
1条答案
按热度按时间lzfw57am1#
Finalizer或析构函数是C++的遗留问题,但在C#中,它延长了GC收集对象的时间。
不必要的终结器(包括空终结器、仅调用基类终结器的终结器或仅调用有条件发出的方法的终结器)会导致不必要的性能损失。
当你使用dispose模式时,你手动释放(或者通过using语句)资源,终结器就不再需要了。方法
GC.SuppressFinalize
是告诉GC忽略目标的终结器,像普通对象一样收集它,从而提高GC的性能。如果你不调用这个方法,你就失去了使用dispose模式的意义,所以总是添加它是一个很好的做法。我感到困惑的部分原因是,我的课上没有这样的东西,但仍然收到警告。
是的,你的类没有终结器,但是你不能保证它的子类也没有。注意,这个分析不会在密封类中报告(根据我的测试,内部/私有类也不会)。
什么时候我不应该这样做,只是抑制警告?
应该是never。如上所述,即使你不调用
GC.SuppressFinalize
,通常也只会影响性能。除非你分别在dispose方法和终结器中释放资源,但感觉这是某种编程错误。