버튼 클릭 중복 코드를 피하는 효율적인 방법

Question

동일한 작업을 수행하지만 다른 객체를 수행해야하는 2 개의 버튼 마녀가 있다고 가정 해보십시오.

현재 나는이 같은 방법에 필요한 모든 참조를 전달 해요 : 그 작업을 수행하는 다른,보다 효율적인 방법이 있는지 궁금 해서요

private void sub1_add_to_db_btn_Click(object sender, EventArgs e) 
    { 
     Add_Substance_To_Database(
      substanse1, sub1_add_to_db_btn, sub2_add_to_db_btn, sub1_found_in_db_list, 
      sub2_found_in_db_list, false, sub1_listBox, sub2_listBox); 
    } 

    private void sub2_add_to_db_btn_Click(object sender, EventArgs e) 
    { 
     Add_Substance_To_Database(
      substanse2, sub2_add_to_db_btn, sub1_add_to_db_btn, sub2_found_in_db_list, 
      sub1_found_in_db_list, false, sub2_listBox, sub1_listBox); 
    } 

. 감사.

편집 :

이 내 코드의 일부처럼 보이는 방법이며 나를 미치게 만드는!

예를 들어

private void sub1_add_to_db_btn_Click(object sender, EventArgs e) 
    { 
     Add_Substance_To_Database(substanse1, sub1_add_to_db_btn, sub2_add_to_db_btn, 
      sub1_found_in_db_list, sub2_found_in_db_list, false, sub1_listBox, sub2_listBox); 
    } 

    private void sub2_add_to_db_btn_Click(object sender, EventArgs e) 
    { 
     Add_Substance_To_Database(substanse2, sub2_add_to_db_btn, sub1_add_to_db_btn, 
      sub2_found_in_db_list, sub1_found_in_db_list, false, sub2_listBox, sub1_listBox); 
    } 

    private void sub1_edit_name_btn_Click(object sender, EventArgs e) 
    { 
     Add_Substance_To_Database(substanse1, sub1_add_to_db_btn, sub2_add_to_db_btn, 
      sub1_found_in_db_list, sub2_found_in_db_list, true, sub1_listBox, sub2_listBox); 
    } 

    private void sub2_edit_name_btn_Click(object sender, EventArgs e) 
    { 
     Add_Substance_To_Database(substanse2, sub2_add_to_db_btn, sub1_add_to_db_btn, 
      sub2_found_in_db_list, sub1_found_in_db_list, true, sub2_listBox, sub1_listBox); 
    } 

    private void sub1_delete_from_db_btn_Click(object sender, EventArgs e) 
    { 
     Delete_Substance_From_DB(sub1_listBox, 
      sub2_listBox,sub2_list_is_from_file,sub1_delete_from_db_btn, 
      sub2_delete_from_db_btn); 
    } 

    private void sub2_delete_from_db_btn_Click(object sender, EventArgs e) 
    { 
     Delete_Substance_From_DB(sub2_listBox, 
      sub1_listBox,sub1_list_is_from_file,sub2_delete_from_db_btn, 
      sub1_delete_from_db_btn); 
    } 

: 나는 물질을 삭제하려면 , 나는 ... 목록 모두에서 삭제하고 다른 목록에서 제거, 다음 물질 등으로 선택을 변경해야

Answer 1

개발자 목적을 위해 객체의 . 이 방법에서는 단순히 보낸 사람에 따라 올바른 컨트롤러를 선택합니다.

substance1_controllers = new object[]{ 
      sub1_main_listbox, sub1_peaks_list,sub1_found_in_db_list, 
      sub1_similar_in_db_list, sub1_eigenvector_list, 
      sub1_sourse_switch_btn, sub1_folder_btn,sub1_add_to_db_btn, 
      sub1_edit_name_btn,sub1_delete_btn, sub1_picture_box, 
      chart_peaks.Series[0], chart_compare.Series[0], true, -1}; 
substance2_controllers = new object[]{ 
      sub2_main_listbox, sub2_peaks_list, sub2_found_in_db_list, 
      sub2_similar_in_db_list,sub2_eigenvector_list, 
      sub2_sourse_switch_btn, sub2_folder_btn, sub2_add_to_db_btn, 
      sub2_edit_name_btn,sub2_delete_btn, sub2_picture_box, 
      chart_peaks.Series[1],chart_compare.Series[1], true, -1}; 

