Статья годная для тех кто только осваивает. Молодец, что делишься своим видением.
В методе findUserById класса UserService вы возвращаете пустого пользователя, если пользователь в базе не найден. Имхо корректнее вернуть Optional - так в дальнейшей логике гораздо проще проверить что вернулось - .isPresent().
Метод usergtList в сервисном слое обходит DAO обращаясь напрямую в БД (через EntityManager). Это нарушает приципы SOLID. Метод нужно разделить на сервисную часть и репозиторную
Статья годная для тех кто только осваивает. Молодец, что делишься своим видением.
В методе findUserById класса UserService вы возвращаете пустого пользователя, если пользователь в базе не найден. Имхо корректнее вернуть Optional - так в дальнейшей логике гораздо проще проверить что вернулось - .isPresent().
Метод usergtList в сервисном слое обходит DAO обращаясь напрямую в БД (через EntityManager). Это нарушает приципы SOLID. Метод нужно разделить на сервисную часть и репозиторную