共通するメソッドがあるクラスのリファクタリング方法

Question

困っていること

二つの似ているクラスがある場合どのようにリファクタリングしたらいいでしょうか？
Baseを継承しているため、新たに継承させることができません。
またBaseに共通しているメソッドを書いてしまうと、この二つ以外のBaseを継承しているクラスにも反映されてしまします。
ヒント、アドバイスなど頂けませんでしょうか。よろしくお願いします。

class Area extends Base
{

    protected static $data = array(
        '0' => '東',
        '1' => '西'
    );

    public static function getData()
    {
        return self::$data;
    }

    public static function getName($code)
    {
        $data = self::getData();
        return $data[$code];
    }
}

<?php
class Pref extends Base
{
  protected $data = array(
    '1' => '北海道',
    '2' => '青森県',
    '3' => '岩手県',
  　// 省略
  );

  public static function getData()
  {
    return self::$data;
  }

  public static function getName($code) {
     $data = self::getData();
     return $data[$code];
  }
}

Answer 1

たぶん元々の設計が悪すぎてどうにもならないのでは…。
Baseに書いたら困る時点で何か間違えてる。

Baseに影響与えず共通化する一番簡単な方法はtrait

Comments

[bws]

現時点ではBaseに書いても問題ないのですが、このままBaseにどんどん書いていいんだろうかという疑問がありましたが良いのかとスッキリしました。
どうにもならない時はtraitを検討してみたいと思います。ありがとうございました！

Answer 2

$prefs は $data の誤りですかね？

一番確実かつ分かりやすいのは、Base の下に１つ共通クラスを挟むことでしょうか。
Base -> Base2 -> Area, Pref
という感じですね。
getName(), getData() は Base2 に記載してそれぞれ末端のクラスからは削除となります。

ただ、上記の内容であれば、Base クラスに同名のメソッドがないのであれば、Base クラスに書いてしまっても問題なさそうな気はしますね。
他の Base の子クラスにも当然影響がありますが、もともと定義されていないのであれば呼び出されることはないでしょうし。

Comments

[bws]

$dataの誤りでした、修正させていただきました。
ありがとうございます。クラスは複数継承出来ないと思っていたのですが、一つずつ継承すれば良いんですね。
現時点ではBaseに書いても問題ないのですが、関係ない処理をこのままBaseにどんどん書いてしまって大丈夫なんだろうかという不安がありましたがスッキリしました。ありがとうございます！