> но если тебе нужно получить инфу, а потом сделать

if на основании этой инфы
Что же криминального в таком ифе. Я вот например написал очередной класс-обработчик, который работает только для США (а проверяю ифом) - мне что, надо в сущность код дописывать (isUsa()) из-за этого?

7 ответов

15 просмотров

нарушаешь инкапсуляцию, повышаешь каплинг если вместо $person->isCitizenOfUsa() ты пишешь в сотне мест $person->getPassportCountry() === 'USA' и при этом через месяц окажется, что условие isUsa немного изменилось, то ты вместо одного метода поправишь 100 мест и при этом все клиенты должны знать точное значение константны 'USA'

Логачев Сергей- Автор вопроса
Шурик
нарушаешь инкапсуляцию, повышаешь каплинг если вм...

Если серьезно, то если я много где это делаю, то могу написать класс, который умеет это делать. И везде его использовать

Шурик
нарушаешь инкапсуляцию, повышаешь каплинг если вм...

А если стран 10, n? Не будет ли лучше isCountry(string) ну или vo/entity страны?

Павел Г.
А если стран 10, n? Не будет ли лучше isCountry(st...

а не пофиг ли на страну? почему сравниваешь со страной, а не проверяешь возможность? не $citizen->isCountryMatches($countries), а например $blabla->isDocumentRequired($citizen)

Павел Г.
А если стран 10, n? Не будет ли лучше isCountry(st...

ну суть в том, что знать страну - никому нахер не нужно в 90% случаев важно понимать применимо ли что-то вот к этому инстансу

Шурик
а не пофиг ли на страну? почему сравниваешь со стр...

Тут уже контекст задачи надо знать. IsUsa как то очень странно и узко, имхо конечно

Шурик
а не пофиг ли на страну? почему сравниваешь со стр...

Кстати вот, в коде внутри метода required все равно надо будет как то обратиться к citizen и как то понять что usa

Похожие вопросы

Обсуждают сегодня

Ребята, всем привет. Подскажите, пожалуйста, можно ли как-то через бота понять, что этого бота добавили в группу\канал и выдали ему права администратора?
Artem Stormageddon
9
Это переведённый текст с английского. Я не говорю на русском, но могу использовать переводчик Телеграм. Приветствую! Я начинающий веб-разработчик и все еще учусь. В настояще...
𐩱𐩪𐩣𐩱𐩲𐩺𐩡
3
А не хотим ли мы развлечься? 😉 Но так чтобы с пользой для наших профессиональных навыков?? 👨‍🎓👩‍🎓 Предлагаю на октябрь запланировать тестовый запуск новой командной игры "Игр...
Andrii Kurdiumov
2
Привет всем! Почему этот код не срабатывает при добавлении или удалении пользователя из чата? bot.on('chat_member', async (ctx) => { console.log(ctx); }) bot.launch({allo...
Alexander
5
у кого сколько оперативы на базе данных ?
АДИЛЬБЕК
4
Через бот апи возможно получить ID стикерпака? Не ссылку.
Vexylon [АФК до 09.09]
5
Привет Хочу сделать аналог iCloud’а для своих проектов, чтобы пользовательская информация хранилась в облаке, была доступна во всех сервисах, её можно было подсасывать везде)...
Виталий
9
В тг можно спарсить всех кто пишет в группе? Если список участников скрыт
S
3
код Event::listen('cms.page.display', function (&$content, $slug, $page, $html) { if (is_object($content)) { dump($content); } else { dump($s...
Point 111
3
Всем привет. Не понимаю, в чём тут шутка юмора. Убирается только разрешение на send_messages. А send_media_messages остаётся. Как сделать, чтобы оба убирались? await b...
Alexander
2
Карта сайта