Как правильно реализовать создание дочерних экземпляров класса?

Question

[RMate]

Всем привет!
Имеется что-то вроде автоответчика, в скрипт могут поступать вопросы, допустим, с почты, по смс, из соцсети.
Задача алгоритма подобрать ответ, и ответить туда, откуда пришёл запрос.
Для чего был сделан интерфейс
IAnswerProvider, который требует всех реализующих его потомков реализовать метод sendAnswer()
Имеются наследующие его классы
SmsAnswerProvider
EmailAnswerProvider
PhoneAnswerProvider

И всё работает хорошо, пока не возникает самый интересный вопрос.
Я хочу отправить уведомление всем, кто когда-либо взаимодействовал со мной... для чего я выбираю из базы всех пользователей (в базе содержится информация о источнике) и начинаю рассылку.
в реализации я вынужден делать что-то вроде

switch(источник)
    case "sms": 
        answerProvider = new SmsAnswerProvider();
        answerProvider.sendAnswer("Hello world")!
        break;
    case "email": 
        answerProvider = new EmailAnswerProvider();
        answerProvider.sendAnswer("Hello world")!
        break;

А если источников будет 30? Я делаю что-то не так. Подскажите, пожалуйста, как правильно реализовать в данном случае мою идею. Спасибо.

PS:
Думал, что мне идеально подойдет паттерн "фабрика", но чем дальше я про него читаю, тем меньше понимаю, нужен ли он мне.

Comments

[Антон Р.]

Непонятно зачем делать перебор если у тебя есть ОБЩИЙ для всех метод sendAnswer() ?
Ты по сути просто создаешь объект реализующий нужный тебе интерфейс, а что это за объект там внутри вообще пофиг по сути, и как он работает тебе не важно.

<?php

class Sender {
	
	public $provider;
	
	public function __construct(Sendable $provider){
		$this->provider = $provider;
	}
	public function send(){
		$this->provider->sendAnswer();
	}
}


$provider = new SmsAnswerProvider();
$sender = new Sender($provider);
$sender->send();

[RMate]

Антон Р., я выгружаю информацию из базы, и все, что у меня есть, допустим, фио и откуда обратился человек. Дочернего объекта ещё нет. И чтобы отправить ответ - мне нужно его создать. Чтобы узнать, какой объект создать и нужен свич. В чем вся и беда)

[kafkiansky]

RMate, не нужен

interface SenderInterface
{
     public function sendAnswer(string $message): void;

     public function supports(string $type): bool;
}

class EmailSender implements SenderInterface
{
    public function sendAnswer(string $message): void
    {
           // тут отправляете сообщение
    }
   
     public function supports(string $type): bool
     {
           return 'email' == $type;
     }
}

class SmsSender implements SenderInterface
{
     public function sendAnswer(string $message): void
    {
           // тут отправляете сообщение
    }

     public function supports(string $type): bool
     {
           return 'sms' == $type;
     }
}

class SenderAggregate
{
     private $providers;
     
     public function __construct(SenderInterface ...$providers)
     {
           $this->providers = $providers
     }

    public function send(string $type, string $message)
    {
           foreach ($this->providers as $provider) {
              if ($provider->supports($type) {
                  $provider->sendAnswer($message);
              }
           }

            throw Exception;
    }
}

[RMate]

kafkiansky, Вау! Отлично, спасибо большое! Можете перенести в решение))

[Антон Р.]

kafkiansky, а к какому паттерну в итоге такая конструкция относится?

[kafkiansky]

Антон Р., вообще, поведенческий, а какой именно - точно не знаю. Немного похож на Посетитель, но это не в точной степени он.

Answer 1

[kafkiansky]

interface SenderInterface
{
     public function sendAnswer(string $message): void;

     public function supports(string $type): bool;
}

class EmailSender implements SenderInterface
{
    public function sendAnswer(string $message): void
    {
           // тут отправляете сообщение
    }
   
     public function supports(string $type): bool
     {
           return 'email' === $type;
     }
}

class SmsSender implements SenderInterface
{
     public function sendAnswer(string $message): void
    {
           // тут отправляете сообщение
    }

