Как назвать метод, который вызывает событие - PullRequest
0 голосов
/ 28 сентября 2011

У меня есть метод (Read), который вызывает событие при появлении новых журналов:

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

но я чувствую, что "Read ()" не выразителен ... есть предложения?

Ответы [ 4 ]

1 голос
/ 28 сентября 2011

Это вопрос вкуса, я лично использую TriggerEvent, где «событие» - это название события, конечно.Это также хорошее место, чтобы проверить, не является ли событие пустым.Но в моем случае это частный метод, тогда как ваш Read метод является общедоступным, и если для вызова события требуется вызвать некоторый метод напоминания «вручную», это противоречит самой идее коммуникации на основе событий imho.

1 голос
/ 28 сентября 2011

Я бы назвал это CheckIfLogChanged()

0 голосов
/ 28 сентября 2011

Реальная область применения этого метода - проверить это и обработать новые записи в журнале.

Так что для меня может быть VerifyNewLogs или что-то вроде этого.

Но лучшим решением было бы добавить для него документацию, в которой вы опишете, что делает метод. Это наверняка поможет вам или кому-то еще в будущем.

0 голосов
/ 28 сентября 2011

Да, это сбивает с толку ... Я бы назвал это Listen () или что-то в этом роде ... возможно, StartListening ()

Кроме того, вы должны проверить, зарегистрировался ли кто-то на событие OnLogsChanged

по телефону

if(OnLogsChanged != null)
{
  OnLogsChanged(EventArgs.Empty);
}
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...