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

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

7 ответов

17 просмотров

нарушаешь инкапсуляцию, повышаешь каплинг если вместо $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

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

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

Добрый вечер, Пока не совсем понимаю как наладить общение между телеграм ботом и ПО для работы с сим боксом. По самому боту так понял: - Нужен некий баланс, который можно поп...
Magic
6
сделал сайт, прикрутил в боте сайт, и виджет логина. как автоматически логинить пользователя в аккаунт(телеграм), при входе с бота?
Александра Чернивецкая
5
Объясните, пожалуйста, почему компилятор ругается на использование в условии неинициализированной переменной: int x; Task.Run(async () => { x = await somefunc(); }).Wait...
Александр
5
Ребят, подскажите, пожалуйста, почему в префиксе к ассетам, которые генерируются через фильтр | theme в шаблоне, стал вдруг появляться index.php? Вот так выглядит ссылка на а...
Виталий
1
Всем привет. Ребята, подскажите, пожалуйста. у ботов есть ограничение на отправку сообщений - 30 сообщений в секунду, эти ограничения накладываются на все сообщения? или на со...
Artem Stormageddon
4
Блин, ребята, сори за тупые вопросы. А можно ли как-то открыть вебапку по нажатию на кнопку в меню(которое появляется слева, команды)?
Artem Stormageddon
3
а плаксы из-под питона умеют только в комфортных условиях что-то выдавить из себя?)
Lencore
9
Но, может, есть уже проверенная? Наши требования такие: 1. Сообщения должны приходить из Инста в CRM оду 2. Должна быть возможность подключить несколько экаунтов Инстаграм. Р...
Alexander Sharoiko MSE / Александр Шаройко
13
Это может быть все-таки не флудвейт? у меня ботфазер принимает изменения и отображает даже что они изменились, на видео видно что он прислал якобы уже измененное описание, н...
OVERLINK
13
Коллеги, может знает кто, можно ли цвет бейджа счётчика в BackendMenu менять без бубнов?
Alex Blaze
3
Карта сайта