如何避免重复代码以提高效率

Question

我有一个 DataGrid view1 和一个 ListView ，每当我选择列表视图项时（我将 ListView 项传递到查询中并填充根据该项目的 DataGrid 视图）

我已经编写了一些这样的代码......

 private void listview_selectedindexchanged(object sender event args)
 {
     if (listview.SelectedItems.Count > 0 && listview.SelectedItems[0].Group.Name == "abc")
     {
            if(lstview.SelectedItems[0].Text.ToString() == "sfs")
            {
              method1();
            }
            else
            {
                // datagrid view1 binding
               blah.....
             }
     }
     if (lstview.SelectedItems.Count > 0 && lstview.SelectedItems[0].Group.Name == "def")
     {

           if(lstview.SelectedItems[0].Text.ToString() == "xyz")
           {
               method 1();
           }
           if(lstview.SelectedItems[0].Text.ToString() == "ghi")
           {
               method 2(a,b);
           }
           if(lstview.SelectedItems[0].Text.ToString() == "jkl")
           {
               method 2(c,d);
           }
           if(lstview.SelectedItems[0].Text.ToString() == "mno")
           {
               method 3();
           }

       }
   }  
private void method 1()
{ 
  // datagrid view1 binding
    blahh     
}
private void method 2(e,g)
{
  // datagrid view1 binding
  blah....blah..
}
private void method 3()
{

    // datagrid view1 binding
}

我已经像上面那样完成了......我认为这不是一种有效的编码方式。而且这段代码由很多重复的行组成，有什么办法可以将这段代码折射成一小堆代码...... 为了提高效率？

任何提高代码效率的想法和示例片段都会对我有所帮助...

提前非常感谢...

我正在使用 c# 并编写 WinForms 应用程序.....

Answer 1

您可以将委托保存到列表视图项中。并在选择封装项时调用它。例如，您将像这样填充列表框：

ListViewItem item = new ListViewItem("abc");
item.Tag = new Delegate(method1);
lstview.Items.Add(item);

现在，当选择此项时，您将执行如下方法：

private void listview_selectedindexchanged(object sender event args)
{
    ((Delegate)lstview.SelectedItems[0].Tag)(); // this will execute method1 if the item with text "abc" gets selected
}

注意：！尚未测试此代码，但类似的内容应该可以工作，并且您不必编写 If 语句，您只需正确构造项目即可。

另请注意，对于刚接触此代码的人来说，这可能有点难以阅读。

Answer 2

您可以轻松地提取一个新方法来执行“datagrid view1 绑定”。然后从所有需要进行绑定的方法中调用此方法。