Reuse event handler хорошая практика в C#

Question

В моем текущем проекте я использую четыре вида сетки в разных вкладках. По мере разработки системы у них есть несколько общих методов, таких как показ пользовательской всплывающей подсказки и контекстное меню, когда в строках.

Теперь я выполняю упражнение по очистке кода. Ниже я вижу, что теперь у меня есть четыре обработчика событий, вызывающих один и тот же метод. Можно ли изменить обработчики событий, чтобы указать прямо на GridMenu, избегая лишнего кода? Повлияет ли это на меня позже?

Очевидно, что в настоящее время я использую стандартные имена обработчиков.

private void grdEnquiriesLevel1_ShowGridMenu(object sender, GridMenuEventArgs e) 
    { 
     GridMenu(sender, e); 
    } 

    private void grdApplicantsLevel1_ShowGridMenu(object sender, GridMenuEventArgs e) 
    { 
     GridMenu(sender, e); 
    } 

    private void grdApplicationsLevel1_ShowGridMenu(object sender, GridMenuEventArgs e) 
    { 
     GridMenu(sender, e); 
    } 

    private void grdInterviewsLevel1_ShowGridMenu(object sender, GridMenuEventArgs e) 
    { 
     GridMenu(sender, e); 
    } 

    private void GridMenu(object sender, GridMenuEventArgs e) 
    { 
     GridView view = (GridView)sender; 

     if (view.CalcHitInfo(e.Point).InRow) 
      popupMenu1.ShowPopup(Cursor.Position); 
    } 

Answer 1

Вместо регистрации непосредственно на GridMenu, создайте общий обработчик событий с именем Grid_ShowGridMenu.

Просто зарегистрируйтесь в том же обработчике событий для каждой сетки, вместо того, чтобы создавать отдельный обработчик событий для каждой сетки.

grdEnquiriesLevel1.ShowGridMenu += Grid_ShowGridMenu; 
grdApplicantsLevel1.ShowGridMenu += Grid_ShowGridMenu; 
grdApplicationsLevel1.ShowGridMenu += Grid_ShowGridMenu; 
grdInterviewsLevel1.ShowGridMenu += Grid_ShowGridMenu; 


private void Grid_ShowGridMenu(object sender, GridMenuEventArgs e) 
{ 
    GridMenu((GridView)sender, e.Point); 
} 

Теперь, вместо передачи sender, e непосредственно GridMenu, проходит только необходимых значения GridMenu и изменить подпись GridMenu поэтому он может быть более многоразовые.

private void GridMenu(GridView grid, Point hitPoint) 
{ 
    if (grid.CalcHitInfo(hitPoint).InRow) 
     popupMenu1.ShowPopup(Cursor.Position); 
} 

Answer 2

Пока код события является общим для всех элементов управления, этот метод является точным и чистым.

Если вы начинаете с ключами if/else или switch в коде, то, возможно, пора передумать.

Answer 3

Вы должны создать пользовательский элемент управления, который инкапсулирует вашу пользовательскую сетку. Таким образом, все ваше поведение будет инкапсулировано и повторно использовано.