В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

symfony

2345 membersпожаловаться на группу
2021 April 17

Ш

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

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

C

CvekCoder in symfony
Ооо, и тут напрашивается трейт)
источник
01:59пожаловаться#2

C

CvekCoder in symfony
Если серьезно, то если я много где это делаю, то могу написать класс, который умеет это делать. И везде его использовать
источник
02:00пожаловаться#3

ПГ

Павел Г. in symfony
А если стран 10, n? Не будет ли лучше isCountry(string) ну или vo/entity страны?
источник
02:00пожаловаться#4

C

CvekCoder in symfony
Но в исходную сущность со своим isUsa не лезть, потому что это частность, которая самой сущности не упала
источник
02:01пожаловаться#5

C

CvekCoder in symfony
Потому как open-close
источник
02:02пожаловаться#6

АЯ

Андрей Ява in symfony
Не надо оверхедить. Оверхед это тоже зло
источник
02:02пожаловаться#7

Ш

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

ПГ

Павел Г. in symfony
Тут самый главный смысл - не спрашивай данные, спрашивай результат /говори что делать
источник
02:02пожаловаться#9

C

CvekCoder in symfony
Есть ли добро в программировании?
источник
02:02пожаловаться#10

C

CvekCoder in symfony
Вот с анемичными моделями говорить - не особо получается
источник
02:02пожаловаться#11

Ш

Шурик in symfony
ну суть в том, что знать страну - никому нахер не нужно в 90% случаев
важно понимать применимо ли что-то вот к этому инстансу
источник
02:03пожаловаться#12

ПГ

Павел Г. in symfony
Тут уже контекст задачи надо знать. IsUsa как то очень странно и узко, имхо конечно
источник
02:04пожаловаться#13

C

CvekCoder in symfony
Бизнес-задача про страну из вашего адреса может появиться сильно позже, и менять интерфейс сущности из-за этого - путь к багам. Обычно это лучше реализовать в том классе, который это новое бизнес-требование реализует. И да, там где-то будет иф
источник
02:06пожаловаться#14

ПГ

Павел Г. in symfony
Кстати вот, в коде внутри метода required все равно надо будет как то обратиться к citizen и как то понять что usa
источник
02:14пожаловаться#15

C

CvekCoder in symfony
Ага, о том и речь. Как там без ифа вокруг гетера. Как глубоко его ни спрячь, но он будет
источник
02:16пожаловаться#16

ПГ

Павел Г. in symfony
If можно спрятать уже внутри сущности которая обладает данными.
источник
02:17пожаловаться#17

C

CvekCoder in symfony
Это про метод isUsa
источник
02:18пожаловаться#18

C

CvekCoder in symfony
Что не ок
источник
02:18пожаловаться#19

ПГ

Павел Г. in symfony
IsCountryMatch например
источник
02:18пожаловаться#20