クラスの設計における継承やインターフェースの使い方(例あり)

Question

うまく継承やインターフェースを使いたいのですが、継承やインターフェースをどのように使うべきか、また使うべきか使わないべきなのかの判断に苦しんでいます

例えば以下のような例を考えます
あるサブスクサービスでは、ユーザーが以下のようなプランを結び、契約したプランに応じたサービスを受けられるとします。プランは複数種類契約可能です。

プランA ラーメンプラン
毎週月曜日の昼にラーメンを宅配します
ラーメンはお店の器に入れられて宅配されるので、食べ終わったらその器を返す必要があります
器をとりにきて欲しい場合はオプションで申し込むことができますが、別途お金がかかります

プランB 牛丼プラン
毎週月曜日の昼に牛丼を宅配します
紙の入れ物に入れて宅配されるので器を返す必要はありません

プランC 健康チェックプラン
毎月の第１月曜日に、健康のプロがユーザーと1時間面談する時間を設けてくれます
そこでユーザーは健康に関するいろいろなことを相談できます。
開始時間は設定することができます。

見ての通り、同じプランという括りであってもAとBは器回収の有無という違いがありますし、Cに関しては全くの別物という感じがします。

しかしこれを全部同じようにプランという括りで見たいこともあるはずです。例えば、あるユーザーがどんなプランを契約しているのか、というのを考えるときにUserオブジェクトがPlanオブジェクトの集まりを持っていて、それぞれ同じように参照できると便利そうです。

このアイディアを以下のような感じでコードで書きました

abstract class Plan{
  public string $planName;

  public function planName(){
    return $this->planName;
  }

  abstract public function totalPrice();
}


class ramenPlan extends Plan{
  public string $planName = 'ラーメンプラン';
  public int $number;
  public bool $collectFlg;//器回収フラグ

  public function __construct($number,$cllectFlg=false){
    //プロパティに代入
  }

  public funtion totalPrice(){
    //計算
  }
}

class beafBowlPlan extends Plan{
  public string $planName = '牛丼プラン';
  public int $number;

  public function __construct($number){
    //プロパティに代入
  }

  public funtion totalPrice(){
    //計算
  }
}

class healthConsultationPlan extends Plan{
  public string $planName = '健康チェックプラン';
  public string $startTime;// 開始時間

  public function __construct($startTime){
    //プロパティに代入
  }

  public funtion totalPrice(){
    //計算
  }

  public function nextSchedule(){
    // 次回は何日の何時から何時までかを返す
  }
}

  class user {
    public string $name;
    public array $plans;

    public function __construct($name,$plans){
      //プロパティに代入
    }

    public function nextHealthConsultationSchedule(){
      // plansの中を調べてhealthConsultationPlanがあればnextScheduleメソッドを使用
      // なければnullを返す
    }

  }

自分がこれを使いにくいと思っている点があります。
1つめは、どのPlan子クラスも持っているプロパティが異なり、またそれに伴ってコンストラクタが異なるという点です。
自分がネットで見る例だとほとんどが親クラスと子クラスでは持っているプロパティは同じだし、せいぜいが振る舞いの結果が異なるくらいです。
しかし自分の例では違います。これでは同じようにPlanクラスとして各Planを作成しようと思っていても、結局これらのPlanクラスを使用する側(例えばDatabaseからデータを抽出して各Planを作成するようなクラス)ではramen,beefBowl,healthConsultationの何を作るのかを何らかの方法を用いて判断しないといけません。なぜならどのPlanクラスを作成するかによって、コンストラクタに渡すものが異なるからです。

2つめに、プランCが、他のクラスが持っていないnextScheduleメソッドを持っている問題です。
これはUserのnextHealthConsultationScheduleメソッドで使用されます。ただプランCを契約していないUserにとって、nextHealthConsultationScheduleメソッドは全くいらないメソッドです。プランCを契約していないUserがこのメソッドを使用できるようなクラスの作り方でいいのでしょうか。何だか気持ち悪く感じます。

このような問題を考えるとき、どのようにクラスを設計すれば上記のような問題点を解決した上で、なおかつ冒頭で述べたように同じようにプランという括りで見ることができるでしょうか。
よければご意見お聞かせいただければと思います。

Comments

[m.ts10806]

言語によって方言や癖、仕様のちがいがあるのでPHPを例題にするのでしたら質問タグにも「PHP」があったほうが良いと思います。

[Zuishin]

プラン一つ一つにクラスを作るべきではなく、委譲を使うべきケースだと思います。

[nuu]

m.ts10806様
ご指摘ありがとうございます！PHPタグを追加しました。

