Как стать автором
Обновить
6
0
Попов Виталий @mskVitalii

Fullstack

Отправить сообщение

Хмм, вы правы. Мне стоит ещё раз пройтись по тексту, спасибо за фидбэк!

Спасибо за code review! Вы указали на ошибки, на которые я бы, вероятно, не обратил внимание в дальнейшем

UserServiceImpl не имплементирует никакой интерфейс/не наследует никакой класс. Под конец работы понял, что это не есть хорошо

Это мой плохой код, нужно добавлять private

В комментариях выше писали о том, что аннотация Autowired необязательна

    final PasswordEncoder passwordEncoder;
    final UserRepository userRepository;
    final ChatUserService chatUserService;

    public UserServiceImpl(@Autowired PasswordEncoder passwordEncoder,
                           @Autowired UserRepository userRepository,
                           @Autowired ChatUserService chatUserService) {
        this.passwordEncoder = passwordEncoder;
        this.userRepository = userRepository;
        this.chatUserService = chatUserService;
    }

Я не совсем корректно вас понял, в данном конструкторе Autowired указан. Вы имеете ввиду, что его необязательно указывать вовсе? Или что-то иное?

Непосредственно с базой работают Entity и Repository. В моём понимании, DTO нужны для удобства представления данных. Не всегда хочется прокидывать сущность со всеми полями или в случае, когда для запроса понадобилась определённая структура. Я ошибаюсь в этом? UPD: осмотрел проект, в Repository я не использую DTO. Действительно, этот момент был глупостью, вы правы. Изменил этот кусок в статье, спасибо)

NoArgConstructor реализует конструктор без параметров. Требуется, соответственно, для создания экземпляра этого класса

Json аннотации нужны для сереализации, Entity для базы. На ваш взгляд, для каждой сущности стоит выделять отдельную DTO для API? Или каково ваше представление?

ДИ лучше делать через конструктор

Я и так делаю Dependency Injection через конструктор. Насколько я знаю, можно к полю в классе задать аннотацию Autowired, однако, в этом проекте я это вроде не использовал / что, впрочем, не возбраняется

Информация

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

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

Backend Developer, Fullstack Developer