Почему вылетает nullpointer?

Question

[Nightmare058]

Доброго времени суток
Помогите понять, почему вылетает nullpointer при попытке регистрации?
Отправляют POST-запрос через postman.

{
    "username": "username",
    "password": 123123,
    "passwordConfirm": 123123
}

Неужели где-то не присваиваю?

2021-11-10 01:31:42.886 INFO 5753 --- [nio-8080-exec-2] o.h.h.i.QueryTranslatorFactoryInitiator : HHH000397: Using ASTQueryTranslatorFactory
Hibernate: select user0_.id as id1_1_, user0_.password as password2_1_, user0_.username as username3_1_ from t_user user0_ where user0_.username=?
2021-11-10 01:31:42.960 ERROR 5753 --- [nio-8080-exec-2] o.a.c.c.C.[.[.[/].[dispatcherServlet] : Servlet.service() for servlet [dispatcherServlet] in context with path [] threw exception [Request processing failed; nested exception is java.lang.NullPointerException] with root cause

java.lang.NullPointerException: null
at com.example.notes.service.UserService.saveUser(UserService.java:57) ~[classes/:na]
at com.example.notes.controllers.RegistrationController.addUser(RegistrationController.java:23) ~[classes/:na]
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[na:na]
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[na:na]
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[na:na]
at java.base/java.lang.reflect.Method.invoke(Method.java:566) ~[na:na]
at org.springframework.web.method.support.InvocableHandlerMethod.doInvoke(InvocableHandlerMethod.java:190) ~[spring-web-5.1.8.RELEASE.jar:5.1.8.RELEASE]
at org.springframework.web.method.support.InvocableHandlerMethod.invokeForRequest(InvocableHandlerMethod.java:138) ~[spring-web-5.1.8.RELEASE.jar:5.1.8.RELEASE]
at org.springframework.web.servlet.mvc.method.annotation.ServletInvocableHandlerMethod.invokeAndHandle(ServletInvocableHandlerMethod.java:104) ~[spring-webmvc-5.1.8.RELEASE.jar:5.1.8.RELEASE]
at org.springframework.web.servlet.mvc.method.annotation.RequestMappingHandlerAdapter.invokeHandlerMethod(RequestMappingHandlerAdapter.java:892) ~[spring-webmvc-5.1.8.RELEASE.jar:5.1.8.RELEASE]
at org.springframework.web.servlet.mvc.method.annotation.RequestMappingHandlerAdapter.handleInternal(RequestMappingHandlerAdapter.java:797) ~[spring-webmvc-5.1.8.RELEASE.jar:5.1.8.RELEASE]
at org.springframework.web.servlet.mvc.method.AbstractHandlerMethodAdapter.handle(AbstractHandlerMethodAdapter.java:87) ~[spring-webmvc-5.1.8.RELEASE.jar:5.1.8.RELEASE]
at org.springframework.web.servlet.DispatcherServlet.doDispatch(DispatcherServlet.java:1039) ~[spring-webmvc-5.1.8.RELEASE.jar:5.1.8.RELEASE]
at org.springframework.web.servlet.DispatcherServlet.doService(DispatcherServlet.java:942) ~[spring-webmvc-5.1.8.RELEASE.jar:5.1.8.RELEASE]
at org.springframework.web.servlet.FrameworkServlet.processRequest(FrameworkServlet.java:1005) ~[spring-webmvc-5.1.8.RELEASE.jar:5.1.8.RELEASE]
at org.springframework.web.servlet.FrameworkServlet.doPost(FrameworkServlet.java:908) ~[spring-webmvc-5.1.8.RELEASE.jar:5.1.8.RELEASE]
at javax.servlet.http.HttpServlet.service(HttpServlet.java:660) ~[tomcat-embed-core-9.0.21.jar:9.0.21]
at org.springframework.web.servlet.FrameworkServlet.service(FrameworkServlet.java:882) ~[spring-webmvc-5.1.8.RELEASE.jar:5.1.8.RELEASE]
at javax.servlet.http.HttpServlet.service(HttpServlet.java:741) ~[tomcat-embed-core-9.0.21.jar:9.0.21]
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:231) ~[tomcat-embed-core-9.0.21.jar:9.0.21]
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166) ~[tomcat-embed-core-9.0.21.jar:9.0.21]
at org.apache.tomcat.websocket.server.WsFilter.doFilter(WsFilter.java:53) ~[tomcat-embed-websocket-9.0.21.jar:9.0.21]
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:193) ~[tomcat-embed-core-9.0.21.jar:9.0.21]
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166) ~[tomcat-embed-core-9.0.21.jar:9.0.21]

RegistrationController

@PostMapping("/registration")
    public String addUser(@RequestBody UserDto userDto) {

        if (!userDto.getPassword().equals(userDto.getPasswordConfirm())) {
            return "Password not match";
        }

        if (!userService.saveUser(userDto)) {
            return "User already exists";
        }

        return "redirect:/";
    }

UserSercive

