デリゲートを使用してイベントを宣言する

Question

他の開発者に使用するクラスライブラリを開発しており、WithEvents（または他の言語でも同様）を使用して自分のクラスのインスタンスを宣言したり、クラス内で定義されたデリゲート。私はちょうどこれのようにそれをすることによってここで冗長であるか？

Public Delegate Sub TimerElapsedDelegate(ByVal sender As Object, ByVal e As System.EventArgs) 
Public Event TimerElapsed(ByVal sender As Object, ByVal e As System.EventArgs) 
Private _TimerElapsed As TimerElapsedDelegate = Nothing 

イベントを宣言して、それらにAddHandlerなどを実行させるだけでいいですか？

このアドバイスをいただきありがとうございます...私は重複していると思うし、DRYの原則を避けることは言うまでもなく、無意味なコードを望んでいないと思います。

{編集}だけコードの残りの部分は、このクラスのインスタンスが実行する「作業」が別のスレッドで行われること応力を投稿したかった。{/編集}

#Region "Delegates" 
Public Delegate Sub TimerElapsedDelegate(ByVal sender As Object, ByVal e As System.EventArgs) 
Public Event TimerElapsed(ByVal sender As Object, ByVal e As System.EventArgs) 
Private _TimerElapsed As TimerElapsedDelegate = Nothing 
Public Property OnTimerElapsed() As TimerElapsedDelegate 
    Get 
     Return _TimerElapsed 
    End Get 
    Set(ByVal value As TimerElapsedDelegate) 
     If value Is Nothing Then 
      _TimerElapsed = Nothing 
     Else 
      If _TimerElapsed Is Nothing Then 
       _TimerElapsed = value 
      Else 
       _TimerElapsed = System.Delegate.Combine(_TimerElapsed, value) 
      End If 
     End If 
    End Set 
End Property 
Private Sub TriggerTimerElapsed() 
    If OnTimerElapsed IsNot Nothing Then 
     OnTimerElapsed.Invoke(Me, New System.EventArgs) 
    End If 
    RaiseEvent TimerElapsed(Me, New System.EventArgs) 
End Sub 

Public Delegate Sub ItemReadyForQueueDelegate(ByVal sender As Object, ByVal e As System.EventArgs) 
Public Event ItemReadyForQueue(ByVal sender As Object, ByVal e As System.EventArgs) 
Private _ItemReadyForQueue As ItemReadyForQueueDelegate = Nothing 
Public Property OnItemReadyForQueue() As ItemReadyForQueueDelegate 
    Get 
     Return _ItemReadyForQueue 
    End Get 
    Set(ByVal value As ItemReadyForQueueDelegate) 
     If value Is Nothing Then 
      _ItemReadyForQueue = Nothing 
     Else 
      If _ItemReadyForQueue Is Nothing Then 
       _ItemReadyForQueue = value 
      Else 
       _ItemReadyForQueue = System.Delegate.Combine(_ItemReadyForQueue, value) 
      End If 
     End If 
    End Set 
End Property 
Private Sub TriggerItemReadyForQueue(ByVal oItem As h3Budgeteer.FileSystem.ReportTemplateFile.ReportTemplate) 
    If OnItemReadyForQueue IsNot Nothing Then 
     OnItemReadyForQueue.Invoke(Me, New ItemReadyForQueueEventArgs(oItem)) 
    End If 
    RaiseEvent ItemReadyForQueue(Me, New ItemReadyForQueueEventArgs(oItem)) 
End Sub 
Public Class ItemReadyForQueueEventArgs 
    Inherits System.EventArgs 
    Private _ReportTemplate As h3Budgeteer.FileSystem.ReportTemplateFile.ReportTemplate = Nothing 
    Public ReadOnly Property ReportTemplate() As h3Budgeteer.FileSystem.ReportTemplateFile.ReportTemplate 
     Get 
      Return _ReportTemplate 
     End Get 
    End Property 
    Public Sub New(ByVal oReportTemplate As h3Budgeteer.FileSystem.ReportTemplateFile.ReportTemplate) 
     _ReportTemplate = oReportTemplate 
    End Sub 
End Class 

終了リージョン

Answer 1

私は完全に委任を完全に削除すると言います。

あなたの代議員は、イベントとまったく同じことを行っています。あなたは、フレームワークのEvent呼び出しを使用する代わりに、あなた自身のイベント配管を書いています。イベントは、あなたが書いたものとほぼ同じですが、使用が簡単であることを除けば、イベントからの退会を容易にします。

両方を提供する利点はありません - イベントはあなたの「委任」が行うすべてのことを行い、はるかに明確です。

（以前:)

あなたはクラスライブラリとしてこれを開発している場合、私はちょうどあなたのクラスがシールされていない作り、そしてより多くの標準的なアプローチを次のようお勧めします。ロジックをオーバーライドしたり、コードに挿入してイベントを許可する通常の方法は、サブクラス化のフックを提供することです。

このような状況では、ユーザーは自分のロジックをプラグインできるようにすることができます。しかし、多くの場合、保護された仮想関数を使用すると、これがより明確になり、達成がはるかに容易になります。

イベントは正確には、ユーザーに何らかの「イベント」を通知するイベントです。これらはユーザーがデリゲートを添付するフックでなければなりません。

たとえば、デリゲートとイベントを提供する代わりに、基本的なWindows Formsコントロールは保護されたメソッド（OnMouseDownなど）と、デフォルトで起動されるイベント（MouseDown）を使用します。

これにより、ユーザーはクラスをサブクラス化し、ロジックをオーバーライドすることができます（これにより、デリゲートが必要な理由が考えられます）。

私が代議員を提供する1つの場所は、まれにあなたのクラスまたはメソッドがロジックをユーザーによって追加する必要がある場所です。この場合、抽象基本クラスを提供するか、そのロジック用に渡される代理人を持つことができます。これの良い例は、LINQの.Where（）メソッドです。フィルタリングに使用される述語なしでは無駄なので、この場合、デリゲートを渡すことは意味があります。ただし、これに関連するイベントはないことに注意してください。実際には別の機能を提供することがあります。

Answer 2

あなたのクラスライブラリに必要なのは、公開イベントコード行を書くことだけです。

Public Event TimerElapsed(ByVal sender As Object, ByVal e As System.EventArgs) 

もちろん、あなたのライブラリのどこにでもイベントを発生させてください。どのクライアント開発者もイベントにハンドラを追加することになります。

あなたのライブラリがそのクラスからのイベントを処理しない限り、その冗長性は不要です。

Answer 3

generic EventHandlerを使用すると、デリゲートを取り除くことができます。 EventArgsを継承する独自のクラスを作成するだけです。

Public Class Foo 
    Inherits EventArgs 
End Class 

Public Class Bar 
    Public Event MyEvent As EventHandler(Of Foo) 
End Class 

私はあなたが冗長であるとは思わない。 this質問への最初の回答を参照してください。空のイベントハンドラーを追加することで、イベントを聴いたり、処理したりしたくない場合には、イベントを使用している人がNullReferenceExceptionを受け取ることはありません。

-EDIT-

あなたのコードを見た後、私はReedに同意します。共有ライブラリーになる予定だから、はコンシューマーのイベントハンドラーを実行する必要があるとは思わない。あなたの図書館の仕事は、ちょうどそのイベントを断ち切って、消費者に起こったことを知らせることです。イベントを処理するかどうかは、自分の責任です。

あなたのプロパティは冗長であると言います。イベントハンドラは本質的にイベントハンドラです。