Как выбирать действие в зависимости от типа объекта?

Question

[vVega]

В проекте есть класс с единственным статическим методом, который получает из статической фабрики объект и запускает нужное действие в зависимости от переданного статуса.

class Executor
{
    public static function runActionByStatus(int $id, string $status): void
    {
        $object = Factory::create($id);

        if ($status === 'create') {
            $object->create();
        }

        if ($status === 'cancel') {
            $object->cancel();
        }

        if ($status === 'commit') {
            $object->commit();
        }
    }
}

До текущего момента фабрика всегда создавала объекты одного типа. Теперь объекты, которые создает фабрика имеют подтипы. Требуется разный алгоритм выбора действия в зависимости от подтипа созданного фабрикой объекта (статусы разные для каждого подтипа).

Подскажите варианты рефакторинга.

Comments

[Дмитрий]

на какой то велосипед похоже

[Дмитрий]

если так нужна статика, то можно с этим пошаманить https://www.php.net/manual/ru/language.oop5.overlo...

[vVega]

Дмитрий, нужен вариант рефакторинга. Думаю, что статика точно не подходит. Возможно паттерн стратегия подойдёт...

[vVega]

vilinyh, не понял мысль. Вы по сути просто изменили сигнатуру текущей реализации.

[Дмитрий]

vVega, можно запилить виртуальные методы, в самом __call разбирать имя метода и выбирать стратегию

Answer 1

[Дмитрий Дерепко]

> Теперь объекты, которые создает фабрика имеют подтипы. Требуется разный алгоритм выбора действия в зависимости от подтипа созданного фабрикой объекта

Значит фабрика у вас не должна ничего решать сама. Разные типы + разные действия = разные классы, фабрики.

1. Может проще создать парочку отдельные фабрик для этих подклассов и в нужном месте вызывать нужную, не ожидая, что какая-то стратегия правильно сматчит вам фабрику?
2. Если у вас еще одна фабрика появилась, то это не значит, что здесь супер узкое бутылочное горлышко и его нужно рефакторить. Тем более, если пока сами не знаете как лучше это сделать. Сделайте пару развилок через if. Когда появится больше запросов к этому коду, тогда и новое поведение подскажет, как всё-таки он должен выглядеть.
3. Если всё-таки хотите как-то отрефакторить сейчас, то вариант от vilinyh подойдет на первое время. А возможно и на всегда.

$object = Factory::create($id);
$action = ActionFactory::create($status, $object);
$action->run();

Только для ясности я бы его переписал так:

$factory = Factory::create($id);
$result = $factory->run($status);

Немного пояснений:
Factory::create() – отдаст вам фабрику по объекту. Там сделаете пару if на проверку типа.
$factory->run() – выполнит нужные действия уже в фабрике нужного класса.

4. А еще можно сделать методы не статическими, обернуть интерфейсам, инжектить через контейнер, сделать промежуточный AggregateFactory, который будет делать if $subFactory->supports() цикле по всем фабрикам, которые реализуют уже логику, умно отлавливать ошибки, логировать результаты и всё в этом духе. Почти стратегию и реализовали таким способом. Но вот только нужно оно вам сейчас? :)

Comments

[vVega]

Можете привести сигнатуру $subFactory->supports()?

[Дмитрий Дерепко]

class AuthorFactory {
  public function supports(object $entity): bool
  {
    return get_class($entity) === Author::class;
  }

  // ...
}

И интерфейс добавить для работы в основной "фабрике" с методами supports($entity): bool и run().

Answer 2

[Сергей delphinpro]

Какая нафиг стратегия?
Создать, отменить и зафиксировать – это не три разные стратегии! Это три разных действия. Вам маршрутизатор нужен, вероятно.

Comments

[vVega]

Да, вы правы, что приведенный код является маршрутизатором (в этом контексте скорее resolver). Однако вопрос был в том, как выбрать разный алгоритм выбора действия в зависимости от подтипа созданного фабрикой объекта. Решил таким образом. Не уверен, что "стратегия" здесь подходит.

class Resolver
{
    private Foo $foo;

    public function __construct(Foo $foo)
    {
        $this->foo = $foo;
    }

    public function resolve(string $status): void
    {
        if ($this->getResolvingStrategy()->needsCreate($status)) {
            $this->foo->create();
        } elseif ($this->getResolvingStrategy()->needsCancel($status)) {
            $this->foo->cancel();
        } elseif ($this->getResolvingStrategy()->needsCommit($status)) {
            $this->foo->commit();
        }
    }

    private function getResolvingStrategy(): ResolvingStrategy
    {
        if ($this->foo instanceof Bar) {
            return new BarResolvingStrategy();
        }

        if ($this->foo instanceof Baz) {
            return new BazResolvingStrategy();
        }

        throw new Exception();
    }
}

class BarResolvingStrategy implements ResolvingStrategy
{
    public function needsCreate(string $status): bool
    {
        return $status === 'A';
    }

    public function needsCancel(string $status): bool
    {
        return $status === 'B';
    }

    public function needsCommit(string $status): bool
    {
        return $status === 'C';
    }
}

class BazResolvingStrategy implements ResolvingStrategy
{
    public function needsCreate(string $status): bool
    {
        return $status === 'D';
    }

    public function needsCancel(string $status): bool
    {
        return $status === 'E';
    }

    public function needsCommit(string $status): bool
    {
        return $status === 'F';
    }
}

interface ResolvingStrategy
{
    public function needsCreate(string $status): bool;
    public function needsCancel(string $status): bool;
    public function needsCommit(string $status): bool;
}

Answer 3

[Антон Шаманов]

паттерн Strategy

Answer 4

[Dark_Dante]

Более того, if тут немного бессмысленен - статус то всегда один, т.е. отработает только один блок if. Я бы написал это через switch. Ну или хотя бы elseif