Pull to refresh
2
Василий Константинов@vasilyk13

User

1
Subscribers
Send message

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

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

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

Information

Rating
Does not participate
Location
Россия
Registered
Activity

Specialization

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