如何简化主窗体和子窗体之间传递的设置值的代码

Question

我有一个类存储我的应用程序的设置。它是在应用程序时实例化的。应用程序运行并保存。关闭。

public class Settings
{
    public bool showPrivacyPageOnBlogs;
    public bool showTermsPageOnBlogs;
    public bool showDisclosurePageOnBlogs;
}

还有一个弹出窗口，显示复选框以使用弹出窗口的公共属性设置这些值。

处理弹出窗口的代码如下：

// Horrible code ahead
private void pagesSettingsToolStripMenuItem1_Click(object sender, EventArgs e)
{
    pagesSettingsForm.showPrivacyPageOnBlogs = settings.showPrivacyPageOnBlogs;
    pagesSettingsForm.showTermsPageOnBlogs = settings.showTermsPageOnBlogs;
    pagesSettingsForm.showDisclosurePageOnBlogs = settings.showDisclosurePageOnBlogs;
    if (pagesSettingsForm.ShowDialog() == DialogResult.OK)
    {
        settings.showPrivacyPageOnBlogs = pagesSettingsForm.showPrivacyPageOnBlogs;
        settings.showTermsPageOnBlogs = pagesSettingsForm.showTermsPageOnBlogs;
        settings.showDisclosurePageOnBlogs = pagesSettingsForm.showDisclosurePageOnBlogs;
    }
    pagesSettingsForm.Dispose();
}

在我的应用程序中。还有更多参数以这种方式处理，所以我想知道是否有某种方法可以简化此代码，以便枚举设置的名称并允许将来添加其他参数。

Answer 1

只需让表单公开一个带有 getter 和 setter 的 Settings 类型的属性即可。这使得您发布的代码片段变得简单，在将成员添加到“设置”时无需进行任何更改。现在工作转移到表单实现上。 PropertyGrid 是一个通用的对象编辑器，很难猜测它在您的情况下是否足够可用。

Answer 2

虽然我还没有尝试过，但我坚信 Automapper 可以处理这个问题。我认为这可能会让你的代码变成这样：

// Horrible code ahead
private void pagesSettingsToolStripMenuItem1_Click(object sender, EventArgs e)
{
    Mapper.Map(settings, pagesSettingsForm);
    if (pagesSettingsForm.ShowDialog() == DialogResult.OK)
        Mapper.Map(pagesSettingsForm, settings);
    pagesSettingsForm.Dispose();
}    

PS：我知道你说代码很糟糕，但我不能不提到你正在处理一个在其他代码中实例化的表单 - 这是错误的，国际海事组织。

Answer 3

使用字典

Dictionary<String,dynamic> 

会很好，因为它会推迟输入，

因此您可以在设置类中为应用程序所需的所有设置添加/设置设置。
您的表单有它自己的实例，但仅包含它需要的实例。
然后，只需扫描 form.settings 中传递的设置中的相同命名版本并覆盖它们即可。