クラスから送られてきた値によって、異なるクラスを返したいのですが…

Question

前提・実現したいこと

システム設計の勉強の為、タクシーを題材にプログラミングしています。

利用者(User)タクシーを呼ぶ→タクシー組織(Organization)最適なタクシーを派遣→タクシー(taxi)利用者を運び、料金請求→利用者支払い

Userクラス:目的地とタクシーの好み（初期値は早い順）
Organizationクラス:Userクラスから送られてきた値によって、最適なTaxクラスを返します。
Taxiクラス:速い、安いなど様々に分岐しています。

Organizationクラスが引数を元に条件分岐で〇〇Taxiクラスを返します。つまりこれはTaxiの種類が増えるごとにどんどん肥大化してしまう。という事になります。
この問題を解決するにはどうしたらいいのでしょうか?

該当のソースコード

<?php
namespace Lib;

class Organization
{
    // システムの成長に従い、どんどん肥大化してしまう
    public function dispatchTaxi($criteria = 'first')
    {
        if ($criteria === 'first') {
            return new Taxi\FirstTaxi();
        }
    }

}

Comments

[takasima20]

それが本当に必要なら、そうするしかないのでは? 特徴を保持するインスタンスを用意するでもいけそうな気もしますが、仕様が分からないことにはなんとも…

[退会済みユーザー]

そもそも条件分岐させる為に引数を受け取っているのが失敗だった感がありますね。
皆さんありがとうございます。

Answer 1

想定がよくわかりませんが、クラスの管理でやるよりRDBなどで
データ管理するような案件だと思います

Comments

[退会済みユーザー]

確かにそうかもしれません。ですが出来ればクラスで管理したいです。
タクシーではなく別の何かにするべきですかね？
引数によって様々な処理をする必要がある時、どうしたら肥大化しないのか。そのような内容で質問するべきだったかもしれません。

[yambejp]

どうしてもクラスでなんとかしたいなら、
それぞれのクラスを例示すべきです。
いまのサンプルですと概要程度しか理解できません

Answer 2

そのクラスの肥大化防ぐだけなら

namespace Lib;

class Organization
{
    public function dispatchTaxi($criteria = 'first')
    {
    $criteriaClass = 'Taxi\' . ucfirst($criteria) . 'Taxi';
        if class_exists($criteriaClass)) {
            new $criteriaClass();
        } else {
             return new Taxi\FirstTaxi();
        }
    }

}

こんなかんじやろな。

Answer 3

ざっくりこんな感じじゃない？

class User 
{
    public function callTaxi(Organization $organization)
    {
        $taxi = $organization->searchTaxi();
        return $taxi;
    }
}

class Organization
{

    private $taxies;

    public function __construct()
    {
        // 自社のタクシーを定義
        $this->taxies = [
            new Taxi(),
            new Taxi(),
            new Taxi(),
            new Taxi()
        ];
    }

    public function searchTaxi()
    {
        // 自社のタクシーから一台を選択する
        $taxi = func($this->taxies); 
        return $taxi;
    }
}

Comments

[退会済みユーザー]

ノーリアクションなのは理解できなかったパターン？