Телеграмм чат группы prophp7 страница 10359

В дто данные о разных вещах. Плюс доп данные нужны только одному.

10:41пожаловаться #1

BIV in PHP

Есть несколько классов для преобразования данных реализующих один интерфейс.
В интерфейсе метод

execute(array $data, DTO1 $dto1)

.
Одному из классов понадобилась еще информация с DTO2, другим она не нужна.
Как лучше поступить в данной ситуации? Пока что делаю

setDto2

перед

execute

Может сделать интерфейс DTO и использовать его в методе execute? Тогда не будешь зависеть от реализации.

10:43пожаловаться #2

BIV

Может сделать интерфейс DTO и использовать его в методе execute? Тогда не будешь зависеть от реализации.

мне нужно не DTO2 вместо DTO1, а они оба

10:43пожаловаться #3

мне нужно не DTO2 вместо DTO1, а они оба

nullable параметр

10:44пожаловаться #4

Руслан in PHP

мне нужно не DTO2 вместо DTO1, а они оба

В смысле "оба"? Ты же передаешь только один

10:44пожаловаться #5

ну или да, композицию пили

10:44пожаловаться #6

приблизительно равные по отстойности варианты

10:45пожаловаться #7

Bohdan Turchyk

nullable параметр

думал о таком, но тоже показалось не очень

10:46пожаловаться #8

Руслан

В смысле "оба"? Ты же передаешь только один

передаю один дто1, но в однох случае мне надо и дто2, пока что просто делаю сетДто2 перед самим вызовом

10:47пожаловаться #9

Maxim Kainov in PHP

думал о таком, но тоже показалось не очень

Это лучше чем сетдто2

10:48пожаловаться #10

передаю один дто1, но в однох случае мне надо и дто2, пока что просто делаю сетДто2 перед самим вызовом

в итоге у тебя никакой уверенности нет

10:48пожаловаться #11

BIV in PHP

Есть несколько классов для преобразования данных реализующих один интерфейс.
В интерфейсе метод

execute(array $data, DTO1 $dto1)

setDto2

перед

execute

Если не хочется делать необязательный параметр, можно запилить декоратор интерфейса и класса с методом execute, которые будут расширять

execute(array $data, DTO1 $dto1)

, принимать 2 параметра dto. А в классе, в котором нужно DTO2 использовать уже этот декоратор.🤔

10:48пожаловаться #12

Руслан in PHP

передаю один дто1, но в однох случае мне надо и дто2, пока что просто делаю сетДто2 перед самим вызовом

в одном случае передавай дто1, во втором гибрид дто1 + дто2

10:50пожаловаться #13

Maxim Kainov in PHP

Руслан

в одном случае передавай дто1, во втором гибрид дто1 + дто2

То есть, должно быть дто3 объединяющее два других?

10:53пожаловаться #14

Руслан in PHP

Maxim Kainov

То есть, должно быть дто3 объединяющее два других?

Почему нет?

10:53пожаловаться #15

Maxim Kainov in PHP

Руслан

Почему нет?

Да, самый верный вариант, похоже.

10:54пожаловаться #16

?DTO2 мне больше подходит, не прийдеться доставать лишнюю информацию, где она не используется

11:00пожаловаться #17

Sergey Protko in PHP

Лучше разные dto

11:07пожаловаться #18

Sergey Protko in PHP

"реюз" кода там только усложняет поддержание обратной совместимости

11:08пожаловаться #19

Sergey Protko in PHP

Если клиенты разные то это вообще опасно