가 그런 식으로 유지하기 어렵하지만 개인적으로 나는 그것이 매우 편안 유지하고이 테이블의 도움으로 사용하는 것으로 (그리고 멋지다)처럼이 보일 수 있습니다 :

 // [0] - Main Listbox 
     // [1] - Peaks Listbox 
     // [2] - Found in Database Listbox 
     // [3] - Found similar Listbox 
     // [4] - Eigenvectors Listbox 
     // [5] - Switch sourse Button 
     // [6] - Change Folder Button 
     // [7] - Add to Database Button 
     // [8] - Edit Name Button 
     // [9] - Delete Button 
     // [10] - Picture Box 
     // [11] - Peaks Chart Series 
     // [12] - Compare Chart Series 
     // [13] - List is from File Boolean 
     // [14] - Previous Selected Index 

방법 예 :

void Delete_Substance_From_DB(object sender, EventArgs e) 
    { 
     object[] controller; 
     object[] other_controller; 
     if (((Button)sender).Name == "sub1_delete_btn") 
     { 
      controller = substance1_controllers; 
      other_controller = substance2_controllers; 
     } 
     else 
     { 
      controller = substance2_controllers; 
      other_controller = substance1_controllers; 
     } 
    } 

사용 examle :

if (((ListBox)other_controller[0]).Items.Count != 0) 
     { 
      if (((ListBox)other_controller[0]).Items.Count == index2) 
      { 
       ((ListBox)other_controller[0]).SelectedIndex = index2 - 1; 
      } 
      else 
      { 
       ((ListBox)other_controller[0]).SelectedIndex = index2; 
      } 
      Main_Listbox_Index_Changed(((ListBox)other_controller[0]), null); 
     } 

Answer 2

다른 값을 Add_Substance_To_Database에 전달하면 감당할 수있는 가장 유지하기 쉬운 코드 일 것입니다.

당신은 두 버튼에 하나의 이벤트 핸들러를 첨부 할 수 있지만, 당신은 눌러 진 버튼 해결해야하고 어쨌든 관련 인수를 전달할 것 : 정확히데도 당신을 얻을 수 없습니다

if (sender == button1) 
{ 
    Add_Substance_To_Database(
     substanse1, sub1_add_to_db_btn, sub2_add_to_db_btn, sub1_found_in_db_list, 
     sub2_found_in_db_list, false, sub1_listBox, sub2_listBox); 
} 
else 
{ 
    Add_Substance_To_Database(
     substanse2, sub2_add_to_db_btn, sub1_add_to_db_btn, sub2_found_in_db_list, 
     sub1_found_in_db_list, false, sub2_listBox, sub1_listBox); 
} 

합니다.

Answer 3

이것은 코드 복제의 경우가 아닙니다. 이러한 경우 리팩토링 만 그것을 만들 것입니다 :

• 더 복잡
• 읽을 수

참조하십시오 Any valid reason for code duplication?

편집 :

당신이 경우에 더 이상 '고기'를 가지고 있다면 복제 된 핸들러는 아마도 다른 것을 제안 할 것입니다. 그러나 당신은 Add_Substance_To_Database에 모든 것을 가져 와서 이미 코드를 성공적으로 중복 제거했습니다.

Answer 4

이벤트 핸들러는 보낸 사람에 당신이 버튼 컨트롤, 호출 이벤트를 버튼으로 버튼을

VAR 방법의 parametrs로 Button 클래스 및 사용 탐에 = 보낸 사람을 캐스팅, 몇 가지 컨트롤의 하나가 될 수있다;

Add_Substance_To_Database ( substanse2, 버튼, sub1_add_to_db_btn, sub2_found_in_db_list, sub1_found_in_db_list, 거짓 sub2_listBox, sub1_listBox); 당신은 속성 태그를 사용할 수있는 다른 매개 변수에 대한

는, 내가 모든 컨트롤러 객체 배열을 생성하고, 컨트롤러 이벤트 핸들러에 동일한 방법을 추가 ... 결국 나는 내 자신의 제안을 따라