public boolean saveUser(UserDto userDto) {
        User user = userRepository.findByUsername(userDto.getUsername());
        if (user != null) {
            return false;
        }

        user.setRoles(Collections.singleton(new Role(1L, "ROLE_USER")));
        user.setUsername(userDto.getUsername());
        user.setPassword(bCryptPasswordEncoder.encode(userDto.getPassword()));
        userRepository.save(user);

        return true;
    }

Ссылка на гит:
https://github.com/Akella058/notes_api

Comments

[Сергей Горностаев]

Ошибка не полная. По стектрейсу видно из какого именно места вылетел NPE.

[Nightmare058]

Сергей Горностаев, спасибо, поправил
Или нужно еще что-то добавить?

[Сергей Горностаев]

Nightmare058, 57-я строка - это какая именно?

[Сергей Горностаев]

Nightmare058, подозрительно выглядит условие if (user != null) return false, явно должно быть наоборот.

[Nightmare058]

Сергей Горностаев,

user.setRoles(Collections.singleton(new Role(1L, "ROLE_USER")));

Answer 1

[BorLaze]

"Компьютер делает не то, что вы хотели, а то, что вы ему приказали"

Внимательно посмотри на логику своего saveUser.
Ты ищешь юзера по имени. Если он найден (user != null), то возвращаешь false.

А если не найден (то есть user == null) то... ты вызываешь user.setRoles(...)
Ессно, ловя NPE.

Comments

[Nightmare058]

спасибо, исправлю)

[Nightmare058]

BorLaze , посмотрел еще повнимательнее
NPE появляется в UserService

user.setRoles(Collections.singleton(new Role(1L, "ROLE_USER")));

Почему-то роль не присваивается

[BorLaze]

Nightmare058, шутишь? Я ж тебе расписал - если у тебя в результате userRepository.findByUsername() юзер найден, то ты из функции возвращаешь false.

А на строку user.setRoles() попадаешь, только если выполняется условие user == null.

Конечно, в этом случае при попытке вызвать любой метод юзера ты поймаешь NPE!

В конце концов, пройдись по saveUser() дебаггером, сразу сам все увидишь.

[Nightmare058]

BorLaze, я сейчас упростил все, хочу просто сохранить пользователя, без проверок.
Но userRepository.save не сохраняет.
Через дебаггер смотрю, вроде присваивает все данные

Вот так пока упростил, чтобы разобраться в вопросе:

public boolean saveUser(UserDto userDto) {
//        User user = userRepository.findByUsername(userDto.getUsername());
//        if (user != null) {
//            return false;
//        }

        User user = new User();

        user.setRoles(Collections.singleton(new Role(1L, "ROLE_USER")));
        user.setUsername(userDto.getUsername());
        user.setPassword(bCryptPasswordEncoder.encode(userDto.getPassword()));
        userRepository.save(user);

        return true;
    }

И вот что вижу в дебаггере

может это быть как-то связано с тем, что у меня id = null?

[Nightmare058]

BorLaze, извиняюсь, думал стрин откроется на полную

[BorLaze]

Nightmare058, так правильней, да. Если что-то с таким именем уже есть, уходим, если нет - создаем нового юзера, сетапим и сохраняем. Первые строчки вполне можно раскомментировать, кстати. Проблема не в них была.

может это быть как-то связано с тем, что у меня id = null?

Вполне. Смотря как у тебя User описан и его поле id.

Что пишет при вызове userRepository.save(user)?

[Nightmare058]

BorLaze,

@Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;

При вызове userRepository.save(user):

2021-11-11 01:10:51.121 ERROR 15507 --- [nio-8080-exec-2] o.a.c.c.C.[.[.[/].[dispatcherServlet]    : Servlet.service() for servlet [dispatcherServlet] in context with path [] threw exception [Request processing failed; nested exception is org.springframework.dao.DataIntegrityViolationException: could not execute statement; SQL [n/a]; constraint [users_id]; nested exception is org.hibernate.exception.ConstraintViolationException: could not execute statement] with root cause

org.postgresql.util.PSQLException: ERROR: null value in column "users_id" violates not-null constraint
  Подробности: Failing row contains (null, 1, 1).

[BorLaze]

Nightmare058, а User и Role как связаны? Не many-to-many, случайно?

[Nightmare058]

BorLaze, да, связаны

User
@ManyToMany(fetch = FetchType.EAGER)
    private Set<Role> roles;
Role
@Transient
    @ManyToMany(mappedBy = "roles")
    private Set<User> users;

[BorLaze]

Nightmare058, ага...

Не уверен на 100%, но похоже, что ругается на то, что нет обратной связи роли к юзеру.

Попробуй что-то типа

User user = new User();
Role role = new Role(1L, "ROLE_USER");

role.setUsers(Collections.singleton(user));

user.setRoles(Collections.singleton(role));
...

Если и так не заработает - ну, тогда гугл в руки, смотреть, как правильно many-to-many сохранять. Давно с таким сталкивался, а заново искать откровенно лень :-)