CA1047「メンバーをプライベート、パブリック、または内部のメンバーにする」とC++/CLIのイベント

Question

シールされたC++/CLIクラスでパブリックイベントを宣言すると、コード解析の警告CA1047が表示されます。この警告は、自動生成された保護されたメンバー関数から来ているようです。この警告を修正するにはどうすればよいですか？

例を示します。このコード

ref class Test sealed { 
public: 
    event EventHandler^ blah; 
}; 

が発生：

警告：CA1047：Microsoft.Design：メンバー作る 'テスト::何とか::昇給（オブジェクト^、のEventArgs ^）'、プライベート、パブリック、または内部

を

Answer 1

私はこの質問をよりよく文書化します。このコード

ref class Test sealed { 
public: 
    event EventHandler^ blah; 
}; 

が発生：

警告：CA1047：Microsoft.Design：メンバー作る 'テスト::何とか::昇給（オブジェクト^、のEventArgs ^）'、プライベート、パブリック、または内部

を

はい、イベントアクセサーを自分で指定しないと、コンパイラーによって生成されます。 add、remove、raiseアクセサーを自動生成します。あなたはIldasm.exeをして見たときに後者の方が次のようになります。

.method family hidebysig specialname instance void 
     raise_blah(object value0, 
        class [mscorlib]System.EventArgs value1) cil managed 
{ 
    // etc.. 
} 

家族属性は、コード分析の警告を引き起こすものです。自動生成された追加および削除アクセサは、もちろん公開されています。それらを自分で書くことは疑わしい回避策です。カスタムアクセサを実装する本当の理由がある場合は、これを実行するだけです。ボイラープレートのバージョンは次のようになります。

using namespace System::Runtime::CompilerServices; 

ref class Test sealed { 
private: 
    EventHandler^ foo; 
public: 
    event EventHandler^ blah { 
     [MethodImpl(MethodImplOptions::Synchronized)] 
     void add(EventHandler^ d) { foo += d; } 
     [MethodImpl(MethodImplOptions::Synchronized)] 
     void remove(EventHandler^ d) { foo -= d; } 
    private: 
     void raise(Object^ sender, EventArgs^ e) { 
      EventHandler^ handler = foo; 
      if (handler != nullptr) handler(sender, e); 
     }; 
    } 
}; 

これは確かに警告を抑制します。プロペラを回転させない場合は、[SuppressMessage]属性を使用することをお勧めします。