хандлеры дао все перенести

все комменты убрать

https://github.com/getCurrentUserName/TheNewBoltalka/blob/master/src/main/java/com/boltalka/services/auth/PasswordEncoderService.java

создать интерфейсов

абсолютно бесполезный пакет и класс

где у тебя шифрование пароля юзается?

при регистрации или где еще?

только в 1 месте

в регистрации авторизации

и при смене

А че Spring Security ниасилил?

кажись

:P

В конфигурации твоего проекта

SecurityConfig

` @Bean public PasswordEncoder passwordEncoder() { return new BCryptPasswordEncoder(); }`

далее инжект в сервис, где аккаунт регаешь/авторизуешь

и потом passwordEncoder.encode(...)

вот и все

если есть аналоигчные классы, сделай им бин дефинишн, чем городить пакеты и классы

this.user.setPassword(passwordEncoderService.encodingPassword(user.getPassword()));

ну я это просто пока так оставил

если есть аналоигчные классы, сделай им бин дефинишн, чем городить пакеты и классы

Типа сервис?

Типа сервис?

ага

а то получается сервис в сервисе

и потом лапша как выше

чтоб потом что нибудь добавить

/** Поиск имени пользователя */ public CommandStatus findByUserToDatabase(User user) {

соррян, просто ору с этого

/** Поиск имени пользователя */ public CommandStatus findByUserToDatabase(User user) {

где это я написал

https://github.com/getCurrentUserName/TheNewBoltalka/blob/master/src/main/java/com/boltalka/services/auth/UserRegisterService.java

??????????

соррян, просто ору с этого

Вот я хотел чтобы мой гейм-сервер так же пролинчевали ))

https://github.com/getCurrentUserName/TheNewBoltalka/blob/master/src/main/java/com/boltalka/services/auth/AuthUserDetails.java

не вижу смысла делать с этого сервис спринговый

сделать SecurityUtils

и статические методы

ведь сама инфа у тебя там статикой дергается

SecurityContextHolder.getContext().getAuthentication().getPrincipal();

плюс все, что связано с юзером объединить в один сервис можно

чем дробить

AccountService

как предложение

я заебался ходить по пака с двумя классами и парой строчек

??

https://github.com/getCurrentUserName/TheNewBoltalka/tree/master/src/main/java/com/boltalka/services/images

тоже самое этого касается

/** * Поиск по id * @param id - id пользователя * */ public User findById(UUID id) { return userDAO.findById(id); }

все

у меня любая ошибка это минус к зп ) на сумму ошибки )

Мотивация рублем. Бежать из такой конторы надо

не могу ) контора то моя ))

ты добил меня

https://github.com/getCurrentUserName/TheNewBoltalka/tree/master/src/main/java/com/boltalka/services

First name Last name, а почему у тебя все сервисы state-full?

auth и user пакеты работают только с user моделью

логично объединитьв один пакет

и слить классы

initializers классы перенсти в пак config

Блин че то инет пропал

Я только учусь) благодарю за все замечании) может через год чуть лучше буду писать)

А как там кстати ентити

Что не правильно и не так

нейминг

ERROR: S client not available

можно так же создать пакет Util

туда помощников различных перенести

А нормально всякие эти наследовании итп

вот вижу у тебя кучу инлайн констант дублирующихся

лучше их вынести в константы

BoltalkaConstants.DATE

лучше их вынести в константы

Я так пробовал, но не понравилось, этих констант в дао использую

https://github.com/getCurrentUserName/TheNewBoltalka/blob/master/src/main/java/com/boltalka/persistence_layer/entities/images/Image.java

вот это вообще не понял

А где это

https://github.com/getCurrentUserName/TheNewBoltalka/tree/master/src/main/java/com/boltalka/persistence_layer/entities/images

Я ща с телефона сижу

Аа все

ну у тебя модель профиль аймэдж

и нахера интерфейс

с геттерами

Я бы еще дал совет, безотностельно Java и Spring по поводу > @org.hibernate.annotations.Cache(usage = CacheConcurrencyStrategy.READ_WRITE) Не используй кэш пока его можно не использовать.

Ну потом другие энтети сделаю

Я бы еще дал совет, безотностельно Java и Spring по поводу > @org.hibernate.annotations.Cache(usage = CacheConcurrencyStrategy.READ_WRITE) Не используй кэш пока его можно не использовать.

почему?

Ну потом другие энтети сделаю

смысл просто в этом?

ну у тебя модель профиль аймэдж

Потом создам другие энтити

И их имплементирую от имадже

т.е. у тебя будет меняться логика в геттере и в сеттере?!

почему?

Потому что кэш это 2е хранилище данны. Пусть и временное. Появляется как минимум лишняя точка отказа, возможность нарушения консистенции кэша и т.д.

т.е. у тебя будет меняться логика в геттере и в сеттере?!

Нет

Ну и самое главное - кэш может и будет маскировать реальные проблемы производительности

И их имплементирую от имадже

ответь на вопрос

Которые надо решать не кэшем.

ответь на вопрос

Для удалении файлов или чему нибудь может пригодятся еще

каких файлов?

геттеры и сеттеры у тебя ТОЛЬКО чистые

Изображений