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

Question

RMate @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:
Думал, что мне идеально подойдет паттерн "фабрика", но чем дальше я про него читаю, тем меньше понимаю, нужен ли он мне.

Вопрос задан более трёх лет назад
167 просмотров

6 комментариев

Подписаться 2 Простой 6 комментариев

Антон Р. @anton_reut
Непонятно зачем делать перебор если у тебя есть ОБЩИЙ для всех метод 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 @RMate Автор вопроса

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

Написано более трёх лет назад

kafkiansky @mad_maximus

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 @RMate Автор вопроса

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

Написано более трёх лет назад
Антон Р. @anton_reut

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

Написано более трёх лет назад
kafkiansky @mad_maximus

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

Написано более трёх лет назад

Непонятно зачем делать перебор если у тебя есть ОБЩИЙ для всех метод 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, не нужен
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; } }
kafkiansky, Вау! Отлично, спасибо большое! Можете перенести в решение))
kafkiansky, а к какому паттерну в итоге такая конструкция относится?
Антон Р., вообще, поведенческий, а какой именно - точно не знаю. Немного похож на Посетитель, но это не в точной степени он.

Answer 1 · 2019-11-01 16:42:55

Создаете 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();
    }
}

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

Войдите на сайт