Hibernate多对多关联不更新关联表

8
在我的应用程序中,用户实体和偏好实体之间存在多对多的关联。由于连接表需要额外的列,我不得不将其拆分为两个一对多的关联,如下所示:
用户实体:
@OneToMany(mappedBy = "user", fetch = FetchType.EAGER, cascade={CascadeType.PERSIST, CascadeType.MERGE}, orphanRemoval = true)
public Set<UserPreference> getPreferences() 
{
    return preferences;
}

首选项实体:

@OneToMany(mappedBy = "preference", fetch = FetchType.EAGER)
public Set<UserPreference> getUserPreferences() 
{
    return userPreferences;
}

用户偏好实体:

@ManyToOne
@JoinColumn(name = "user_id", nullable = false)
public User getUser() 
{
    return user;
}

public void setUser(User user) 
{
    this.user = user;
}

@ManyToOne
@JoinColumn(name = "preference_id", nullable = false)
public Preference getPreference() 
{
    return preference;
}

public void setPreference(Preference preference) 
{
    this.preference = preference;
}

@Column(nullable = false, length = 25)
public String getValue() 
{
    return value;
}

public void setValue(String value) 
{
    this.value = value;
}

要更新偏好之一,我遍历用户的偏好集并按如下方式更新值:

@RequestMapping(value = {"/edit-{id}-preference"}, method = RequestMethod.POST)
public String updateUserPreference(@ModelAttribute("userPreference") UserPreference preference, BindingResult result, ModelMap model) 
{
    User loggedInUser = (User)session.getAttribute("loggedInUser");

    for (UserPreference pref : loggedInUser.getPreferences())
    {
        if (Objects.equals(pref.getId(), preference.getId()))
        {
            pref.setValue(preference.getValue());
        }
    }

    userService.update(loggedInUser);

    return "redirect:/users/preferences";
}

我已确认要更新的用户变量确实在此代码运行后包含新值。更奇怪的是,当重定向发生时,网页上的值确实会更新,但数据库并未更新!这是我用于进行更新的代码,此类使用@Transactional进行注释,调用该方法的每个其他调用(例如更新用户角色)都可以完美运作:

@Override
public void update(User user) 
{
    User entity = dao.findById(user.getId());
    if (entity != null)
    {
        entity.setUserId(user.getUserId());
        entity.setPassword(user.getPassword());
        entity.setFirstName(user.getFirstName());
        entity.setLastName(user.getLastName());
        entity.setRole(user.getRole());
        entity.setPreferences(user.getPreferences());
    }
}

这类似于Hibernate的会话“缓存”具有更新的值,但实际上没有将其持久化。我已经使用这种更新方法风格为其他大约30个实体,一切正常。这是我唯一不得不分解为2个一对多关系的多对多关系,因此我没有可比较的内容。
我做错了什么吗?当我使用新的HashSet创建一个用户并使其持久化时,“join table”中的值会被正确写入。
*****编辑*****
为了比较,这是我用于创建具有默认首选项的新用户的代码。首选项已经存在,但是连接表完全为空,并且此代码正确地持久化了实体:
User user = new User();
        user.setUserId("admin");
        user.setPassword(crypter.encrypt("admin"));
        user.setFirstName("admin");
        user.setLastName("admin");
        user.setRole(roleService.findByName("Admin"));

        Set<UserPreference> userPreferences = new HashSet<>();

        Preference preference = preferenceService.findByName("anchorPage");
        UserPreference userPreference = new UserPreference();
        userPreference.setUser(user);
        userPreference.setPreference(preference);
        userPreference.setValue("System Statistics");
        userPreferences.add(userPreference);

        preference = preferenceService.findByName("showOnlyActivePatients");
        userPreference = new UserPreference();
        userPreference.setUser(user);
        userPreference.setPreference(preference);
        userPreference.setValue("true");
        userPreferences.add(userPreference);

        user.setPreferences(userPreferences);

        userService.save(user);

谢谢

2个回答

2

不要

entity.setPreferences(user.getPreferences());

做类似以下操作:

for( UserPreference uf : user.getPreferences() ) {
    entity.getPreferences().add( uf );
}

这里的主要区别在于您不会更改由 Hibernate 管理的列表引用,而只会向其添加元素。

setPreferences() 需要一个 UserPreference 对象的集合。如果将一个 UserPreference 对象传递给该方法,代码将无法编译。 - Martin
抱歉,@Martin,是我的错。你是正确的!我已经更新了我的答案。请看一下。 - MiguelKVidal
@Martin,你看过我的更新答案了吗?能否告诉我它是否解决了你的问题? - MiguelKVidal

0

使用合并(merge)怎么样?毕竟这就是您正在级联的操作,您已经修改了一个脱离管理状态的对象,并且需要将更改合并回去:

public void update(User user)  {
  dao.merge(user);
}

编辑:为了清晰起见,此替换旧的更新方法,因此应像以前一样从客户端使用loggedInUser调用它。

编辑2:如评论中所述,合并将更新所有字段。旧的更新方法似乎也是这样做的? 乐观锁(版本号)可用于防止错误地覆盖其他更改。


我同意你的看法,loggedInUser是一个分离实体。但请看一下update方法的第一行:User entity = dao.findById(user.getId());。如果我没错的话,从这个点开始,他正在更新一个已附加的实体。你能否为我澄清一下?也许我读错了什么。抱歉发表评论,但我不想在这里引发争端,只是想更好地理解问题和解决方案。 - MiguelKVidal
当然可以,但是我的更新方法的第一行是dao.merge(user)。我完全替换了他的更新方法。在两种情况下,传递给更新方法的用户都是loggedInUser。他读取用户并将其分配给实体,并将值从loggedInUser复制到实体中。我将这留给Hibernate处理。合并方法将从数据库中读取用户并解决问题。希望这样清楚了吗?如果没有,我不介意进一步的评论,也许我需要以某种方式澄清答案。 - ewramner
从他的更新方法来看,他似乎已经在更新每个字段(甚至是密码,但肯定没有更改),但我同意这可能会引起问题。我的建议是使用乐观锁(版本号)来避免错误地覆盖字段,并可选择在客户端处理冲突,但如果当前代码库不使用版本,则可能需要大量工作才能在各处添加它。 - ewramner
合并确实有效!我是在一年多前学习Spring MVC / Hibernate集成时获得了这种更新方法的。能否详细说明为什么在这种情况下会出现这种行为以及为什么合并有效? - Martin
在https://www.thoughts-on-java.org/persist-save-merge-saveorupdate-whats-difference-one-use中详细解释了这一点。当您调用update方法时,Hibernate仅会选择您提供的实体作为方法参数。但是,当您调用JPA的merge方法时,Hibernate还将选择所有级联类型为CascadeType.MERGE的关联实体。(引自文章)。太棒了,它起作用了! - ewramner
显示剩余3条评论

网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接