search

book-hub-umsp / book-hub-api

1 stars 0 forks source link

Feature/#27 book repos controller #41

Closed Tigeroff2002 closed 4 days ago

Tigeroff2002 commented 6 days ago

Добавил тут модели запросов, ответов для контроллера, занимающегося работой с верхне уровневым описанием книги Для работы использовал сервис

Также немного подредактировал репозиторий книги - сделал единый метод обновления с выбором конкретных параметров того, что было изменено

Получил схему из сваггера: https://docs.google.com/document/d/1fw4h4whRP0_WEKsOxeaQ3LrD5WjHzcevVjA3mjQN8nY/edit?tab=t.0

resolved #27

Tigeroff2002 commented 6 days ago

Возможно, стоит еще добавить сюда метод контроллера и репозитория на получение всех жанров (и добавление нового жанра) - так как информация о них закрыта от фронта сейчас

Tigeroff2002 commented 5 days ago

Ты ничего не разбивал, ты мне завел задачу на контроллер - который подразумевает сразу все

AlexBimlnk commented 4 days ago

Фиксани REST и добавление только с автором сделай пока. Из предложений - унести try-catch в контроллеры и там уже отдавать Ok/Bad и не городить лишних абстракций. К тому же api будет грязным когда оно возвращет какой-то "результат" команды, в объекте которого может быть что угодно. Это Rest http контроллеры, здесь не нужен подход с командами. У нас уже есть разные конечные точки, которые и разделяют логику путем get/post/put/delete запросов и роутинга данных (т.е. мы знаем что это за команда и с какими сущностями работает), в данном случае с книгой. А ты пытаешься сделать унифицированный response, а это не нужно, у нас для того контроллеры и эндпоинты разные что они уже делят всё.

Tigeroff2002 commented 4 days ago

Уницифированный response удобен при использовании одного единственного конвертера - где идет вся логика конвертации

AlexBimlnk commented 4 days ago

Я про CommandResult, а не конкретные модели