Всем привет! как лучше реализовать вот что Есть несколько кошельков с

Question

Всем привет! как лучше реализовать вот что Есть несколько кошельков с

разными валютами USD, EUR, RUB

Допустим нужно добавить новую валюту UAG - как организовать архитектуру таким образом чтобы не прищлось переписывать бизнесс логику?

Сейчас делаю вот так:
public function getWalletType(int $clientId, string $currency): Wallet
{
if ($currency === EURWallet::WALLET_CURRENCY) {
$wallet = new EURWallet($clientId, new \DateTimeImmutable());
} elseif ($currency === USDWallet::WALLET_CURRENCY) {
$wallet = new USDWallet($clientId, new \DateTimeImmutable());
} elseif ($currency === JPYWallet::WALLET_CURRENCY) {
$wallet = new JPYWallet($clientId, new \DateTimeImmutable());
}

if (!isset($wallet)) {
throw new NotCorrectCurrencyTypeException('Not correct currency type');
}

return $wallet;
}

Если добавится новая валюта придется тут еще один блок в IF городить

как лучше сделать? все новые валютные кошельки наследуются от общего родителя Wallet

вообщем нужно не нарушать принцип Open-Closed из Солида

#backend #php #programming #russian

0

07.02.2021

8 ответов

14 просмотров

A S Автор вопроса

The Ant 🐜
паттер стратегия тебе поможет

пошел пытаться применять стратегию

0

07.02.2021

A S Автор вопроса

The Ant 🐜
паттер стратегия тебе поможет

так у меня уже стратегия по сути реализована в коде

0

07.02.2021

The Ant 🐜

A S
так у меня уже стратегия по сути реализована в код...

м.б. весь код не видно же

0

07.02.2021

A S Автор вопроса

The Ant 🐜
м.б. весь код не видно же

попробую описать более подробно Вот метод который отдает нужный кошелек в зависимости от валюты public function getWalletType(int $clientId, string $currency): Wallet Внури тупо if ($currency === EURWallet::WALLET_CURRENCY) { $wallet = new EURWallet($clientId, new \DateTimeImmutable()); } elseif ... Получается уже стратегия, т.к дальше везде можно работать с кошельком, не важно какая валюта Но мне нужно именно здесь успростить или сделать согласно принципу открытости акртыост добавление новой валюты

0

07.02.2021

🦔

Уберите код на пастбин или гист

0

07.02.2021

First Last

A S
попробую описать более подробно Вот метод который...

Это невозможно

0

07.02.2021

Roman