0

所以我正在使用 WinForms 在 C# 中构建一个 MDI 应用程序。在这种情况下，我一生都无法弄清楚如何遵循 DRY。我正在使用我的主菜单条中的工具条菜单项打开新的 MDI 子项。我重用了“NewChild”方法作为每个事件处理程序。我试图不必为每个子窗口重复自己，因为它们都遵循相同的实例化模式。

我研究了泛型并使用了 Type 类，但它并没有真正让我到达我想要的地方。理想情况下，我只想说

    // etc...
    TypeOfForm = ConfigurationForm;
}

new TypeOfForm();

但我不认为这种丢失的语言结构存在。

public partial class MainForm : Form
{
    private AboutForm aboutForm;
    private ConfigurationForm configForm;
    private ResultsForm resultForm;
    private LogForm logForm;

    private void NewChild(object sender, EventArgs e)
    {
        Form newForm;

        if (sender == testConfigurationToolStripMenuItem)
        {
            if (configForm == null)
            {
                configForm = new ConfigurationForm();
            }
            newForm = configForm;
        }
        else if (sender == resultsToolStripMenuItem)
        {
            if (resultForm == null)
            {
                resultForm = new ResultsForm();
            }
            newForm = resultForm;
        }
        else if (sender == logToolStripMenuItem)
        {
            if (logForm == null)
            {
                logForm = new LogForm();
            }
            newForm = logForm;
        }
        else
        {
            return;
        }

        newForm.MdiParent = this;
        newForm.Disposed += new EventHandler(ChildDisposed);
        newForm.Show();
    }
}

在这种情况下实施 DRY 的好方法是什么？

4

2 回答 2

1

我会不惜一切代价避免检查类型。它确实使代码混乱。

你真的想为这个通用代码使用泛型：

// for multiple instance forms (and instantiating a "singleton" form)
private void AddNewChild<T>() where T: Form
{
    T newForm = new T();
    newForm.MdiParent = this;
    newForm.Disposed += new EventHandler(ChildDisposed);
    newForm.Show();   
}

// for "singleton" forms
private void ActivateChild<T>() where T: Form
{
    // off-the-cuff guess, this line may not work/compile
    var child = this.MdiChildren.OfType<T>().FirstOrDefault();

    if (child == null) 
    {
        AddNewChild<T>();
    }
    else
    {
        child.Show();
    }
}

// usage
logToolStripMenuItem.Click += (s,e) => ActivateChild<LogForm>();
testConfigurationToolStripMenuItem.Click += (s,e) => ActivateChild<ConfigurationForm>();
multipleInstanceFormMenuItem.Click += (s,e) => AddNewChild<FormX>();
...
于 2013-03-26T17:48:54.317 回答
0

这里有一个建议：

使用工具条对象（或使用 Tag 属性）及其匹配的表单或表单类型制作字典

// you can't use the UI controls before Init, you could use their Tags
// so this should be considered pseudo code
private Dictionary<string, Type> ToolstripForms = new Dictionary<string, Type>
{
    { testConfigurationToolStripMenuItem, typeof(ConfigurationForm) }, 
    { resultsToolStripMenuItem, typeof(ResultsForm) }, 
};

并在您的 NewForm 方法中：

private void NewChild(object sender, EventArgs e)
{
    Form newForm = null;

    // some casting and exception handling would go well here
    if (sender != null && ToolstripForms.ContainsKey(sender)) // or sender.Tag?
    {
        newForm = Activator.CreateInstance(ToolstripForms[sender]) as Form;
    }
    ...
}

您可能需要一种方法来确定表单是否已经打开并关注它，而不是每次都打开一个新的，但这取决于您。一种方法是使用一对对象作为该字典中的值 - 类型和现有实例，尽管其他方法可能看起来更好。

于 2013-03-26T18:04:06.273 回答