键“P​​RIMARY”的重复条目“string1-string2”

Question

在通过 MySQL 数据库使用 hibernate 和 jpa 的 Spring MVC 应用程序中，每当我尝试保存包含子实体的父实体时，都会收到以下关于子实体的错误消息：

Duplicate entry 'string1-string2' for key 'PRIMARY'  

这里，string1和string2指的是子实体的复合主键的两部分。 如何解决此错误？

这里是在父Address实体中定义实体间关系的方式：

@ManyToOne(cascade = { CascadeType.ALL }, fetch=FetchType.EAGER)
@JoinColumns({ @JoinColumn(name = "usecode", referencedColumnName = "code", insertable = false, updatable = false),
        @JoinColumn(name = "usecodesystem", referencedColumnName = "codesystem", insertable = false, updatable = false)
})
public HL7GeneralCode use;

这是在子GeneralCode实体中定义关系的方式：

@OneToMany(mappedBy = "use", cascade = {CascadeType.ALL})
private Set<HL7Address> addresses;

完整的堆栈跟踪可以查看by clicking on this link。
Address 实体的完整代码可以在 at this link 找到。

GeneralCode 实体的完整代码可以阅读at this link。

复合主键类的代码可以在at this link找到。
而由Address扩展的BaseEntity类可以在at this link找到。

我已经阅读了很多关于此错误消息的帖子。其他帖子的答案无法解决我的错误消息，并且它们通常无法解决我的实体使用复合主键的事实。

---

编辑：

持久化地址的代码是：

@Override
public void savehl7Address(HL7Address addr) {
    if ((Integer)addr.getId() == null) {
        System.out.println("[[[[[[[[[[[[ about to persist address ]]]]]]]]]]]]]]]]]]]]");
        this.em.persist(addr);}
    else {
        System.out.println("]]]]]]]]]]]]]]]]]] about to merge address [[[[[[[[[[[[[[[[[[[[[");
        this.em.merge(addr);}
}

---

第二次编辑：

我尝试遵循@Ben75 的建议，但代码在this.em.persist(addr.getUse()); 行崩溃。请注意，他的 if 子句不适合我的实际对象模型，因此我将下面的 if 子句更改为 if(addr.getUse() != null && addr.getId()==null)。这是我的代码。

@Override
public void savehl7Address(HL7Address addr) {
    if(addr.getUse() != null && addr.getId()==null){
        //this next line prints in the stack trace right before the app crashes
        System.out.println("about to this.em.persist(addr.getUse());");
        //HL7GeneralCode is not persistent yet
        this.em.persist(addr.getUse());
        //since there is a cascade ALL on the adresses relationship addr is now persistent
        return;
    }
    System.out.println("=========================== inside jpahl7patientrespository.savehl7Address(addr)");
    if ((Integer)addr.getId() == null) {
        System.out.println("[[[[[[[[[[[[ about to persist address ]]]]]]]]]]]]]]]]]]]]");
        this.em.persist(addr);}
    else {
        System.out.println("]]]]]]]]]]]]]]]]]] about to merge address [[[[[[[[[[[[[[[[[[[[[");
        this.em.merge(addr);}
}

HL7Address的相关部分现在是：

@ManyToOne(fetch=FetchType.EAGER)
@JoinColumns({ @JoinColumn(name = "usecode", referencedColumnName = "code", insertable = false, updatable = false),
        @JoinColumn(name = "usecodesystem", referencedColumnName = "codesystem", insertable = false, updatable = false)
})
public HL7GeneralCode use;

HL7GeneralCode的相关部分现在是：

@OneToMany(mappedBy = "use")
private Set<HL7Address> addresses;

可以读取新的堆栈跟踪by clicking on this link。

我该如何解决这个错误？

---

第三次编辑：

我按照 ben75 的建议在保存地址方法中添加了以下代码：

if(addr.getUse() != null && !this.em.contains(addr.getUse())){
    System.out.println("about to this.em.persist(addr.getUse());");
    this.em.persist(addr.getUse());return;
}

不幸的是，尽管堆栈跟踪SYSO 表明上述代码在应用程序崩溃之前正在运行，但我仍然遇到相同的错误。

您可以阅读生成的堆栈跟踪by clicking on this link。

Answer 1

首先有几点要搞清楚：

1. 您在 HL7GeneralCode（父）和 HL7Address（子）之间存在双向关联。如果 HL7GeneralCode.addresses 是“反向”端（mappedBy），那么为什么拥有端 HL7Address.use 具有可插入/可更新的错误？拥有方应控制此关联，因此您应删除 insertable/updatable=false 标志。

2. 从父级级联到子级总是有意义的，而不是相反。但是在您的用例中，您尝试保留 Child 并自动保留 Parent。这就是多对一的 CASCADE.ALL 没有意义的原因。

3. 使用双向关联时，双方都必须设置：

   HL7Address addr = new HL7Address();
   HL7GeneralCode code = new HL7GeneralCode();
   ...
   code.getAddresses().add(addr);
   addr.setUse(code); 
   

