新しい歴史

コードの臭い 286 - 親子メソッドの重複

に Maximiliano Contieri4m2025/01/16

長すぎる; 読むには

親子クラスでプライベートメソッドが衝突すると、バグやコードの臭いが発生する可能性があります。明確な命名を維持し、重複を避け、クリーンなコードプラクティスに従うことで、この問題を回避します。親クラスのメソッドには、子クラスが使用できる名前を付けます。

親メソッドと子メソッドが衝突したとき！

TL;DR: 子クラスが使用できる名前を持つプライベートメソッドを親クラスで使用しないでください。

問題点

最も驚きが少ない原則違反
予期しない動作や欠陥
隠れた依存関係
拡張性が限られている
コードの曖昧さ
オープン/クローズ原則違反
誤解を招くデザイン

ソリューション

階層構造を避ける
プライベートメソッドの名前を変更する
明確な命名を維持する
名前の重複を避ける
保護されたメソッドを避ける
コードを再利用するのではなく、重要な関係に基づいてサブ分類する

コンテクスト

親クラスと子クラスで同じメソッド名を使用すると、混乱が生じます。

子クラスに同じ名前のパブリックメソッドが存在する場合でも、親クラスのプライベートメソッドをオーバーライドすることはできません。

これは、ほとんどの静的言語の設計に見られる問題です。この断絶によりバグが発生し、コードの保守が困難になります。

サンプルコード

間違っている

<? class ParentClass { private function greet() { // This method is private return "Hello from ParentClass"; } public function callGreet() { return $this->greet(); } } class ChildClass extends ParentClass { public function greet() { // Overriding a concrete method is a code smell // Compilers SHOULD warn you return "Hello from ChildClass"; } } $child = new ChildClass(); echo $child->callGreet(); // When callGreet() is invoked on the $child object, // it executes the following: // It calls $this->greet(), // which refers to the greet() method of ParentClass // because the original method is private // and cannot be overridden or accessed from ChildClass. // The unexpected output is 'Hello from ParentClass'

右

<? class ParentClass { protected function greet() { // notice the 'protected qualifier' return "Hello from ParentClass"; } public function callGreet() { return $this->greet(); } } class ChildClass extends ParentClass { public function greet() { return "Hello from ChildClass"; } } $child = new ChildClass(); echo $child->callGreet(); // The output is "Hello from ChildClass" // This is the standard (and wrong) solution // Also fixed by most AIs

 <? abstract class ParentClass { // Declare greet() as an abstract method // Following the template-method design pattern abstract protected function greet(); public function callGreet() { return $this->greet(); } } class ChildClass extends ParentClass { protected function greet() { return "Hello from ChildClass"; } } class OtherChild extends ParentClass { protected function greet() { return "Hello from OtherChild"; } } $child = new ChildClass(); echo $child->callGreet(); // Output: Hello from ChildClass $otherChild = new OtherChild(); echo $otherChild->callGreet(); // Output: Hello from OtherChild