search

nurlandadasev / mobile-banking-app

mobile-banking-app
0 stars 0 forks source link

Returning ResponseEntity from service methods #4

Closed aandrey1993 closed 3 years ago

aandrey1993 commented 3 years ago

Описание

При запросе всех счетов, из методов сервиса AccountServiceImpl (getAccounts, getAccountsWithFiltering), сразу возвращается объект ResponseEntity, что не является хорошей практикой.

изображение

Формированием ответа ResponseEntity должен заниматься контроллер, сервис же должен вернуть просто список найденных объектов.

Исправьте, пожалуйста, данный момент.

nurlandadasev commented 3 years ago

Хорошо буду иметь в виду.

28 мая 2021 г., в 17:34, aandrey1993 @.***> написал(а):

Описание

При запросе всех счетов, из методов сервиса AccountServiceImpl (getAccounts, getAccountsWithFiltering), сразу возвращается объект ResponseEntity, что не является хорошей практикой.

https://user-images.githubusercontent.com/49781313/119997882-dd34d380-bfd8-11eb-800b-ff40479a4d03.png Формированием ответа ResponseEntity должен заниматься контроллер, сервис же должен вернуть просто список найденных объектов.

Исправьте, пожалуйста, данный момент.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nurlandadasev/mobile-banking-app/issues/4, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASDU4ENEO4P3KS2MHXMTFLDTP6SXDANCNFSM45WY3BQQ.