4. persist 操作旨在插入瞬态实体，从不合并它们或重新附加实体。这意味着当您调用服务方法时，HL7Address 和 HL7GeneralCode 都是新实体。如果你已经保存了一个相同ID的HL7GeneralCode，你会得到主键约束违规异常。

5. 如果 HL7GeneralCode 可能存在，那么您应该从 db 中获取它。

   HL7GeneralCode code = em.find(HL7GeneralCode, pk);
   HL7Address addr = new HL7Address();
   if(code != null) {
      code = new HL7GeneralCode();
      em.persist(code);    
   }
   code.getAddresses().add(addr);
   addr.setUse(code);            
   em.persist(addr);
   

更新

6. HL7Address 地址不会覆盖 equals/hashCode，因此默认对象相同的引用检查规则适用。这将确保我们可以从 code.addresses 列表中添加/删除地址。如果您以后改变主意，请确保您implement equals and hashCode properly。

7. 虽然与您的问题无关，但您可能希望使用 getter/setter 而不是公开您的字段。这提供了更好的封装，您将避免将 setter 与公共字段访问混合。

savehl7Address 方法：

@Override
public void savehl7Address(HL7Address addr) {
    HL7GeneralCode code = addr.use();
    if(code != null && code.getId()==null){
    //HL7GeneralCode is not persistent. We don't support that
        throw new IllegalStateException("Cannot persist an adress using a non persistent HL7GeneralCode");
       //In case you'd want to support it
       //code = em.find(HL7GeneralCode, code.getId());
    }
    //Merge the code without any address info        
    //This will ensure we only reattach the code without triggering the address 
    //transitive persistence by reachability
    addr.setUse(null);
    code.getAddresses().remove(addr);
    code = em.merge(code); 

    //Now set the code to the address and vice-versa  
    addr.setUse(code);
    code.getAddresses().add(addr);

    if ((Integer)addr.getId() == null) {
        System.out.println("[[[[[[[[[[[[ about to persist address ]]]]]]]]]]]]]]]]]]]]");
        em.persist(addr);
    }
    else {
        System.out.println("]]]]]]]]]]]]]]]]]] about to merge address [[[[[[[[[[[[[[[[[[[[[");
        addr = em.merge(addr);
    }       
}

Answer 2

看来问题是use 关系上的CascadeType.ALL。

发生了什么事？

• 您的数据库中有一个 HL7GeneralCode persistent 实例。我们称之为：code1。

• 您创建一个新的Address 并使用以下内容定义use 关系：

  theNewAdress.setUse(code1);

• 您拨打savehl7Address(theNewAddress)，由于地址是新地址，您拨打了persist。 问题在于级联规则CascadeType.ALL 将强制调用persist(code1) 并且因为code1 已经在db 中：由于重复条目而崩溃。

解决方案：

为use 关系定义无级联规则：

@ManyToOne(fetch=FetchType.EAGER)
@JoinColumns({ @JoinColumn(name = "usecode", referencedColumnName = "code", insertable = false, updatable = false),
        @JoinColumn(name = "usecodesystem", referencedColumnName = "codesystem", insertable = false, updatable = false)
})
public HL7GeneralCode use;

但是您必须手动管理它，特别是如果存在地址使用的HL7GeneralCode 不在数据库中的用例。

过度简化的解决方案（以便您理解问题）：

@Override
public void savehl7Address(HL7Address addr) {
    if(addr.use() != null && addr.use().getId()==null){
        //HL7GeneralCode is not persistent yet
        this.em.persist(addr.use());
        //since there is a cascade ALL on the adresses relationship addr is now persistent
        return;
    }
    if ((Integer)addr.getId() == null) {
        System.out.println("[[[[[[[[[[[[ about to persist address ]]]]]]]]]]]]]]]]]]]]");
        this.em.persist(addr);}
    else {
        System.out.println("]]]]]]]]]]]]]]]]]] about to merge address [[[[[[[[[[[[[[[[[[[[[");
        this.em.merge(addr);}
}

如您所见，此解决方案肯定不是最好的，也未准备好生产。真正的解决方案是研究所有用例并相应地调整级联规则（在use 和adresses 关系上）。

在我看来，最好的办法是确保在您调用savehl7Address 时HL7GeneralCode 已经是持久的，因此这样的事情可能是一个更好的解决方案：

@Override
public void savehl7Address(HL7Address addr) {
    if(addr.use() != null && addr.use().getId()==null){
        //HL7GeneralCode is not persistent. We don't support that
        throw new IllegalStateException("Cannot persist an adress using a non persistent HL7GeneralCode");
    }
    if ((Integer)addr.getId() == null) {
        System.out.println("[[[[[[[[[[[[ about to persist address ]]]]]]]]]]]]]]]]]]]]");
        this.em.persist(addr);}
    else {
        System.out.println("]]]]]]]]]]]]]]]]]] about to merge address [[[[[[[[[[[[[[[[[[[[[");
        this.em.merge(addr);}
}