search

Mercur1y / con_calculator

0 stars 0 forks source link

Айдишники в DTO #9

Closed Prifiz closed 2 years ago

Prifiz commented 2 years ago

https://github.com/Mercur1y/con_calculator/blob/53786ff497ef77adb7dc5c3f9d116b5fbdf54ac8/src/main/java/com/simbirsoft/con_calc/dto/CustomerDto.java#L14

Айдишник тебе в DTO точно нужен? Во всех случаях? DTO надо проектировать исходя из того, где оно нужно. DTO не должен быть универсальным и переиспользуемым. Допустим, есть запрос на создание некого объекта. Тут id не нужен, т.к. он сгенерится при создании сущности (sequence и всё такое в entity) Если же у тебя есть запрос прямо вот на поиск по известному id, то в таком DTO айдишник нужен. Сейчас выглядит так, как будто DTO универсальные. Пример возможных DTO (далеко не полный, могут быть совершенно разные, исходя из задач): CreateUserRequest SearchUserRequest EditUserRequest

Mercur1y commented 2 years ago

Создал разные дтохи для всего https://github.com/Mercur1y/con_calculator/commit/e0ff017ba4631f3b455202373add135f3af8724c