压缩大量按钮单击事件

时间:2015-11-26 10:55:29

标签: c# refactoring

我有40个按钮,点击时都会略微不同,如果可以的话,我想把它浓缩一下。我还想说,如果单击其中一个按钮,则创建一个时间戳,该类可以被类访问。

以下是40个按钮中的2个的代码:

private void Btn1_Click(object sender, RoutedEventArgs e)
{
   for (int i = 1; i < 5; i++)
   {
       CheckBox CheckBox = (this.FindName(string.Format("Check{0}", i)) as CheckBox);

       if (CheckBox != null)
       {
          CheckBox.IsChecked = true;
       }
    }
}

private void BtnDisable1_Click(object sender, RoutedEventArgs e)
{
  for (int i = 1; i < 5; i++)
  {
    CheckBox CheckBox1 = (this.FindName(string.Format("Check_D{0}", i)) as CheckBox);

    if (CheckBox1 != null)
    {
       CheckBox1.IsChecked = false;
    }
  }
}

我认为这样做的一种方法是将它放在一个数组中,每当点击40个按钮中的一个时,它会在数组中查看下一步该做什么?我不太确定,谢谢!

1 个答案:

答案 0 :(得分:1)

您可以使用一种方法简化这一过程。

根据this讨论

更新答案
private void DoWork(int checkboxGroup, bool enable)
{
    int start = checkboxGroup * 4;
    for (int i = start; i < start + 4; i++)
    {
        CheckBox CheckBox = this.FindName("CheckBox" + i) as CheckBox;

        if (CheckBox != null)
        {
            CheckBox.IsChecked = enable;
        }
    }
}

private void Btn1_Click(object sender, RoutedEventArgs e)
{
    DoWork(1 , true);
}
private void BtnDisable1_Click(object sender, RoutedEventArgs e)
{
    DoWork(1 , false);
}

因为有40种这样的方法,所以你可以使用Expression bodied方法。您必须拥有C#6才能使用此功能。

private void Btn1_Click(object sender, RoutedEventArgs e) => DoWork(1 , true);
private void BtnDisable1_Click(object sender, RoutedEventArgs e) => DoWork(1 , false);

private void Btn2_Click(object sender, RoutedEventArgs e) => DoWork(2, true);
private void BtnDisable2_Click(object sender, RoutedEventArgs e) => DoWork(2, false);

// and so on