Persistance en cascade

Version imprimable

22/11/2007, 16h36
aloha

Persistance en cascade
Bonjour,

Je recontre des problème lors de la création d'enregistrements avec des EJB3.0+JPA.

Description:
Une entité A a pour attribut un set d'entité B.
L'entité B est un objet qui mappe une table d'association entre la table A(mappée par l'objet A) et la table C(mappée par l'objet C).

Lorsque je veux persister une entité A j'obtiens le message d'erreur suivant:
Code:

1 2 org.apache.openjpa.lib.jdbc.ReportingSQLException: [BEA][Oracle JDBC Driver][Oracle]ORA-02291: integrity constraint (FK_B_A) violated - parent key not found
L'enregistrement dans la table B ne passe pas car il ne détecte pas que le parent A a été créé comme si les deux inserts se faisaient dans des transactions différentes.

Merci pour votre aide.
22/11/2007, 16h45
mow

Salut,

Je n'ai peut être pas la solution mais ci-dessous quelques pistes de recherche :

1) Les 2 transactions ont-elles lieu dans la même session factory ?

2) es-tu sûr que ton mapping est bien fait ? Pour vérifier, insère toi même des données dans tes tables A et B et vérifie si tu les récupères dans ton application...

Bon courage,
Mow
22/11/2007, 16h48
djo.mos

Citation:

Une entité A a pour attribut un set d'entité B.
L'entité B est un objet qui mappe une table d'association entre la table A(mappée par l'objet A) et la table C(mappée par l'objet C).

:koi: ....

Si je comprends bien, si tu ajoutes un B à l'attribut Set de A tu obtiens cette erreur ?
Comment tu as déclaré la relation entre A et B ? est elle bi-diréctionnelle ou pas ? dans quel ordre tu persistes tes données ?
:arrow: Bref, quelques extraits de code s'il te plaît.

C'est parti pour le code.

