Обновить
2
Василий Константинов@vasilyk13

Пользователь

1
Подписчики
Отправить сообщение

Статья годная для тех кто только осваивает. Молодец, что делишься своим видением.

В методе findUserById класса UserService вы возвращаете пустого пользователя, если пользователь в базе не найден. Имхо корректнее вернуть Optional - так в дальнейшей логике гораздо проще проверить что вернулось - .isPresent().

Метод usergtList в сервисном слое обходит DAO обращаясь напрямую в БД (через EntityManager). Это нарушает приципы SOLID. Метод нужно разделить на сервисную часть и репозиторную

Информация

В рейтинге
Не участвует
Откуда
Россия
Зарегистрирован
Активность

Специализация

Бэкенд разработчик, Пресейл менеджер
ООП
Java