程式碼異味 - 重疊方法-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>

登入後複製

偵測與預防：