Discussion :

[fvisticot]

J'ai un EJB pour un object Bookmark qui contient une collection de BookmarkEntry (un autre EJB)

Code :

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

1
2
3
4
@OneToMany(cascade={CascadeType.PERSIST, CascadeType.MERGE}, fetch=FetchType.LAZY) 
public Collection<BookmarkEntry> getEntries() { 
return _entries; 
}

Rien n'est défini au niveau de l'EJB BookmarkEntry.

Lors de l'insert d'un BookmarkEntry dans la collection trop de requetes SQL sont générées !!!

Code :

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

1
2
3
4
5
6
7
8
9
10
11
        EntityManagerFactory emf = 
            Persistence.createEntityManagerFactory(JPA_PROVIDER);
        EntityManager em = emf.createEntityManager();
        em.getTransaction().begin();
        Bookmark bookmark = em.getReference(Bookmark.class, 1);
 
        BookmarkEntry entry = new BookmarkEntry();
        entry.setUrl("URL4");
        bookmark.getEntries().add(entry);
        em.merge(bookmark);
        em.getTransaction().commit();

Hibernate: select bookmark0_.id as id8_0_, bookmark0_.location as location8_0_, bookmark0_.name as name8_0_ from Bookmark bookmark0_ where bookmark0_.id=?
Hibernate: select entries0_.Bookmark_id as Bookmark1_1_, entries0_.entries_id as entries2_1_, bookmarken1_.id as id10_0_, bookmarken1_.description as descript2_10_0_, bookmarken1_.url as url10_0_ from Bookmark_BookmarkEntry entries0_ left outer join BookmarkEntry bookmarken1_ on entries0_.entries_id=bookmarken1_.id where entries0_.Bookmark_id=?
Hibernate: insert into BookmarkEntry (description, url) values (?, ?)
Hibernate: delete from Bookmark_BookmarkEntry where Bookmark_id=?
Hibernate: insert into Bookmark_BookmarkEntry (Bookmark_id, entries_id) values (?, ?)
Hibernate: insert into Bookmark_BookmarkEntry (Bookmark_id, entries_id) values (?, ?)
Hibernate: insert into Bookmark_BookmarkEntry (Bookmark_id, entries_id) values (?, ?)
Hibernate: insert into Bookmark_BookmarkEntry (Bookmark_id, entries_id) values (?, ?)
Hibernate: insert into Bookmark_BookmarkEntry (Bookmark_id, entries_id) values (?, ?)

[TheEwook]

Salut,

A mon avis un mauvais paramétrage de ton fichier hbm correspondant?
Tu peux nous le montrer ton fichier hbm ?

Merci

[fvisticot]

J'utilise Hibernate comme provider EJB3. Je ne défini qu'un fichier persistence.xml et des annotations dans mes Entity.

Quelle est la plus value d'ajouter en plus ce fichier hbm ??? pour moi ,il n'y en a pas besoin !!!

[JeitEmgie]

J'utilise Hibernate comme provider EJB3. Je ne défini qu'un fichier persistence.xml et des annotations dans mes Entity.

Quelle est la plus value d'ajouter en plus ce fichier hbm ??? pour moi ,il n'y en a pas besoin !!!

la relation OneToMany unidirectionnelle est un classique des discussions des forums Hibernate, et pour résumer : les concepteurs d'Hibernate déconseillent toujours cette solution quand elle est utilisée sans une table de jointure (votre cas)…
et préfèrent souvent la solution bidirectionnelle…

"7.2.3. one to many
A unidirectional one-to-many association on a foreign key is a very unusual case, and is not really recommended. "

dans votre cas, passez au bidirectionnel et ajouter un "mappedBy" du côté @OneToMany
et du côté @ManyToOne ne pas oublier de mettre "nullable=false" sur la foreign key…

(tout est dans la doc Hibernate… mais avec des exemples .hbm pas d'exemple avec annotation…)

[fvisticot]

Merci pour la réponse.
Mais j'ai quand meme un peu de mal à choisir cette solution car si j'ai bien compris, le coté manyToOne est le "master" de la relation.
Donc dans mon cas, il s'agit du bookMarkEntry qui est le maitre de la relation.

Donc pour sauver les éléments que contient mon bookmark (les élements ont été ajoutés par bookmark.getEntries().add(bookmarkEntry)), nous sommes obligé de persister chaque element du bookmark:

Code :

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

1
2
3
4
5
 
for (BookmarkEntry entry: bookmark.getEntries()) {
  entry.setBookmark(bookmark)
  em.persist(entry)
}

Il aurait été plus simple que le bookmark soit le maitre de la relation et faire uniquement un

Code :

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

em.persist(bookmark);

Avez vous une solution à ce pb ?

[JeitEmgie]

Merci pour la réponse.
Mais j'ai quand meme un peu de mal à choisir cette solution car si j'ai bien compris, le coté manyToOne est le "master" de la relation.

…

le "master" ?

[djo.mos]

Bonjour,
Tu as deux méthode pour le persist:
- Soit tu déclares un cascade sur l'insert, ainsi, un simple :

Code :

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

1
2
3
 
bookmark.setEntry(entry);
entry.getBookmarks().add(bookmark)

suffit à persister le bookmark. (Mais j'aime pas trop cete approche là).
- Soit tu y vas à la dure, mais au fur et à mesure:
A chaque fois que tu as un bookmarke à persister, tu fais:

Code :

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

1
2
3
4
 
bookmark.setEntry(entry);
em.persist(bookmark);
em.persist(entry);