更干净的抽象方式

Question

我正在重构一些旧代码，并对设计决策感到震惊

AbstractClassA
-Step 1
-Step 2
--Step 2.1
--Step 2.2
-Step 3

上面的抽象类具有抽象方法Step 1，Step 2和Step3。Step2始终需要调用方法2.1和2.2。 但是在当前设计中，步骤2.1和2.2并未声明为抽象，而是在每个继承的类中都已实现并调用了该步骤。 我打算通过将所有这些方法（包括2.1和2.2）拉入接口来重构代码。 然后，我打算对该接口进行抽象类实现，其中第2步将调用2.1和2.2。 但是，这似乎并不整洁。 我想知道这个设计是否有缺陷？

InterfaceA
-Step1
-Step2
-Step 2.1
-Step 2.2
-Step3

Answer 1

听起来您正在寻找的是：

interface IMyInterface
{
    void Step1();
    void Step2();
    void Step3();
}

abstract class MyBaseClass : IMyInterface
{
    public abstract void Step1();
    public void Step2()
    {
        Step2_1();
        Step2_2();
    }
    public abstract void Step3();

    protected abstract void Step2_1();
    protected abstract void Step2_2();
}

当您想通用地引用某些东西时，可以更改运行时使用的确切实例，请使用抽象接口。 使用基类进行共享实现，因此您可以在每个子类中定义在步骤2.1和步骤2.2中发生的情况，但是您的基类定义了步骤2意味着先执行步骤2.1，然后执行步骤2.2。

Answer 2

您必须问自己一个问题，子类中的步骤2.1和步骤2.2的实现是否是专业化的，这对每个子类实现自己的步骤有意义。 如果是这样，则将它们抽象化。 如果不是，请使用通用实现，甚至不要将其虚拟化。 如果这样做，它甚至可能会很整洁。

Answer 3

好吧，您需要该接口，还是只是认为自己需要接口？

我听到的是您有AbstractClassA，它有两个方法，另一个方法调用了该方法。 如果应该公开访问这两种方法，则将它们放入接口中。 如果不是，请不要这样做，而应在接口的抽象实现中使它们成为受保护的虚拟或受保护的抽象。

Answer 4

因此，似乎Step2是接口的一部分-此类的调用者希望能够调用Step2。 但是Step2.1和Step2.2是实现细节，您不希望调用者看到的东西，尽管接口的所有实现者都将以这种方式实现-我是否正确？

如果是这样，那么我的界面将包括Step1，Step2和Step3（仅）。 抽象类将实现Step2，并进一步定义受保护的Step2.1和Step2.2，但将它们定义为抽象，从而强制子类提供实现。

除非您的意思是在所有子类中Step2.1和Step2.2都是相同的，否则我将把它们设为私有并将其放在抽象类中。