0

У меня есть две сущности User Role со связей Many to Many,я использую метод save, он делает вот такой запрос, но мне не нужно, что бы он создавал новую роль, а нужно, что бы извлекал id из таблицы role вставлял в связную таблицу id этой role и связывал ее user_id как это сделать? Спасибо большое. Извините за глупый вопрос.

Запрос при save

  *user.insert into users (first_name, last_name, password, state, username, id)
      values (?, ?, ?, ?, ?, ?)
  Hibernate: insert into role (role_name, id) values (?, ?)*

User

@Entity
    @Table(name = "users")
    public class User implements Serializable {
        @Id
        @GeneratedValue(strategy = GenerationType.AUTO)
        private Long id;
        @Column(name = "username")
        private String username;
        @Column(name = "password")
        private String password;
        @Column(name = "state")
        private Boolean state;
        @Column(name = "first_name")
        private String firstName;
        @Column(name = "last_name")
        private String lastName;
        @ManyToMany(fetch = FetchType.EAGER, cascade = CascadeType.ALL,targetEntity = Role.class)
        @JoinTable(name = "user_roles", joinColumns =
        @JoinColumn(name = "user_id",referencedColumnName = "id"), inverseJoinColumns =
        @JoinColumn(name = "role_id",referencedColumnName = "id"))
        private Set<Role> roles;



        public Set<Role> getRoles() {
            return roles;
        }

        public void setRoles(Set<Role> roles) {
            this.roles = roles;
        }


        public Long getId() {
            return id;
        }

        public void setId(Long id) {
            this.id = id;
        }

        public String getUsername() {
            return username;
        }

        public void setUsername(String username) {
            this.username = username;
        }

        public String getPassword() {
            return password;
        }

        public void setPassword(String password) {
            this.password = password;
        }

        public Boolean getState() {
            return state;
        }

        public void setState(Boolean state) {
            this.state = state;
        }

        public String getFirstName() {
            return firstName;
        }

        public void setFirstName(String firstName) {
            this.firstName = firstName;
        }

        public String getLastName() {
            return lastName;
        }

        public void setLastName(String lastName) {
            this.lastName = lastName;
        }
    }

Role

@Entity
        @Table(name = "role")
        public class Role implements Serializable {

            @Id
            @GeneratedValue(strategy = GenerationType.AUTO)
            private Long id;
            @Column(name = "role_name")
            private String roleName;
            @ManyToMany(mappedBy = "roles", cascade = CascadeType.ALL, fetch = FetchType.EAGER,targetEntity = User.class)
            private Set<User> users;

            public Set<User> getUsers() {
                return users;
            }

            public void setUsers(Set<User> users) {
                this.users = users;
            }

            public void setId(Long id) {
                this.id = id;
            }

            public long getId() {
                return id;
            }


            public String getRoleName() {
                return roleName;
            }

            public Role setRoleName(String roleName) {
                this.roleName = roleName;
                return this;
            }
        }

Repository

@Repository("UserRepository")
public interface UserRepository extends JpaRepository<User, Long> {
    @Query("SELECT u FROM  User u WHERE  u.username=:username")
    User getUserByUsername(@Param("username") String username);


}

UserService

@Service
public class UserServiceImpl implements UserService {

    @Autowired
    UserRepository userRepository;


    @Override
    public void addUser(User user) {
        user.setRoles(new HashSet<Role>() {{
            add(new Role().setRoleName("USER"));
        }});
        user.setState(true);
        userRepository.save(user);


    }
}
3
  • 1
    Так вы же сами в методе addUser создаёте новую роль. Выбирайте и устанавливайте существующую роль. 2 июн 2017 в 12:44
  • Хм,я думал,что будет автоматически искать эту роль в таблице и сам ее вытаскивать и связывать,то есть мне нужно самому написать запрос на выборку этой id_role и вставку в связную таблицу?
    – user233714
    2 июн 2017 в 12:51
  • @MaximBogunenko Если бы ключ был roleName, то можно было бы так делать.
    – Alex78191
    2 июн 2017 в 13:14

1 ответ 1

0

Роль я бы заменил на enum и упростил себе жизнь. Останется только одна сущность User. Примерно так:

@Entity
public class User {

    private String name;

    @ElementCollection
    private List<Role> roles = new ArrayList<>();

    public User(String name, Role... roles) {
        this.name = name;
        this.roles.addAll(Arrays.asList(roles));
    }

    //...
}

public enum Role implements GrantedAuthority {
    USER, ADMIN

    @Override
    public String getAuthority() {
        return name();
    }
}

В итоге, сохранение юзера будет выглядеть так:

User user = userRepository.save(new User("user", USER));

Рабочий пример.

Позволю пару советов...

  1. Отношение Юзер-Роль никак не ManyToMany. Это OneToMany (один юзер может иметь много ролей). Пример ManyToMany - это Школьники-Кружки: Школьник может посещать несколько школьных кружков. И каждый кружок посещает несколько школьников.

  2. Роль вообще ничего не знает про юзеров, поэтому в ней не стоит иметь ссылку на юзера.

  3. У тебя роль записывается вместе с юзером, потому что установлен Cascade=All. В этом случае ВСЕ операции с верхней сущностью каскадно выполняются и с подчиненной. Каскад здесь совсем лишний, т.к. роль не зависит от юзера.

Очень рекомендую почитать этот материал.

9
  • Это не всегда возможно. Например, Spring Security ожидает, что в базе будет отдельная таблица для ролей. 2 июн 2017 в 13:25
  • @SergeyGornostaev Возможно )) Все зависит от того как мы реализуем UserDetailsService#loadUserByUsername
    – Cepr0
    2 июн 2017 в 13:28
  • @SergeyGornostaev добавил в ответе имплементацию GrantedAuthority для Role.
    – Cepr0
    2 июн 2017 в 13:39
  • Вы были правы насчет,что cascade all тут точно лишний,как только я его убрал,save заработал как нужно,а насчет many to many например автор это го туториала делает так websystique.com/springmvc/…
    – user233714
    3 июн 2017 в 9:56
  • @User Рад, что помог (только не забудь проголосовать...). В статье автор не совсем подходящий пример использовал, отношение User-Profile - это скорее OneToOne.
    – Cepr0
    3 июн 2017 в 12:36

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.