OneToMany Composite Key Mapping

[OnDemand]

Hallo zusammen,

ich habe folgendes Beispiel, bekomme aber ein Problem beim Speichern, Select:

public class Product{
@ManyToOne
@JoinColumn(name = "products_shippingtime")
@LazyCollection(LazyCollectionOption.FALSE)
private ShippingStatus shippingStatus;
}

public class ShippingStatus{
@Id
@Column(name = "shipping_status_id")
private int shippingStatusId;

@ManyToOne
@JoinColumn(name = "language_id")
private GLanguage language;

}

ShippingStatus setzt sich aber aus 2 ID zusammen und zwar aus der shipping_status_id und der language_id. Mein Mapping kann daher nicht korrekt sein, es fehlt dass der ShippingStatus aus 2 ID zusammengesetzt ist. Kann mir jemand helfen, wie ich das korrekt mappen muss?

 

[mihe7]

Wofür steht denn die shippingStatusId?

 

[OnDemand]

Hey , die ist in der Tabelle shipping_status drin und gehört zum Primary Key. Hat keinen Bezug zu einer anderen Tabelle würde ich behaupten. shippig_status_id und language_id sind der zusammengesetzte Primary key. Ist auch kein Autoincrement, seltsam. Dann müsste ich ja die ID selber hochzählen oder?

 

[mihe7]

Ich verstehe nicht ganz, wo hier ein zusammengesetzter Primärschlüssel sein soll. Wenn ID die ID des ShippingStatus ist, dann ist das der Primary Key und Language_ID ist einfach eine Referenz auf die Language-Tabelle.

 

[OnDemand]

Hab ich einen Denkfehler? Schau mal hier die Tabellenstruktur

 

[kneitzel]

Die Kernfrage ist doch:
Gibt es mehrere Einträge mit einer shipping_status_id und unterschiedlichen language_id? In meinen Augen passt das nicht ganz.

Das ist doch eher die Beziehung:
Der Shipping Status hat eine ID, die eindeutig ist (Primary key ist dann nur die id!)
Und dann hat der Shipping Status halt auch eine Sprache zugewiesen.

Wenn beides der Primary Key ist, dann wäre die Sprache auch nicht mehr zu wechseln, da sich ja sonst die id ändern würde und es damit ein anderer Datensatz wäre. Das wird doch auch nicht erwünscht sein.

 

[OnDemand]

Ja es gibt zb

1 - 1 -> sofort lieferbar
1 - 2 -> directly shipable

language_id 1 ist englisch und die 2 ist deutsch. Ich finde das auch unglücklich gelöst kann da aber nichts machen da es nicht meine Datenbank ist

 

[kneitzel]

Och jee, das Design finde ich unglücklich gewählt. Der Status selbst ist ja eher sowas wie:
1 -> Directly shipable

Und die Meldungen sind dann eher in einer weiteren Tabelle shipping_status_messages, in der jeder Eintrag eine ID als Primary Key hat und dann eben noch die shipping_status_id und language_id.
Das wäre dann eine etwas weiter normalisierte Datenbank...

Aber ok, Datenbank ist so vorgegeben.

Hast Du denn in Deinem Product auch eine language id? Oder hast Du nur die shipping_status_id?

 

[mihe7]

Man kann es natürlich auch so sehen, dass der Status nur die StatusID ist und man sich die Tabelle dazu gespart hat. Dann gibt es aus Sicht des Produkts aus keine Fremdschlüsselbeziehung (mangels Tabelle) sondern nur den Status.

 

[Thallius]

Welche Spezialist hat sch das denn ausgedacht? Eine Lokalisierung gehört doch nicht in eine Datenbank!. Wie wird denn hier eine neue Sprache hinzugefügt? Muss dann jemand die ganzen Datensätze einzippen oder wurde ein extra Programm geschrieben um das zu importieren?

Manchmal kann man wirklich nur mit dem Kopf schütteln...

 

[OnDemand]

Genau so hätte ich es auch gemacht, in den Artikelnamen haben sie es auch so gelöst, nur bei den SchippingStati ist so uncool
Das Product hat nur eine Referenz auf die shipping_status_id, keine language_id

 

[kneitzel]

Das Product hat nur eine Referenz auf die shipping_status_id, keine language_id

Dann gibt es aber doch keinen einzelnen Shipping Status sondern mehrere. Sprich: Du bekommst dann alle Stati mit ID 1 - egal welche Sprache die haben. Aber wie man so eine Beziehung darstellen kann? Ich würde das ggf. nicht mehr automatisiert versuchen, sondern die ID wirklich nur als Zahl nehmen. Und dann setzt du da halt nur noch die Zahl/ID und keine Referenz vom Shipping_Status mehr. Aber evtl. hat jemand anderes da ja noch eine andere Idee.

 

[mihe7]

Das Product hat nur eine Referenz auf die shipping_status_id, keine language_id

Tada. Wenn Du die Struktur nicht ändern kannst, dann brauchst Du Dir über Objekt-Beziehungen via JPA keine Gedanken zu machen. Speicher in Product einfach die shippingStatusID als Wert.

 

[OnDemand]

Hab es schon als int versucht, beim Laden eines Product bekomme ich dann aber die Fehlermeldung, dass mehrere Rows mit shipping_status_id 1 geladen wurden (Das Endsystem trägt je Sprache automatisch was dort ein, sodass es IMMER eine ID mit merhreren Sprachen bekommt)

Dann müsste ich das Laden der ShippingStatus komplett deaktivieren (@Transient oder wie kann ich das annotieren?) Dann hole ich mit den ShippingStatus manuell, an den Stellen wo ich ihn brauche. Genau an den Stellen kenne ich dann auch die language_id so ich dann hole ... where shipping_status_id = x AND language_id = y

 

[mihe7]

So ist es. Du brauchst ShippingStatus in Product gar nicht, sondern nur eine shippingStatusId. Dort, wo Du die Informationen zum ShippingStatus brauchst, lässt Du Dir sie separat geben. Verwende das aber nicht, um in einer Produkttabelle die Informationen des ShippingStatus anzuzeigen. Dann brauchst Du für jede Produktzeile nämlich einen weiteren SELECT (auch bekannt als das n+1-Problem).

 

[OnDemand]

Supi danke! Wie meinst du das mit der Tabelle? Hohe Ladezeit, da ein eigener Select abgegeben werden müsste? Hab ich nicht den Fall derzeit, zumindest nicht in einer Tabelle, sondern eher im Product-Detail

 

[mihe7]

Hohe Ladezeit, da ein eigener Select abgegeben werden müsste?

Genau.

 

[OnDemand]

Tip Top. Hab jetzt mein Product geändert, hat nun nur ein

@Column(name = "products_shippingtime")
private int shippingStatusId;

Somit sollte das passen oder muss ich am ShippingStatus noch was ändern?

  @Id
@Column(name = "shipping_status_id")
private int shippingStatusId;

@ManyToOne
@JoinColumn(name = "language_id")
private GLanguage language;

 

[OnDemand]

Alles paletti, klappt wie ich das will, vielen Dank für die Unterstützung Leute