Hibernate的saveOrUpdate()方法在应该更新时尝试保存。

6
我有一个叫做IssueParticipant的Hibernate实体。它基本上描述了用户和问题(类似于JIRA或Bugzilla问题)之间的关系。它代表了数据库中一种多对多的链接表,将用户ID与问题ID链接在一起,但也包括与通知设置相关的其他信息,因此被视为自己的实体。
我使用userId和issueId作为复合键时遇到了很大的问题,因此我创建了一个合成键,它是一个字符串(在postgres数据库中是varchar),其形式为:_。
现在,我有一个屏幕,用户可以在其中编辑与问题相关的所有用户,同时还可以编辑通知设置。在控制器类中,我像这样创建了一个IssueParticipants列表:
IssueParticipant participant = new IssueParticipant();
participant.setUser(accountUser);
participant.setIssue(issue);

这些当然在此阶段不由Hibernate管理。

然后在我的DAO中遍历它们并调用saveOrUpdate(),期望如果数据库中存在具有相同合成键的IssueParticipant,则会更新;否则将插入:

    for (IssueParticipant participant : participants) {
        getCurrentSession().saveOrUpdate(participant);
        savedIds.add(participant.getIssueUserKey());
    }

(savedIds是我维护的一个列表,以便稍后知道我应该从数据库中删除哪些IssueParticipants。)

然而,与我的预期不同,我收到了异常:

org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "issue_participant_pkey"

这是我的实体类,缩写:

public class IssueParticipant extends Entity {

    private String issueUserKey;
    private Long issueId;
    private Long userId;

     // Edit: adding 'dateAdded' definition
    private Date dateAdded;
// ...

    // below may be null
    private SPUser user;
    private Issue issue;

    public static IssueParticipant nulledIssueParticipant() {
        IssueParticipant ip = new IssueParticipant();
        return ip;
    }
    public String getIssueUserKey() {
        return issueUserKey;
    }

    public void setIssueUserKey(String issueUserKey) {
        this.issueUserKey = issueUserKey;
    }

    public Long getId() {
        // currently meaningless
        return 0L;
    }

    public Long getIssueId() {
        return this.issueId;
    }

    public void setIssueId(Long issueId) {
        this.issueId = issueId;
        updateKey();
    }

    public Long getUserId() {
        return this.userId;
    }

    public void setUserId(Long userId) {
        this.userId = userId;
        updateKey();
    }

    private void updateKey() {
        issueUserKey = getIssueId() + KEY_SEP + getUserId();
    }

    public SPUser getUser() {
        return user;
    }

    public void setUser(SPUser user) {
        this.user = user;
        setUserId(user.getId());
    }

    public Issue getIssue() {
        return issue;
    }

    public void setIssue(Issue issue) {
        this.issue = issue;
        setIssueId(issue.getId());
    }

// edit: adding 'dateAdded' methods
public Date getDateAdded() {
    return dateAdded;
}

public void setDateAdded(Date dateAdded) {
    this.dateAdded = dateAdded;
}

...

}

这是它的hbm文件:

<?xml version="1.0"?>
<!DOCTYPE hibernate-mapping PUBLIC
        "-//Hibernate/Hibernate Mapping DTD 3.0//EN"
        "http://hibernate.sourceforge.net/hibernate-mapping-3.0.dtd">

<hibernate-mapping default-lazy="false">
    <class name="com.xxx.yyy.IssueParticipant" table="issue_participant">
        <id name="issueUserKey" column="issue_user_key" type="string">
            <generator class="assigned"/>
        </id> 
        <version name="dateAdded" column="date_added" type="timestamp" unsaved-value="null" />
        <property name="issueId" column="issue_id" />
        <many-to-one name="user" column="user_id" class="com.xxx.yyy.SPUser" not-null="true" cascade="none" />
        <property name="alertRss" column="alert_rss" type="boolean" />
        <property name="alertEmail" column="alert_email" type="boolean" />
        <property name="alertWeb" column="alert_web" type="boolean" />
        <property name="alertClient" column="alert_client" type="boolean" />

    </class>
</hibernate-mapping>

实际上,user_issue_key是相应数据库表中的主键。

我认为在这种情况下使用SpringJDBC可能是正确的解决方案,但我真的很想弄清楚这里发生了什么。有人有什么想法吗?提前感谢。

2个回答

12

saveOrUpdate()不会查询数据库来决定是保存还是更新给定的实体,而是根据实体的状态进行决策,具体如下:

  • 如果对象已经存在于此session中,则什么都不做
  • 如果与该session关联的另一个对象具有相同的标识符,则抛出异常
  • 如果对象没有标识符属性,则将其保存(save)
  • 如果对象的标识符具有分配给新实例化对象的值,则将其保存(save)
  • 如果对象由 或 版本控制,并且版本属性值与分配给新实例化对象的值相同,则将其保存(save)
  • 否则,更新(update)该对象

因此,在您的情况下,我理解决策基于dateAdded字段的值,因此您需要保留它以区分新实例和分离实例。

另请参阅:


从前,我一定知道这个,因为我创建了HBM文件(大约两年前)。无论如何,你指出dateAdded中未保存值的作用是我所需要的;谢谢!一旦我获得更多声望点数,或者我可以找到我的先前的登录凭据,我会给你的答案点赞。 - Dave Taubler

2

如果您真的想这样做,可以让它查询数据库来进行确定。将您的标识符映射更改为:

<id name="issueUserKey" column="issue_user_key" type="string" unsaved-value="undefined">
    <generator class="assigned"/>
</id> 

通常情况下,这并不是最好的方法。实际上,您确实有一个<version/>映射,通常比标识符更好地确定保存/更新之间的回退,而无需查询数据库的开销。您尝试将其设置为unsaved-value="null"。但是,您没有显示IssueParticipant.dateAdded属性的属性映射(这是您尝试映射为<version/>的内容;您知道每次更新都会增加版本号吗?您也知道可以使用<timestamp/>代替<version/>吗?)。无论如何,这就是您的问题所在。您能展示一下IssueParticipant.dateAdded属性的定义吗?

当然,我编辑了OP中的代码以添加dateAdded。它只是一个java.util.Date。关于<version>的使用,我一直认为它更多用于乐观锁定;但是,根据axtavt的链接,我想它也可以在状态检测中发挥作用。 - Dave Taubler

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