Skip to content

Conversation

@titovilya
Copy link

Описание проекта в INSTRUCTION.md

@yaroslav163
Copy link

yaroslav163 commented Nov 8, 2021

@titovilya очень круто, что написал подробную инструкцию, и, по сути, запуск идёт всего набором одной строки.
Понятно зачем отделил модуль api в отдельный модуль, но не понятно зачем разделил на модули application, model, security, service? Логичнее всего иметь 2 модуля api и сервис, твоя версия валидная, если каждый модуль ты будешь использовать где-то отдельно, но будет ли так в реальности? Всё хорошо собирается, поднимается и запускается как локально, так и в облаке. Нет вообще Unit тестов - так ты не гарантируешь, что код работает в рамках заданных ожиданий - т.е. приходится тестировать твой код вручную поднимая и подготавливая данные для теста.

@@ -0,0 +1,18 @@
package api.mappers;

Choose a reason for hiding this comment

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

@titovilya логичнее было бы вынести mapper-ы в сервисы нежели в api. Api должно содержать лишь данные по внешним контрактам, всё остальное находится в зоне ответственности самого сервиса.

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