Skip to content

Add controllers#2

Merged
dendzim merged 8 commits intomainfrom
add-controllers
Feb 25, 2026
Merged

Add controllers#2
dendzim merged 8 commits intomainfrom
add-controllers

Conversation

@dendzim
Copy link
Copy Markdown
Owner

@dendzim dendzim commented Feb 20, 2026

Добавлен функционал согласно 14 спринту
Реализованы контроллеры предметов и пользователей
Также размечены каталоги и сущности для последующих спринтов

Comment thread src/main/java/ru/practicum/shareit/exceptions/OnCreate.java Outdated
Comment thread src/main/java/ru/practicum/shareit/item/Item.java Outdated
Comment thread src/main/java/ru/practicum/shareit/item/dao/InMemoryItemStorage.java Outdated
Comment thread src/main/java/ru/practicum/shareit/item/dao/InMemoryItemStorage.java Outdated
return ++currentMaxId;
}

public ItemDto addItem(ItemDto itemDto, Long ownerId) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Стораджи работают только с классами-моделями, преобразование в Dto и обратно лучше делать в классе-сервисе.

Comment thread src/main/java/ru/practicum/shareit/exceptions/ErrorHandler.java
Copy link
Copy Markdown

@VadimZharkov VadimZharkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В целом хорошее начало, нужно поработать над структурой проекта и распределением функциональности между слоями приложения. См. комментарии выше.

@dendzim
Copy link
Copy Markdown
Owner Author

dendzim commented Feb 22, 2026

Внес правки согласно замечаниям. По поводу размещения модели в пакете. Я разбивал проект согласно ТЗ по фичам.
"В этом модуле вы будете использовать структуру не по типам классов, а по фичам (англ. Feature layout) — весь код для работы с определённой сущностью должен быть в одном пакете." Поэтому показалось создавать пакет для одного класса не целесообразным.
Хотя лично мне нравится структура проекта как в прошлом модуле. Все модели в моделях все контролеры в контролерах и.т.д.

@VadimZharkov
Copy link
Copy Markdown

Внес правки согласно замечаниям. По поводу размещения модели в пакете. Я разбивал проект согласно ТЗ по фичам. "В этом модуле вы будете использовать структуру не по типам классов, а по фичам (англ. Feature layout) — весь код для работы с определённой сущностью должен быть в одном пакете." Поэтому показалось создавать пакет для одного класса не целесообразным. Хотя лично мне нравится структура проекта как в прошлом модуле. Все модели в моделях все контролеры в контролерах и.т.д.

Ничего страшного в одном классе в пакете нет, структура с пакетами лучше и понятней. В данной работе вы сначала разбили по фичам, но внутри фичи никто не мешает делать пакеты для улучшения структуры. Хотя в данной работе классов не много и в пределах фичи и так все понятно и можно на делать доп пакетов. Просто нужно единообразие, или раскладываем все по пакетам или же нет.

Copy link
Copy Markdown

@VadimZharkov VadimZharkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Доработки увидел, принимается.

@dendzim dendzim merged commit c0cb1b2 into main Feb 25, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants