在大型窗体中管理EventHandlers

Question

我正在开发一个WinForm应用程序，到目前为止，我在管理大小和内容方面做得很差。我希望有人能给我一个例子，告诉我如何打破我在主表单cs文件中的一些逻辑。

以下是我在MainWindow.cs:中拥有的EventHandler函数的示例

    private void GroupBoxRequestTypeCheckedChanged(object pSender, EventArgs pEventArgs)
    {
        RadioButton vRadioButton = pSender as RadioButton;
        if (vRadioButton != null)
        {
            this.fSelectedButton = vRadioButton.Checked ? vRadioButton : null;
            if (vRadioButton.Equals(this.RadioButton_Copy) || vRadioButton.Equals(this.RadioButton_Delete) || vRadioButton.Equals(this.RadioButton_Download)
                || vRadioButton.Equals(this.RadioButton_Move) || vRadioButton.Equals(this.RadioButton_Upload))
            {
                this.GroupBox_Files.Enabled = true;
                this.GroupBox_Variables.Enabled = false;
            }
            else
            {
                this.GroupBox_Files.Enabled = false;
                this.GroupBox_Variables.Enabled = true;
            }
            if (this.fSelectedButton != null)
            {
                if (this.fSelectedButton.Equals(this.RadioButton_Delete))
                {
                    this.TextBox_DestinationFile.Enabled = false;
                    this.Button_DestinationBrowse.Enabled = false;
                }
                else
                {
                    this.TextBox_DestinationFile.Enabled = true;
                    this.Button_DestinationBrowse.Enabled = true;
                }
            }
        }
    }

因此，这只是我在表单中拥有的众多事件处理程序之一。我创建了一个MainForm，它有一个Tabbed Pane，并有一组选项卡，每个选项卡中都有按钮、文本框、复选框等。我处理的所有事件都进入MainForm.cs文件，现在我在这个文件中有近1000行。

有人能给我一个简单的例子（或一篇文章/文件）来详细说明良好的结构吗？我可以在一个单独的类中定义我的EventHandler函数吗（如果可以，这将如何工作…）我是否可以创建某种静态Helper类，在那里我只传递我需要操作的对象的实例？即

    private void GroupBoxRequestTypeCheckedChange(object pSender, EventArgs pEventArgs)
    {
        HelperClass.HandleGroupBoxRequestTypeCheckedChanged(pSender, pEventArgs, this);
    }

"this"在哪里是Form本身，它具有对我需要操作的对象的所有引用？

可能值得注意的是，我已经学习了很多关于跨线程调用的知识，并且我已经开始为许多我需要的简单实例制作Extension方法。

另一个问题-我注意到，默认情况下，Visual Designer会自动将用它创建的所有组件设为私有组件，通常情况下，将这些组件设为内部组件并使用表单对象从类外部根据需要引用这些组件是个坏主意吗？如果这不是一个好主意，还有什么更好的呢？

Answer 1

首先，我建议将独立的用户界面部分分离为UserControls或Components。然后，如果需要，使用事件（例如，您自己的专门事件和属性。

例如，您可以将主要内容（TabControl/Contain）放在UserControl中，并将该用户控件放在主窗体中。所有选项卡/页面切换逻辑/UI等都属于该用户控件。例如，在UserControl中，您可以定义自己的事件，当用户切换选项卡时，该事件会被触发。然后，主窗体可以注册到此事件，就像它可以注册其他Winforms控件事件一样，并执行它的任务（例如，更改窗口标题以表示当前活动的选项卡）。

接下来，您可以将每个选项卡的内容移动到其自己的用户控件中，并在新的选项卡用户控件中使用这些用户控件。将逻辑下移到负责给定任务的UserControl。

一些典型应用程序的表单/控件层次结构可能如下所示：

• 主窗体（窗体）
  • 主选项卡ContainerControl（UserControl）
  • Page1控件（UserControl）
  • Page2Control（UserControl）
    • MyImprovedDbRowGridControl（UserControl或组件）
  • Page3控件（UserControl）
  • 边栏控件（UserControl）
  • SearchControl（UserControl）
    • MyImprovedDbRowGridControl（UserControl或组件）
  • QuickHelpControl（用户控制）

下一件事是让所有的UI事件处理程序尽可能小，只做UI的事情。将其他逻辑（如业务或数据访问逻辑）移动到用户界面之外的其他类。

如果在应用程序中有多次需要的控件组合：请将它们移动到可重用的UserControl。（例如面包屑）。

关于您的示例代码，您可以通过简化其逻辑使其更加紧凑，从而可以进行维护：

if (this.fSelectedButton.Equals(this.RadioButton_Delete))
{
    this.TextBox_DestinationFile.Enabled = false;
    this.Button_DestinationBrowse.Enabled = false;
}
else
{
    this.TextBox_DestinationFile.Enabled = true;
    this.Button_DestinationBrowse.Enabled = true;
}

可能是：

    var delete = fSelectedButton == RadioButton_Delete;
    this.TextBox_DestinationFile.Enabled = !delete;
    this.Button_DestinationBrowse.Enabled = !delete;

更新：当涉及到重构和代码清理时，一个非常有用的工具是Resharper（R#）。我可以强烈推荐它。

希望这能给你一些从哪里开始的想法。