ДБ
$serializer->deserialize($data, MyDTO::class);Что это за трешак, простите?)
Size: a a a
2021 January 13
AB
если всего два поля, то конструктор ок
AB
Что это за трешак, простите?)
why?
ES
Интересен кейс когда их 10
AB
Интересен кейс когда их 10
тут вопрос, а в каком случае их может быть 10? скорее всего это форма, это приходит в json и можно исползьовать сериалайзер
ДБ
Меня смущает передача имени класса в виде строки
AB
Меня смущает передача имени класса в виде строки
константа же, рефакторинку поддается. это же не
’\App\MyDTO’ДБ
Слабоватый контракт, как по мне
ES
Ребята, всем спасибо за ответы.
Очень полезный опыт. 👍
Очень полезный опыт. 👍
AB
Слабоватый контракт, как по мне
а в чем проблемы могут быть?
ДБ
а в чем проблемы могут быть?
Проблемы могут быть в чём угодно, когда к коду подходят джуны
AM
Проблемы могут быть в чём угодно, когда к коду подходят джуны
😂
ДБ
Так что я бы обезопасился от имён классов в строках
AM
DTO просто разные бывают.
AM
Бывает что они не пересекают границ абстракций. Между внутренностями всякое тусуют. Тогда можно во все тяжкие.
AM
А бывает что они — часть контракта. Тогда лучше задуматься.
AB
Так что я бы обезопасился от имён классов в строках
psalm в помощь, там есть https://psalm.dev/docs/annotating_code/type_syntax/scalar_types/#class-string-interface-string
ДБ
psalm в помощь, там есть https://psalm.dev/docs/annotating_code/type_syntax/scalar_types/#class-string-interface-string
что делать, если не юзаю аннотации?
AB
в плане псалм, а не аннотации? 🙂
AB
ну я бы побоялся джунов в таком случае вообще к проекту допускать, если нет способа за ними следить