     public function supports(string $type): bool
     {
           return 'sms' === $type;
     }
}

class SenderAggregate
{
     private $providers;
     
     public function __construct(SenderInterface ...$providers)
     {
           $this->providers = $providers
     }

    public function send(string $type, string $message)
    {
           foreach ($this->providers as $provider) {
              if ($provider->supports($type) {
                  return $provider->sendAnswer($message);
              }
           }

            throw Exception;
    }
}

Answer 2

[Евгений Ромашкан]

Как то так

class UserData {
    /**
     * @readonly
     * @var string
     */
    public $source;

    public function __construct(string $source)
    {
        $this->source = $source;
    }
}

class NoSupportingSenderFound extends \Exception {}
class AnswerTransportError extends \Exception {}

interface AnswerSender {
    public function supportSource(string $source): bool;

    /**
     * @throws AnswerTransportError
     */
    public function sendAnswer(string $answer): void;
}

class AnswerSenderFacade
{
    /**
     * @var AnswerSender[]
     */
    private $senders = [];

    public function __construct(AnswerSender ... $answerSenders)
    {
        $this->senders = $answerSenders;
    }

    /**
     * @throws AnswerTransportError
     * @throws NoSupportingSenderFound
     */
    public function sendAnswer(UserData $userData, string $answer)
    {
        foreach ($this->senders as $sender) {
            if($sender->supportSource($userData->source)) {
                $sender->sendAnswer($answer);
                return;
            }
        }
        throw new NoSupportingSenderFound("...");
    }
}

В конструктор класса AnswerSenderFacade нужно передать инстансы всех реализаций интерфейса AnswerSender. В Symfony это просто делается через tagged services (Навесить тег на _instanceof AnswerSender через конфиг и в конфиге же указать что все помеченные тегом классы нужно заинжектить)

Над названием конечно можно ещё подумать.

p.s. в случае Симфони, правда, в конструкторе вместо ...$providers придётся использовать iterable и дополнительно проверять что пришли инстансы SenderInterface через instanceof

Comments

[Антон Р.]

То есть фасад сам внутри себя уже решает какой класс-отправитель использовать?

[Евгений Ромашкан]

Антон Р., Да. Над названием конечно можно ещё подумать.

[RMate]

Спасибо!

[kafkiansky]

Вообще, необязательно через теги, можно и так:

App\Service\SenderInterface:
        arguments:
            - '@App\Service\SmsSender'
            - '@App\Service\EmailSender'

Но это уже к слову о симфони.

Answer 3

[Alex Wells]

Создаете Registry, где храните RegistryEntry. Для каждого типа провайдера по соответствующей реализации RegistryEntry.

pseudo:

class Registry<T extends RegistryEntry> {
    private Map<string, T> registries = new HashMap();

    public void register(T entry) {}
    public T find(string registryName) {}
}

class RegistryEntry {
    public string registryName;
}

class AnswerProviderRegistry<AnswerProviderRegistryEntry> extends Registry {}
class AnswerProviderRegistryEntry<T extends IAnswerProvider> extends RegistryEntry {
    abstract public T create();
}

class SmsAnswerProviderRegistryEntry<SmsAnswerProvider> extends AnswerProviderRegistryEntry {
    constructor() {
        this.registryName = "sms";
    }

    public SmsAnswerProvider create() {
        return new SmsAnswerProvider();
    }
}

Comments

[Евгений Ромашкан]

RMate , Кстати, приведённый вариант с дженериками лучше костыля с supports(). Если вопрос безотносительно конкретного ЯП, по возможности лучше такой вариант рассмотреть.

[Alex Wells]

Евгений Ромашкан, такой вариант будет работать и без дженериков. Без подсветки в ide, конечно, но что уж поделать

[Евгений Ромашкан]

Alex Wells, Хм. Действительно, что-то я совсем не подумал. supports() тут не замена дженерикам, чуть разные области применения.

Дженерики могут лишь помочь с типизацией когда различаются типы аргументов в реализациях интерфейса (Event handler`ы там, RequestHandler`ы для разных объектов Request, нормалайзеры для объектов etc.).