清楚表明重写方法应该调用 super 的好方法吗？

Question

这个问题困扰着我，在重写方法时很容易忘记调用 super() 。

就我而言，我正在重构一些现有的东西，其中已经有大约十个类覆盖了一个方法。直到昨天，该方法还有一个空的默认实现，因此子类是否调用 super 并不重要。 您可以发现重写器与任何值得使用的 IDE 配合得很好，但您知道它是怎么回事，电话铃声响起，同事在您背后闲聊……很容易忘记检查某个位置或以其他方式忽略它。 理想情况下，@Override 注释会有一个对应项，并且如果基类方法被注释但重写不调用 super，编译器将为这些位置生成警告。

我能做的下一件最好的事情是什么？

Answer 1

不太优雅，但可能的解决方案是将该方法分成两个：

public abstract class Foo {
    public void doBar() {
        // do your super logic
        doBarInternal();
    }

    public abstract void doBarInternal();
}

Answer 2

如果必须始终调用超类实现，您可以使用“模板方法”模式。

所以你现在所拥有的是这样的：

public static class Parent {
    public void doSomething() {
        System.out.println("Parent doing something");
    }
}

public static class Child extends Parent {
    public void doSomething() {
        // MUST NOT FORGET SUPER CALL
        super.doSomething();
        System.out.println("Child doing something");
    }
}

public static void main(String[] args) {
    Child child = new Child();
    child.doSomething();
}

这将变成：（

public abstract static class Parent {
    public final void doSomething() {
        System.out.println("Parent doing something");
        childDoSomething();
    }

    public abstract void childDoSomething();
}

public static class Child extends Parent {
    public void childDoSomething() {
        System.out.println("Child doing something");
    }
}

public static void main(String[] args) {
    Child child = new Child();
    child.doSomething();
}

类被设为静态，以便在一个类中轻松测试）

我将 doSomething 设置为 Final 以避免它被覆盖，因为在这个解决方案中应该实现 childDoSomething 。

当然，这个解决方案意味着 Parent 不能再用作具体类。

编辑：阅读有关 Child 实现第三方接口的评论后；这不一定是问题：

public interface ThirdPartyInterface {
    public void doSomething();
}

public abstract static class Parent {
    public final void doSomething() {
        System.out.println("Parent doing something");
        childDoSomething();
    }

    public abstract void childDoSomething();
}

public static class Child extends Parent implements ThirdPartyInterface{
    public void childDoSomething() {
        System.out.println("Child doing something");
    }

//    public final void doSomething() {
//        // cannot do this because Parent makes it final
//    }
}

public static void main(String[] args) {
    Child child = new Child();
    child.doSomething();
}    

Answer 3

在寻找其他东西时，我发现了有趣的 FindBugs 中的 OverrideMustInvoke 注释：
http://findbugs.sourceforge.net/api/edu /umd/cs/findbugs/annotations/OverrideMustInvoke.html

Answer 4

如果您不坚持编译时安全，您可以使用一种机制，只要子类的行为不符合逻辑要求，就会抛出异常：如何强制对 super 方法进行多态调用？

Answer 5

我有 2 个不同的建议：

1）构建一个 junit 测试，发现基类的所有子类，然后选择所有用 @Override 注释修饰的方法。我有一些执行类似操作的单元测试（例如，查找所有子类，检查它们是否真正是可序列化的）。

不幸的是，验证他们是否称为“超级”有点不那么简单。您要么需要让测试查找源文件并搜索它，要么更好（但我不知道如何执行此操作），读取字节代码并查看是否可以检测到对 super 的调用。

2）需要保证对super的调用可能是设计/接口问题的指示，而不是编码/实现问题。如果你真的想保证用户调用super，最好将super类抽象化，明确指定一个抽象实现方法供他们重写，并让super控制执行流程。

如果您想定义一个默认实现，以便并非所有用户都需要子类提供实现该方法，您可以定义一个默认实现类供人们使用。 （如果您确实想控制它，请将默认类实现方法定义为final，以强制它们返回子类化抽象父级。）

代码重用继承总是更难管理，因此它需要仔细做。任何进行代码重用继承的人都必须对超类的内部结构有很好的了解才能正确地做到这一点（这有点令人讨厌）。例如，您是否必须在重写代码的开头或末尾调用 super.method() ？ （或者你可以在中间做...）

所以总而言之，最好的解决方案是尝试避免必须强制执行的设计。

Answer 6

什么可以工作 - 创建一些标记注释（即@MustCallParent），然后创建一些注释处理器来检查用它标记的方法是否符合约束。