Classe A(ProductList):

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public class ProductList implements java.io.Serializable {
...
private Set<TfcList> tfcLists = new HashSet<TfcList>(0);
...
	@OneToMany(cascade = { CascadeType.ALL }, fetch = FetchType.LAZY, mappedBy = "productList")
	public Set<TfcList> getTfcLists() {
		return this.tfcLists;
	}
 
	public void setTfcLists(Set<TfcList> tfcLists) {
		this.tfcLists = tfcLists;
	}
...

Classe B(TfcList) qui est une classe d'association

Code:

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
29
30
31
32
33
34
35
36
37
38
39
 
public class TfcList implements java.io.Serializable {
...
	private TfcListId id;
	private ProductList productList;
	private Tfc tfc;
...
	@EmbeddedId
	@AttributeOverrides( {
			@AttributeOverride(name = "productListId", column = @Column(name = "PRODUCT_LIST_ID", unique = false, nullable = false, insertable = true, updatable = true, precision = 8, scale = 0)),
			@AttributeOverride(name = "tfcId", column = @Column(name = "TFC_ID", unique = false, nullable = false, insertable = true, updatable = true, precision = 8, scale = 0)) })
	public TfcListId getId() {
		return this.id;
	}
 
	public void setId(TfcListId id) {
		this.id = id;
	}
 
	@ManyToOne(cascade = {}, fetch = FetchType.LAZY)
	@JoinColumn(name = "PRODUCT_LIST_ID", unique = false, nullable = false, insertable = false, updatable = false)
	public ProductList getProductList() {
		return this.productList;
	}
 
	public void setProductList(ProductList productList) {
		this.productList = productList;
	}
 
	@ManyToOne(cascade = {}, fetch = FetchType.LAZY)
	@JoinColumn(name = "TFC_ID", unique = false, nullable = false, insertable = false, updatable = false)
	public Tfc getTfc() {
		return this.tfc;
	}
 
	public void setTfc(Tfc tfc) {
		this.tfc = tfc;
	}
...

Code de l'ejb de création:

Code:

1
2
3
4
5
6
7
8
 
ProductList transientInstance = new ProductList();
Set<TfcList> tfcLists = new HashSet<TfcList>(0);
/*Alimentation de la liste
Code d'alimentation
*/
transientInstance.setTfcLists(tfcLists);
entityManager.persist(transientInstance);

Je précise qu'en consultation cela fonctionne.

J'ai comme l'impression que le dernier bout de code (le plus intéressant) n'est pas complet.
Mais en tout cas, il ne fait pas ecraser le champ de type Set de ProductList : :nono:.
Il faut plutôt y aller par:
Code:

1 2 transientInstance.getTfcLists().addAll(tfcLists);
Cette règle est générale. Il faut instancier la collection dans le constructeur et ne plus écraser cete instance, mais plutôt la modifier.

Sinon, dans le code d'alimentation, t'es sur d'avoir assigné le champ ProductList de toutes les instances de transientInstance ? Normalement, c'est ça la cause de tes problèmes.

De plus, pour pouvoir faire ça, il te faut d'abord persister le type parent avant de spécifier ses fils, sinon, les fils pointeront vers un parent non persisté.

Je sais que j'explique comme une patate, voici donc un petit exemple:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 ProductList transientInstance = new ProductList(); entityManager.persist(transientInstance);//<-Premier ajout /*Alimentation de la liste */ TfcList x1 = new TfcList(); x1.setProductList(transientInstance);//<-Très important: faire pointer le fils vers le parent transientInstance.getTfcLists().add(x1); //de même pour les autres instances de TfcList entityManager.persist(transientInstance);
En espérant que ce soit utile.
Bonne chance.

22/11/2007, 22h39
aloha

Tout d'abord merci pour ces conseils.

Citation:

Envoyé par djo.mos

...Sinon, dans le code d'alimentation, t'es sur d'avoir assigné le champ ProductList de toutes les instances de transientInstance ? Normalement, c'est ça la cause de tes problèmes...

Tu peux préciser stp, je ne vois pas ce que tu veux dire.
23/11/2007, 00h04
djo.mos

Bonjour.
En fait, il aurait suffit que tu ailles plus loin dans ma réponse jusqu'au bout de code ;)
Il s'agit exactement de la ligne:

Code:

x1.setProductList(transientInstance);//<-Très important: faire pointer le fils vers le parent

Pour résumer, voici ce qu'il faut faire pour lier une entité fille B à une entité parente A:
:arrow: Persister A.
:arrow: Faire le lien de B vers A (b.setParent(A))
:arrow: Persister B
:arrow: Ajouter B à l'ensemble des fils de A (a.getFils().add(b) ). Si on a Cascade Persist, alors l'étape précédente devient inutile, voire source de problèmes.

Bonne chance.

Voici le code de ma méthode buisiness
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 entityManager.persist(transientInstance); Set<TfcList> tfcLists = new HashSet<TfcList>(0); //TFCs from client Set <TfcDto> tfcs = productList.getTfcs(); Iterator<TfcDto> it = tfcs.iterator(); while (it.hasNext()) { TfcDto elem = it.next(); Tfc tfc = entityManager.find(Tfc.class, elem.getTfcId()); TfcList tfcList = new TfcList(); tfcList.setTfc(tfc); tfcList.setProductList(transientInstance); tfcLists.add(tfcList); } transientInstance.getTfcLists().addAll(tfcLists); entityManager.persist(transientInstance);
Qui génère toujours la même exception:

Caused by: <0|false|0.9.7> org.apache.openjpa.persistence.PersistenceException: [BEA][Oracle JDBC Driver][Oracle]ORA-02291: integrity constraint (EFNTA.FK_TFCLIST_PRODUCTLIST) violated - parent key not found
{prepstmnt 33 INSERT INTO EFNTA.TFC_LIST (PRODUCT_LIST_ID, TFC_ID) VALUES (?, ?) [params=(long) 0, (long) 0]} [code=2291, state=23000]
FailedObject: lu.efa.ejb.generated.TfcList@15aa488

Je suis en type cascade=ALL

30/11/2007, 12h39
aloha
Je reviens un peu tardivement sur ce post.
La solution a mon problème est la suivante:
Il fallait setter l'id composite de la facon suivante:
Code:

1 2 3 4 tfcList.setId( new TfcListId(transientInstance.getProductListId(),elem.getTfcId()) );
Je pensais que le mapping le faisait automatiquement.
Merci pour votre aide.