Conversation
| return ++currentMaxId; | ||
| } | ||
|
|
||
| public ItemDto addItem(ItemDto itemDto, Long ownerId) { |
There was a problem hiding this comment.
Стораджи работают только с классами-моделями, преобразование в Dto и обратно лучше делать в классе-сервисе.
VadimZharkov
left a comment
There was a problem hiding this comment.
В целом хорошее начало, нужно поработать над структурой проекта и распределением функциональности между слоями приложения. См. комментарии выше.
|
Внес правки согласно замечаниям. По поводу размещения модели в пакете. Я разбивал проект согласно ТЗ по фичам. |
Ничего страшного в одном классе в пакете нет, структура с пакетами лучше и понятней. В данной работе вы сначала разбили по фичам, но внутри фичи никто не мешает делать пакеты для улучшения структуры. Хотя в данной работе классов не много и в пределах фичи и так все понятно и можно на делать доп пакетов. Просто нужно единообразие, или раскладываем все по пакетам или же нет. |
Добавлен функционал согласно 14 спринту
Реализованы контроллеры предметов и пользователей
Также размечены каталоги и сущности для последующих спринтов