CA1821空のファイナライザを削除

Question

なぜこのファイナライザについてVSが苦情を言いますか？

VS 2017 - 15.3.5
マイクロソフトコード分析2017 - 2.3.0.62003

using System; 

namespace ConsoleApp 
{ 
    class DisposableClass : IDisposable 
    { 
#if DEBUG 
     ~DisposableClass() // CA1821 Remove empty Finalizers 
     { 
      System.Diagnostics.Debug.Fail ("Forgot Dispose?"); 
     } 
#endif 

     public void Dispose() 
     { 
#if DEBUG 
      GC.SuppressFinalize (this); 
#endif 
     } 
    } 

    class Program 
    { 
     static void Main (string[] args) 
     { 
      Console.WriteLine ("Hello World!"); 
     } 
    } 
} 

Answer 1

これはa bug in the Analyzerになりそうです。

6月23日号のコメントから：

@nguerreraのおかげで、あなたはアナライザがリリースビルドで有効な問題が報告されていることが正しいです。しかし、アナライザには依然として問題があります。包囲方法も条件的に除外されていれば発射すべきではありません。たとえば、リリースとデバッグの両方のビルドでは、次のように診断が実行されます。

#if DEBUG 
    ~InvisibleEditor() 
    { 
     Debug.Assert(Environment.HasShutdownStarted, GetType().Name + " was leaked without Dispose being called."); 
    } 
#endif