代码异味 - 重叠方法-php教程-PHP中文网

代码异味 - 重叠方法

Barbara Streisand

发布： 2025-01-16 12:33:58

原创

441 人浏览过

当父子方法发生冲突时：代码气味分析

执行摘要：避免将私有父类方法与子类中的方法命名相同。这可以防止意外行为，提高代码清晰度并增强可维护性。

问题区域：

违反最小惊讶原则：由于父类私有方法优先而未调用子类的方法时，会出现意外行为。
不可预见的行为和缺陷：私有父方法默默地覆盖子方法，导致微妙且难以调试的错误。
隐藏依赖：父子方法之间的关系模糊，使得代码更难理解和修改。
有限的可扩展性：由于隐藏的冲突，添加新功能或改变现有行为变得具有挑战性。
代码歧义：代码的意图变得不清楚，增加了误解的风险。
违反开闭原则：修改父类需要更改子类，违反了这一关键设计原则。
误导性设计：代码结构没有准确反映类之间的预期关系。

解决策略：

避免继承层次结构（如果可能）：如果关系并不能真正保证继承，请考虑替代设计模式，例如组合。
重命名私有方法：在父类和子类中为私有方法使用不同的名称，以消除命名冲突。
保持一致的命名约定：在整个代码库中采用清晰一致的命名方案，以防止意外重叠。
防止名称重叠：仔细选择方法名称以避免任何潜在的冲突。
避免受保护的方法（不必要时）：虽然受保护的方法提供了更大的灵活性，但过度使用可能会导致类似的问题。
为真实关系进行子分类，而不是代码重用：继承应该反映“is-a”关系，而不仅仅是重用代码的愿望。考虑替代方法，例如辅助函数或实用程序类。

说明性代码示例：

错误实施：

<code class="language-java">class ParentClass {
    private void greet() {
        System.out.println("Hello from ParentClass");
    }

    public void callGreet() {
        this.greet();
    }
}

class ChildClass extends ParentClass {
    public void greet() {
        System.out.println("Hello from ChildClass");
    }
}

ChildClass child = new ChildClass();
child.callGreet(); // Output: Hello from ParentClass (Unexpected!)</code>

登录后复制

正确实现（使用受保护）：

<code class="language-java">class ParentClass {
    protected void greet() {
        System.out.println("Hello from ParentClass");
    }

    public void callGreet() {
        this.greet();
    }
}

class ChildClass extends ParentClass {
    @Override
    public void greet() {
        System.out.println("Hello from ChildClass");
    }
}

ChildClass child = new ChildClass();
child.callGreet(); // Output: Hello from ChildClass</code>

登录后复制

正确实现（使用抽象方法）：

<code class="language-java">abstract class ParentClass {
    protected abstract void greet();

    public void callGreet() {
        this.greet();
    }
}

class ChildClass extends ParentClass {
    @Override
    protected void greet() {
        System.out.println("Hello from ChildClass");
    }
}

ChildClass child = new ChildClass();
child.callGreet(); // Output: Hello from ChildClass</code>

登录后复制

检测与预防：