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在这种情况下使用,但我真的很想弄清楚这里发生了什么。有人有什么想法吗?提前致谢。

4

2 回答 2

12

saveOrUpdate()不查询数据库来决定是否应该保存或更新给定的实体。它根据实体的状态做出该决定,如下所示:

  • 如果对象已在此会话中持久存在,则不执行任何操作
  • 如果与会话关联的另一个对象具有相同的标识符,则抛出异常
  • 如果对象没有标识符属性,则保存()它
  • 如果对象的标识符具有分配给新实例化对象的值,则保存()它
  • 如果对象由 <version> 或 <timestamp> 进行版本控制,并且 version 属性值与分配给新实例化对象的值相同,则 save() 它
  • 否则更新()对象

因此,据我了解,您的情况决定是基于dateAdded字段的值,因此您需要保留它以区分新实例和分离实例。

也可以看看:

于 2012-06-06T15:35:57.207 回答
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 属性的定义吗?

于 2012-06-06T17:20:17.903 回答