Zuishin様
回答ありがとうございます！例えばどのような形で作るとよいでしょうか...

[Zuishin]

器を返す、商品を宅配する、面談すると言った各サービス・条件の個々をクラスにし、プランは名称とそれらの条件の集合を保持し、金額を算定するクラスにします。

[nuu]

Zuishin様
ありがとうございます！

>>器を返す、商品を宅配する、面談すると言った各サービス・条件の個々をクラスにし

これらをクラスにすることは思いつきませんでした、考えてみようと思います！

Answer 1

オブジェクト指向設計を学ぶといいですよ。SOLID原則, クリーンアーキテクチャ, デザインパターン など。
あと、ドメイン駆動設計 (DDD: Domain-Driven Design) も有効かと思います。

1つめは、どのPlan子クラスも持っているプロパティが異なり、またそれに伴ってコンストラクタが異なるという点です。
これでは同じようにPlanクラスとして各Planを作成しようと思っていても、結局これらのPlanクラスを使用する側(例えばDatabaseからデータを抽出して各Planを作成するようなクラス)ではramen,beefBowl,healthConsultationの何を作るのかを何らかの方法を用いて判断しないといけません。なぜならどのPlanクラスを作成するかによって、コンストラクタに渡すものが異なるからです。

Factoryパターンの出番ですね。

2つめに、プランCが、他のクラスが持っていないnextScheduleメソッドを持っている問題です。
これはUserのnextHealthConsultationScheduleメソッドで使用されます。ただプランCを契約していないUserにとって、nextHealthConsultationScheduleメソッドは全くいらないメソッドです。プランCを契約していないUserがこのメソッドを使用できるようなクラスの作り方でいいのでしょうか。何だか気持ち悪く感じます。

持っているか持っていないかに左右されないメソッドを作るといいです。
どのプランにもオプションがあり、オプション処理の一環として内部からnextScheduleを呼び出せばいいと思います。

Comments

[nuu]

回答ありがとうございます！
ファクトリーパターンは考えていないのですが、こんな感じでしょうか？
といってもかなり汚いし使いにくそうなので、多分全然できてないでしょうが、ここおかしいとかあればご指摘いただければ...

<?php

interface Plan
{

public function planName();

public function totalPrice();

public function option();
}

class ramenPlan implements Plan
{
public string $planName = 'ラーメンプラン';
public int $number;
public bool $collectFlg; //器回収フラグ

public function __construct($number, $cllectFlg = false)
{
//プロパティに代入
}

public function planName(){
return $this->planName;
}

public function totalPrice(){
//計算
}

public function option(){
return ['器回収'=>$this->collectFlg];
}
}

class beafBowlPlan implements Plan
{
public string $planName = '牛丼プラン';
public int $number;

public function __construct($number)
{
//プロパティに代入
}

public function planName(){
return $this->planName;
}

public function totalPrice(){
//計算
}

public function option(){
return [];
}
}

class healthConsultationPlan implements Plan
{
public string $planName = '健康チェックプラン';
public string $startTime; // 開始時間

public function __construct($startTime)
{
//プロパティに代入
}

public function planName(){
return $this->planName;
}

public function totalPrice(){
//計算
}

public function option(){
return ['次の時間'=>$this->nextSchedule()];
}

public function nextSchedule()
{
// 次回は何日の何時から何時までかを返す
}

}

また、ファクトリーパターンを使う際はファクトリーパターン用のクラスなりメソッドなりに
どのプランか、ラーメンの個数、器回収フラグ、牛丼の個数、健康チェック日時
などを全部渡して、ファクトリーパターンを行うクラス内で判断してクラスを作成するのでしょうか？
ファクトリーパターンも結局作成対象のコンストラクタが同じ例しか見たことがなく...

[shiracamus]

メニュー選択画面でメニューを選び、どのオプションを選ぶか決まっていくのではありませんか？
Optionクラスを作って、料金や選択情報を持たせるのがいいかと思います。オプション情報の変更はOptionインスタンスのコピーや入れ替えで対応することもできます。
メニューの種類や使い方はドメイン側が知っていて、生成方法はメニュー側が知っているので、生成と利用を分けて考えられるかと思います。

[shiracamus]

もしクリーンアーキテクチャやDDDについてご存知なければ、参考になることがあると思うので動画を紹介しておきます。
クリーンアーキテクチャ： https://www.youtube.com/watch?v=BvzjpAe3d4g
DDD： https://www.youtube.com/watch?v=19Gbx9jWLdc

[nuu]

ありがとうございます！
勉強させていただきます！