2011-09-28 31 views
0

我有一個方法(讀),其募集當有新的日誌事件方法:如何命名這引發事件

public class LoggerModel: ILoggerModel { 
    public event System.EventHandler LogsChanged; 
    ... 
    public void Read() { 
    if (ThereAreNewLogs()) { 
     OnLogsChanged(System.EventArgs.Empty); 
    } 
    } 
} 

,但我覺得「閱讀()」不是表現...有什麼建議麼?

+0

LogWatchDog !!! – Burimi

回答

1

我叫它CheckIfLogChanged()

0

是的,這是令人困惑...我會稱之爲聽()或類似的東西..也許StartListening()

此外,您必須檢查是否有人通過調用

if(OnLogsChanged != null) 
{ 
    OnLogsChanged(EventArgs.Empty); 
} 
1

這是一個品味的問題註冊的OnLogsChanged事件

,我個人使用TriggerEvent,W這裏「事件」當然是事件的名稱。這也是檢查事件是否爲空的好地方。但對我而言,這是一種私人方法,而您的方法是公開的,而且如果引發事件需要「手動」調用某種提醒方法,這與基於事件的通信imho的想法不一致。

+1

或RaiseLogsChanged – Gishu

0

該方法的實際範圍是驗證並處理日誌中的新條目。

所以對我來說可能是VerifyNewLogs,或類似的東西。

但是,最好的解決方案是爲它添加文檔,在那裏描述該方法在做什麼。這肯定會幫助你或未來的其他人。