Fehler beim Aktualisieren der JPA-Entität

Question

Ich habe das folgende Domänenmodell

Währung ----< Preis >---- Produkt

Oder auf Englisch

Ein Produkt hat einen oder mehrere Preise. Jeder Preis ist in einer bestimmten Währung angegeben.

Preis hat einen zusammengesetzten Primärschlüssel (dargestellt durch PricePK unten), der aus den Fremdschlüsseln zu Währung und Produkt besteht. Die relevanten Abschnitte der mit JPA annotierten Java-Klassen sind unten aufgeführt (meist ohne Getter und Setter):

@Entity @Inheritance(strategy = InheritanceType.TABLE_PER_CLASS)
public class Currency {

    @Id 
    private Integer ix;

    @Column 
    private String name;

    @OneToMany(mappedBy = "pricePK.currency", cascade = CascadeType.ALL, orphanRemoval = true)
    @LazyCollection(LazyCollectionOption.FALSE)
    private Collection prices = new ArrayList();
}

@Entity @Inheritance(strategy = InheritanceType.TABLE_PER_CLASS)
public class Product {

    @Id @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Integer id;

    @OneToMany(mappedBy = "pricePK.product", cascade = CascadeType.ALL, orphanRemoval = true)
    @LazyCollection(LazyCollectionOption.FALSE)
    private Collection defaultPrices = new ArrayList();
}

@Embeddable
public class PricePK implements Serializable {

    private Product product;    
    private Currency currency;

    @ManyToOne(optional = false)
    public Product getProduct() {
        return product;
    }

    @ManyToOne(optional = false)
    public Currency getCurrency() {
        return currency;
    }    
}

@Entity @Inheritance(strategy = InheritanceType.TABLE_PER_CLASS)
public class Price {

    private PricePK pricePK = new PricePK();

    private BigDecimal amount;

    @Column(nullable = false)
    public BigDecimal getAmount() {    
        return amount;
    }

    public void setAmount(BigDecimal amount) {    
        this.amount = amount;
    }

    @EmbeddedId
    public PricePK getPricePK() {
        return pricePK;
    }    

    @Transient
    public Product getProduct() {
        return pricePK.getProduct();
    }

    public void setProduct(Product product) {
        pricePK.setProduct(product);
    }

    @Transient
    public Currency getCurrency() {
        return pricePK.getCurrency();
    }

    public void setCurrency(Currency currency) {
        pricePK.setCurrency(currency);
    }
}

Wenn ich versuche, eine Instanz von Product zu aktualisieren, erhalte ich einen StackOverflowError, daher vermute ich, dass es irgendeine Art von Zyklus (oder anderer Fehler) in der obigen Zuordnung gibt, kann jemand das erkennen?

Answer 1

Ich habe diesen Fehler schon ein paar Mal gesehen, aber ich kann mich nicht mehr an die genaue Lösung erinnern. Ich habe die Idee, dass du das Mapping von PricePK (beides @ManyToOne) entfernen musst und das mit @AssociationOverrides auf Price ersetzen musst.

@Entity @Inheritance(strategy = InheritanceType.TABLE_PER_CLASS)
@AssociationOverrides({
    @AssociationOverride(name = "pricePK.product", 
                         joinColumns = @JoinColumn(name = "product_id")),
    @AssociationOverride(name = "pricePK.currency", 
                         joinColumns = @JoinColumn(name = "currency_id"))
})
public class Price extends VersionedEntity {
    [...]
}

Bitte überprüfe, ob die Spaltennamen in Ordnung sind, da ich die ID-Spalten auf Product oder Currency nicht sehen kann.

Answer 2

Ich hatte ein ähnliches Problem und es stellte sich heraus, dass es durch CascadeType.ALL verursacht wurde, das in der OneToMany-Annotation definiert war. Wenn Sie ein Produkt aktualisieren, wird versucht, die Preise zu aktualisieren, die im persistenten Kontext sind.

Je nach Situation können Sie möglicherweise darauf verzichten, die Preise zu aktualisieren, wenn das Produkt im Entity Manager aktualisiert wird, wie zum Beispiel:

@OneToMany(mappedBy = "pricePK.product", cascade = {
        CascadeType.PERSIST, CascadeType.MERGE
    }, orphanRemoval = true)
@LazyCollection(LazyCollectionOption.FALSE)
private Collection defaultPrices = new ArrayList();

Answer 3

Sollten Sie nicht einfach die Beziehungen von Preis zu Produkt und Währung als ManyToOne direkt in der Preis-Entität deklarieren und Preis mit @IdClass(PricePK) kennzeichnen?

Ich weiß allerdings nicht, wie Hibernate damit umgeht, aber ich habe dies erfolgreich mit OpenJPA implementiert. In diesem Fall muss PricePK seine Felder mit den gleichen Namen wie in Price deklarieren, jedoch mit dem einfachen Typ (Integer anstelle von Währung oder Produkt). In Price müssen Sie möglicherweise Produkt und Währung mit @Id kennzeichnen. Beachten Sie, dass dies außerhalb der JPA-Spezifikation liegt (@IdClass unterstützt nur einfache Felder).