Spring Boot sql Beziehungen

[Raphael_]

Hi, ich arbeite derzeit an einer spring boot Anwendung, wo man Bots erstellen kann. Meine Bot entity sieht so aus:

@Data
@Entity
@Table(name = "bots")
@NoArgsConstructor
@AllArgsConstructor
@Builder
public class Bot {
    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    private Long id;

    @Column(name = "name")
    private String name;

    @Column(name = "currentMoney", precision = 19, scale = 4)
    private BigDecimal currentMoney;

    @Column(name = "moneyLeftToInvest")
    private BigDecimal moneyLeftToInvest;

    @JoinColumn(name = "bot_id")
    @OneToMany(cascade = CascadeType.ALL, fetch = FetchType.LAZY, orphanRemoval = true)
    private List<Stock> stocks;

    @ElementCollection
    @CollectionTable(name = "botAlgorithms", joinColumns = @JoinColumn(name = "bot_id"))
    @Column(name = "algorithm_name")
    private List<Algorithms> algorithms;
}

Und meine Stock Entity so:

@Data
@Entity
@Table(name = "stocks")
@NoArgsConstructor
@AllArgsConstructor
@Builder
public class Stock {
    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    private Long id;

    @Column(name = "bot_id")
    private Long botId;

    @Column(name = "name")
    private StockType stockType;

    @Column(name = "amount")
    private BigDecimal amount;
}

Im Feld stocks der Bot entity wird mein stock repository durchsucht, das nach allen stocks in dem Feld bot_id nach der aktuellen id des Bots sucht. Damit möchte ich eben in die Liste alle Stocks reinschreiben, die zu diesem Bot gehören. Das funktioniert auch, wenn ich allerdings einen Bot löschen möchte, dann geht das nicht und ich bekomme diesen Fehler:

ERROR: update or delete on table "bots" violates foreign key constraint

Wie kann ich die Beziehung so setzen, dass wenn ich einen Bot lösche automatisch alle Stocks, die zu ihm gehören auch gelöscht werden und ich nicht diese exception bekomme?

 

[Marinek]

Hi,

ehrlich gesagt strugle ich an der Stelle auch. Das Cascade All propagiert save / updat und delete nicht.

Siehe auch: https://stackoverflow.com/questions/306144/jpa-cascadetype-all-does-not-delete-orphans

Je nachdem, was du benutzt, musst du set orphan remove = true setzen.

Gruß,
Martin

 

[KonradN]

Das orphanRevoval = true und den cascadeType.ALL hat er ja beides.

Daher frage ich mich gerade, ob es nicht evtl. an algorithms liegen könnte.
Da könnte dann ggf. eine Annotation wie
@org.hibernate.annotations.Cascade(org.hibernate.annotations.CascadeType.ALL)
ein Versuch wert sein.

 

[thecain]

Unabhängig vom Problem würde ich auch mal das @Data entfernen. Über kurz oder lang wird das bei Entities Probleme machen.

 

[Raphael_]

Hi,

ehrlich gesagt strugle ich an der Stelle auch. Das Cascade All propagiert save / updat und delete nicht.

Siehe auch: https://stackoverflow.com/questions/306144/jpa-cascadetype-all-does-not-delete-orphans

Je nachdem, was du benutzt, musst du set orphan remove = true setzen.

Gruß,
Martin

So wie ich das verstanden habe ist das Problem, dass ich ja einfach nur die Ids abfrage und das mit dem Constraint nur funktioniert, wenn ich den kompletten Bot im Stock entity referenzieren würde... Ich finde es macht aber halt keinen Sinn den Bot im Stock zu "verbinden". Was ich mir gedacht habe, ist dass ich die deleteById Funktion aus dem repo überschreibe und dann dort eben immer die Stocks, die auch gelöscht werden müssen davor eben lösche. Dabei bekomme ich aber denselben Fehler.

 

[Raphael_]

Unabhängig vom Problem würde ich auch mal das @Data entfernen. Über kurz oder lang wird das bei Entities Probleme machen.

Wieso denn? Sollte ich dann stattdessen @Getter und @Setter machen? Warum wird das auf lange Sicht Probleme machen?

 

[Raphael_]

Das orphanRevoval = true und den cascadeType.ALL hat er ja beides.

Daher frage ich mich gerade, ob es nicht evtl. an algorithms liegen könnte.
Da könnte dann ggf. eine Annotation wie
@org.hibernate.annotations.Cascade(org.hibernate.annotations.CascadeType.ALL)
ein Versuch wert sein.

Das hat leider auch nicht geholfen... Ich glaube, dass es was damit zu tun hat, da ich nicht den ganzen Bot referenziere, sondern eben nur die Bot id und dann greift das Constraint aus irgendeinem Grund nicht.

 

[thecain]

Wieso denn? Sollte ich dann stattdessen @Getter und @Setter machen? Warum wird das auf lange Sicht Probleme machen?

Data überschreibt auch toString, equals und hashcode. Das bereitet
Probleme beim LazyLoading und beim Tracken der Entities.

 

[Raphael_]

Hab das jetzt mit getter und setter ersetzt, hat aber das Problem natürlich auch nicht behoben. Ich versteh auch garnicht, warum das so kompliziert ist eine gute Lösung zu finden. Ich meine das kommt doch tausendfach vor, dass man in einer entity die id einer anderen entity hinterlegt...

 

[LimDul]

Meist hinterlegt man entweder die direkte referenz oder macht es in beiden Richtungen nur über die ID. Diesen Mischmasch - A nach B über Objektreferenz, aber B nach A über id habe ich tatsächlich bisher noch nie gesehen

 

[Raphael_]

Meist hinterlegt man entweder die direkte referenz oder macht es in beiden Richtungen nur über die ID. Diesen Mischmasch - A nach B über Objektreferenz, aber B nach A über id habe ich tatsächlich bisher noch nie gesehen

Aber dann würde ich doch unnötige Beziehungen schaffen, was man eigentlich vermeiden sollte, oder nicht?

 

[mihe7]

@Column(name = "algorithm_name")

Das sollte raus, da Du hier ja ein komplexes Objekt (Embeddable) und keinen Basic-Type in der Liste hast.

 

[LimDul]

Aber dann würde ich doch unnötige Beziehungen schaffen, was man eigentlich vermeiden sollte, oder nicht?

Die hast du doch bereits geschaffen. Ich würde sagen entweder kennt stock das Bot Objekt oder nicht mal die Id - aber diesen Mischmasch halte ich für falsch. Das Problem ist, aus Sicht von JPA hast du eine Spalte in Stock, die eine zweifache Bedeutung hat - bot_id.

Das ist einmal ein reines Long-Attribut des Stock Objektes - ohne weitere Bedeutung.
Zum anderen ist es aber auch die Spalte, in der Bot die Rück-Referenz reinspeichert.
Diese beiden Bedeutungen sind getrennt. Ich vermute mal stark, dass sich JPA deswegen lang legt, weil es kann ja die Spalte z.B. nicht auf null setzen, nur weil ein Bot gelöscht wird. Schließlich ist in der Spalte ein Attribut von Stock gespeichert, was nix mit Bot zu tun.

Entweder du machst die Beziehung unidrectional und schmeißt das Attribut in Stock komplett raus (https://www.baeldung.com/spring-jpa-unidirectional-one-to-many-and-cascading-delete) oder du modellierst es sauber bidrectional und machst ein Bot-Objekt mit einer @ManyToOne Beziehung dran.