C# Aksiyon/Delege Stil Soru

Question

bir olay tanımı için daha iyi bir çözümdür Ne

:

public event Action<object, double> OnNumberChanged; 

veya

public delegate void DNumberChanged(object sender, double number); 
public event DNumberChanged OnNumberChanged; 

ilk

az yazar alır, ama temsilci bir parametrelere isimlerini verir. Bunu yazarken, bence 2 numara kazanıyor, ama yanılıyor olabilirim.

Düzenleme: Farklı (üçüncü) bir yaklaşım kazanır. Alttarafı oku.

Answer 1

Ne 1 veya 2. Üçüncü bir seçenek kazanan

public event EventHandler<NumberChangedEventArgs> NumberChanged; 

Böyle EventArgs kapsamaz olay args için bir tür kullanmak gibi C# geliştirmek için stil kurallar bir dizi kırıyorsun olduğunu .

Evet, derleyicinin umurunda olmadığı gibi bunu yapabilirsiniz. Ancak, kodunuzu okuyan insanlar bir WTF yapacaklar.

Answer 2

Yapmanız gerekmiyorsa yeni bir tür oluşturmayın. Bunun daha iyi olduğunu düşünüyorum:

public event Action<object, double> OnNumberChanged; 

Action ve Func temsilci aileleri var olduğunu sebebi bu çok amaçlı hizmet etmek ve geliştiriciler tarafından yeni temsilci tipi yaratılması ihtiyacını azaltmaktır.

Answer 3

Seçenek 1'i seçtiğimde daha iyi olacağını düşünüyorum, ancak IIRC, olayların resmi yönergeleri, ikinci parametrenizin XxxEventArgs adıyla bir sınıf olması gerektiğini ve miras zincirinde EventArgs olmalıdır.

Answer 4

Kodlama stili ile ilgili tüm sorularda olduğu gibi. Tercih ettiğiniz birini seçin veya takımınızın tercih ettiğini ve proje boyunca tutarlı olmasını sağlayın. İhtiyacı olan herkes bunu okuyabildiği sürece iyi olacaksın.

Answer 5

Genellikle argüman olarak bir EventArgs türetilmiş sınıfını kullanmaya devam ediyorum. Kodu daha tutarlı hale getirir.

public class ApplyClickedEventArgs : EventArgs 
{ 
    ... 
} 

ve bir işleyici:

void cpy_ApplyClicked(object sender, ApplyClickedEventArgs e) 
{ 
    ... 
} 

beyanıdır: Aşağıda belirtildiği gibi

public event EventHandler<ApplyClickedEventArgs> ApplyClicked;