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

групп:

каналов:

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

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

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

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

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

go.backupland.com

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

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

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

Size: a a a

Software Design/Architecture/Zen

1885 membersпожаловаться на группу
2020 December 28

КГ

Константин Грачев... in Software Design/Architecture/Zen
У меня похоже кейсы не те, ибо что в случае с смс что с оплатой, я записываю лишь информацию о попытке запроса успешно/неуспешно.
Об успешности доставки/оплаты в этих кейсах от внешней системы ещё дополнительный callback обычно есть
источник
15:42пожаловаться#1

k

knopkod4v in Software Design/Architecture/Zen
Константин Грачев
Конкретно тут хотелось бы вместо 44 строки сразу получить сущность аргументом, а последние 3 строки убрать в метод переданной сущности. Сразу минус зависимость
по идее сущность для чтения получать как-то не оч (т.к. геттеры будут)
К тому же это интеграция с внешней системой - тебе, вполне вероятно, может понадобиться не только информация от того объекта, где родился ивент. Т.е. в бд всё равно сходить надо будет.
источник
15:45пожаловаться#2

КГ

Константин Грачев... in Software Design/Architecture/Zen
knopkod4v
по идее сущность для чтения получать как-то не оч (т.к. геттеры будут)
К тому же это интеграция с внешней системой - тебе, вполне вероятно, может понадобиться не только информация от того объекта, где родился ивент. Т.е. в бд всё равно сходить надо будет.
Можно вьюху сделать
источник
15:46пожаловаться#3

КГ

Константин Грачев... in Software Design/Architecture/Zen
Ну и тут не только чтение же
источник
15:46пожаловаться#4

k

knopkod4v in Software Design/Architecture/Zen
Константин Грачев
Ну и тут не только чтение же
для Sms только чтение
источник
15:47пожаловаться#5

DT

Dmitriy Tkachenko in Software Design/Architecture/Zen
knopkod4v
по идее сущность для чтения получать как-то не оч (т.к. геттеры будут)
К тому же это интеграция с внешней системой - тебе, вполне вероятно, может понадобиться не только информация от того объекта, где родился ивент. Т.е. в бд всё равно сходить надо будет.
Главное чтобы геттеры ссылки не ввплевывали, через которые можно поменять стейт. Тогда ничего страшного в этом нет
источник
15:47пожаловаться#6

КГ

Константин Грачев... in Software Design/Architecture/Zen
knopkod4v
для Sms только чтение
Я же написал, что 3 последние строчки хотелось бы убрать в метод Sms
источник
15:48пожаловаться#7

DT

Dmitriy Tkachenko in Software Design/Architecture/Zen
И главное чтобы геттеры были на данные, а не на структуру внутреннюю
источник
15:48пожаловаться#8

КГ

Константин Грачев... in Software Design/Architecture/Zen
Sms::send() -> $this->sends[] = new SmsSend(...)
источник
15:48пожаловаться#9

DT

Dmitriy Tkachenko in Software Design/Architecture/Zen
Тогда и тесты более стойкие, и те кто читает более защищены от случайных изменений
источник
15:49пожаловаться#10

DT

Dmitriy Tkachenko in Software Design/Architecture/Zen
Ну тоесть агрегаты которые только испускают ивенты и реагируют на команды- идеальные штуки для юнит тестов
источник
15:50пожаловаться#11

k

knopkod4v in Software Design/Architecture/Zen
Константин Грачев
Я же написал, что 3 последние строчки хотелось бы убрать в метод Sms
а нафига оно в Sms? Какие инварианты объекта Sms надо соблюдать, чтобы записать SmsSend или какие инварианты в Sms зависят от SmsSend? Выглядит как сеттер. Ему бы жить где-то отдельно
источник
15:50пожаловаться#12

DT

Dmitriy Tkachenko in Software Design/Architecture/Zen
А как тестировать без геттеров?
источник
15:51пожаловаться#13

КГ

Константин Грачев... in Software Design/Architecture/Zen
knopkod4v
а нафига оно в Sms? Какие инварианты объекта Sms надо соблюдать, чтобы записать SmsSend или какие инварианты в Sms зависят от SmsSend? Выглядит как сеттер. Ему бы жить где-то отдельно
Никакие, но позволяет спрятать инфраструктуру $em->persist
источник
15:51пожаловаться#14

DT

Dmitriy Tkachenko in Software Design/Architecture/Zen
Лазить внутрь через рефлексию? Тогда тесты слишком хрупкие
источник
15:51пожаловаться#15

ЕР

Евгений Ромашкан... in Software Design/Architecture/Zen
Dmitriy Tkachenko
Лазить внутрь через рефлексию? Тогда тесты слишком хрупкие
Норм)
источник
15:52пожаловаться#16

k

knopkod4v in Software Design/Architecture/Zen
Dmitriy Tkachenko
А как тестировать без геттеров?
вот как раз чтобы люди не лезли тестить с геттерами - геттеров лучше и избегать.
проверять поведение объекта, а него стейт (проверка через геттеры - та же самая проверка стейта)
источник
15:52пожаловаться#17

DT

Dmitriy Tkachenko in Software Design/Architecture/Zen
knopkod4v
вот как раз чтобы люди не лезли тестить с геттерами - геттеров лучше и избегать.
проверять поведение объекта, а него стейт (проверка через геттеры - та же самая проверка стейта)
А как тытпротестируешь поведение? Например установку каких то параметров? Если это единственная реакция
источник
15:53пожаловаться#18

DT

Dmitriy Tkachenko in Software Design/Architecture/Zen
На команду
источник
15:53пожаловаться#19

DT

Dmitriy Tkachenko in Software Design/Architecture/Zen
Евгений Ромашкан
Норм)
Этоточно не про тестирование поведения) а реализации
источник
15:53пожаловаться#20