Discussion :

[chido]

Bonjour à tous,

Encore en pleine galère sur JPA et les relations ! Malgré mes recherches sur le net depuis des heures je ne trouve pas les cas dans lesquels je suis.

Actuellement j'ai ce message d'erreur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

More than one row with the given identifier was found: 000002, for class: com.icl.m3.model.OrderLine

Le message d'erreur est du au faite que j'ai 2 orderLine avec un BOOKS_ID similaire (000002) MAIS ces 2 orderLine on un ID différent donc ça devrait passer.

Voici mes tables abrégées :

table BOOKS :
- ISBN13(string)
- UNIT_PRICE

table ORDER_LINES:
- ID (PK)
- BOOKS_ID(string)

Je veux pouvoir faire en sorte que :
- 1 ligne de commande ne peut avoir qu'un livre
- 1 livre peut être sur plusieurs ligne de commande
- Relation imposée : OneToOne

Je veux pouvoir également :
- Récupérer mon livre via ma ligne de commande

Voici ma classe de book:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
@Entity
@XmlRootElement
@Table(name = "BOOKS")
public class Book implements Serializable {
 
	private static final long serialVersionUID = 1L;
 
	// ////////////////Colonnes//////////////////
 
	@Id
	@NotNull
	@Length(max = 17, message = "Référence trop grande")
	@Column(name = "ISBN13")
	private String isbn;
 
	@Column(name = "UNIT_PRICE")
	private Double unitPrice;
 
 
	// ////////////////Relations//////////////////
 
	// 06
	@OneToOne(mappedBy = "book")
	private OrderLine orderLine;

Voici ma classe OrderLine :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
@Entity
@XmlRootElement
@Table(name = "ORDER_LINES")
public class OrderLine implements Serializable {
 
	private static final long serialVersionUID = 1L;
 
	// ////////////////Colonnes//////////////////
 
	@Id
	@NotNull
	@GeneratedValue(strategy = GenerationType.IDENTITY)
	@Column(name = "ID")
	private Integer id;
 
	//@NotNull
	//@Length(max = 17)
	//@Column(name = "BOOKS_ID")
	//private String booksIsbn13;
 
 
	// ////////////////Relations//////////////////
 
	// 06
	// Création/liaison du champs BOOKS_ID dans la table OrderLine pointant vers l'ID ISBN13 de la table BOOKS
	@OneToOne
	@JoinColumn(name = "BOOKS_ID", referencedColumnName = "ISBN13")
	private Book book;

Je voudrais aussi pouvoir déclarer ma @Column(name = "BOOKS_ID") afin de lui mettre les annotations que vous pouvez déjà voir en commentaire (Sauf si il y a une autre manière de faire).

Merci d'avance

[Pill_S]

Hello,

La relation one to one ne peut pas fonctionner. Tu ne peux pas réutiliser les tuples de la table BOOKS avec one to one car... UN livre ne peut appartenir qu'à UNE SEUL ligne de commande. C'est donc du many to one (un même livre peut appartenir à plusieurs lignes de commande).

De plus, je vois que tu as mis l'annotation @XmlRootElement, probablement pour exporter tes entités directement en xml ou json. C'est assez mauvais, de s'asspuyer sur des beans persistés pour en faire le modèle sérialisé directement, mieux vaut créer une classe dédiée à la sérialisation xml/json et faire un mapping entre les 2 (ça permet de découpler la base et l'exposition des services web)

[chido]

Merci de la confirmation Pill_S, j'avais pensé à cette éventualité comme quoi le OneToOne me trompé... tant pis je change le cahier des charges

L'annotation @XmlRootElement a était généré par défaut avec mon projet de base donc je l'ai laissé dans mes entités. Je n'exporte rien en xml ou json, je vais donc les supprimer

Serais-tu comment faire pour créer ma colonne :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
@NotNull
@Length(max = 17)
@Column(name = "BOOKS_ID")
private String booksIsbn13;

et faire la jointure OneToMany pointant vers cette colonne ?
Car avec le OneToOne j'ai du mettre le code ci-dessus en commentaire car le

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

@JoinColumn(name = "BOOKS_ID", referencedColumnName = "ISBN13")

n'accepté pas le fait que BOOKS_ID existe déjà.

[Pill_S]

Hello,

en jpa, tu ne référence jamais les clés étrangères, mais plutôt les instances correspondant à la table pointée par une clé étrangère. Donc bookIsbn13 devrait être de type book. Si tu veux l'isbn, tu fais book.getIsbn()

Donc une relation de ce genre:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
@ManyToOne(fetch=FetchType.LAZY)
@JoinColumn("BOOKS_ID")
private Book book;

[chido]

Donc je devrais mettre tout ça dans ma classe orderLine ? :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
class....
....
@NotNull
@Length(max = 17)
@Column(name = "BOOKS_ID")
private String booksIsbn13;
 
@ManyToOne(fetch=FetchType.LAZY)
@JoinColumn("BOOKS_ID")
private Book book;
......

Mais comment, en base, l'orderLine va comprendre que le contenu de sa colonne c'est bien le contenu de la colonne ISBN13(PK) de ma table BOOK ?

[Cafeinoman]

C'est tout le principe de jpa, passer du relationnel à l'objet. Il comprend comme un grand que la référence à book doit être traduite dans la table par l'id de book! Le nom book_id que tu met dans l'annotation ne sert qu'à nommer la colonne qui stocke la référence.
Par contre, comme tu as plusieurs orderline possible pour un book, tu dois avoir me semble t il :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
@ManyToOne(fetch=FetchType.LAZY)
@JoinColumn("BOOKS_ID")
private Book book;

et :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
@OneToMany(mappedBy="book")
@JoinColumn ("ORDERLINE_ID")
private Set<Orderline> orderlines;

Bon courage.

[Pill_S]

Donc je devrais mettre tout ça dans ma classe orderLine ? :

Non. Tu enlèves complètement la déclaration du champ "bookIsbn13". Tu ne références *pas* les clés étrangères, mais *que* les instances référencées. Ne pense plus relationnel avec entités, relations et clés. Penses en objet, avec instances et références entre ces instances.