c# - C#重构:您将如何重构2个方法，除了使用不同的接口(interface)外，它们会执行相同的操作

必须有2个不同的接口。

您将如何重构呢？

我应该完全重构此代码吗？

private void CreateInstanceForProviderA()
{
    a = FactorySingleton.Instance.CreateInstanceA("A");

    if (a == null)
    {
        ShowProviderNotInstanciatedMessage();
        return;
    }

    a.Owner = Handle.ToInt32();
    lbl_Text.Text = a.Version();
}

private void CreateInstanceForProviderB()
{
    b = FactorySingleton.Instance.CreateInstanceB("B");

    if (b == null)
    {
        ShowProviderNotInstanciatedMessage();
        return;
    }

    b.Owner = Handle.ToInt32();
    lbl_Text.Text = b.Version();
}

如果会有一个通用的接口，我可以这样写：

private void CreateInstanceForProvider(string provider)
{

    p = FactorySingleton.Instance.CreateInstanceB(provider);
    // p is shared over the whole class
    if (p == null)
    {
        ShowProviderNotInstanciatedMessage();
        return;
    }

    var tmpProvider = p as ICommonProvider;

    tmpProvider .Owner = Handle.ToInt32();
    lbl_Text.Text = tmpProvider .Version();
}

最佳答案

好吧，首先要做的是对FactorySingleton的作者大喊大叫，以修复该死的代码，以便使ClassA和ClassB具有用于其公共字段的公共接口。

同时，您几乎无法使用反射，这太丑陋，仅此一点就不值得了。