Problème de persistance et de cascade

Version imprimable

13/06/2011, 23h23
chatis

Problème de persistance et de cascade
Bonjour,

j'ai une classe Personne qui est liée à une classe Adresse avec une annotation @OneToMany comme ci-dessous :
Code:

1 2 3 @OneToMany(mappedBy="personne", cascade= CascadeType.ALL) private List<Adresse> adresses;
Sur la base de donnée, c'est la table Adresse qui contient un IDPersonne car je dis que une adresse appartient à une seule personne, mais une personne peut avoir plusieurs adresse.

Sur mon serveur RESTFul, j'ai une méthode @Post qui reçoit en paramètre une Personne
comme ci-dessous :
Code:

1 2 3 4 5 6 7 8 9 10 11 <personne> <nom>asd</nom> ... <adresses> <adresse> <rue>asdf</rue> .... </adresse> </adresse> </personne>
et j'aimerais bien pouvoir ajouter les personnes et les adresses en 1 seule commande. Actuellement je fais.
Code:

1 2 em.persist(personne);
La personne est enregistrées, les adresses aussi.

Mais le problème c'est que le champ IDPersonne de ma table adresse reste null.

Il y a t'il un moyen de faire comme ca ou je suis obligé de faire des requêtes différentes ?

Merci.
14/06/2011, 11h33
DevServlet

Peut on voir le code des classes entités Personne et Adresses?

Voila pour personne :

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
 
/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package ch.plugin.sportreserv.entity;
 
import java.io.Serializable;
import java.util.Date;
import java.util.List;
import javax.persistence.Basic;
import javax.persistence.CascadeType;
import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.FetchType;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.JoinColumn;
import javax.persistence.ManyToMany;
import javax.persistence.ManyToOne;
import javax.persistence.NamedQueries;
import javax.persistence.NamedQuery;
import javax.persistence.OneToMany;
import javax.persistence.OneToOne;
import javax.persistence.Table;
import javax.persistence.Temporal;
import javax.persistence.TemporalType;
import javax.xml.bind.annotation.XmlRootElement;
import javax.xml.bind.annotation.XmlTransient;
 
/**
 *
 * @author Chatis
 */
@Entity
@Table(name = "personne")
@XmlRootElement
@NamedQueries({
    @NamedQuery(name = "PersonneByNumReservation", query = "SELECT p FROM Personne p, Personnereservation pr WHERE pr.personnereservationPK.numReservation=:NumReservation"),
    @NamedQuery(name = "PersonneByLangue", query = "SELECT p FROM Personne p WHERE langue=:IDLangue"),
    @NamedQuery(name = "PersonneByTypeReservation", query = "SELECT p FROM Personne p WHERE p.type=:IDType")})
public class Personne implements Serializable {
    private static final long serialVersionUID = 1L;
    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    @Basic(optional = false)
    @Column(name = "IDPersonne")
    private Integer iDPersonne;
    @Column(name = "Titre", length=10)
    private String titre;
    @Column(name = "Nom", length=20)
    private String nom;
    @Column(name = "Prenom",length=20)
    private String prenom;
    @Column(name = "Telephone", length=20)
    private String telephone;
    @Column(name = "DateNaissance")
    @Temporal(TemporalType.DATE)
    private Date dateNaissance;
    @Column(name = "aModifieProfil")
    private Boolean aModifieProfil;
    @Column(name = "Email", length=50)
    private String email;
    @Column(name = "CarteRFID", length=16)
    private String carteRFID;
    @ManyToOne(cascade= CascadeType.ALL)
    @JoinColumn(name = "IDLangue")
    private Langue langue;
    @OneToMany(mappedBy="personne", cascade= CascadeType.ALL)
    private List<Adresse> adresses;
    @OneToOne(mappedBy="personne", cascade= CascadeType.ALL)
    private User user;
 
    @OneToMany(mappedBy = "personne")
    private List<Personnereservation> personneReservations;
 
    @OneToMany(mappedBy = "personne", cascade= CascadeType.ALL)
    private List<Heureadeduire> heureADeduires;
 
    @ManyToMany(mappedBy="personnes",cascade= CascadeType.ALL)
    private List<Typereservation> type;
 
    public Personne() {
    }
 
    public Personne(Integer iDPersonne) {
        this.iDPersonne = iDPersonne;
    }
 
    public Integer getIDPersonne() {
        return iDPersonne;
    }
 
    public void setIDPersonne(Integer iDPersonne) {
        this.iDPersonne = iDPersonne;
    }
 
    public String getNom() {
        return nom;
    }
 
    public void setNom(String nom) {
        this.nom = nom;
    }
 
    public String getPrenom() {
        return prenom;
    }
 
    public void setPrenom(String prenom) {
        this.prenom = prenom;
    }
 
    public String getTelephone() {
        return telephone;
    }
 
    public void setTelephone(String telephone) {
        this.telephone = telephone;
    }
 
    public Date getDateNaissance() {
        return dateNaissance;
    }
 
    public void setDateNaissance(Date dateNaissance) {
        this.dateNaissance = dateNaissance;
    }
 
    public String getEmail() {
        return email;
    }
 
    public void setEmail(String email) {
        this.email = email;
    }
 
    public String getCarteRFID() {
        return carteRFID;
    }
 
    public void setCarteRFID(String carteRFID) {
        this.carteRFID = carteRFID;
    }
 
    public Langue getLangue() {
        return langue;
    }
 
    public void addAdresse(Adresse adresse){
        adresses.add(adresse);
        adresse.setPersonne(this);
    }
 
    public void setLangue(Langue langue) {
        this.langue = langue;
    }
 
    public User getUser() {
        return user;
    }
 
    public void setUser(User user) {
        this.user = user;
    }
 
    @XmlTransient
    public List<Personnereservation> getPersonneReservations() {
        return personneReservations;
    }
 
    public void setPersonneReservations(List<Personnereservation> personneReservations) {
        this.personneReservations = personneReservations;
    }
 
    @XmlTransient
    public List<Heureadeduire> getHeureADeduires() {
        return heureADeduires;
    }
 
    public void setHeureADeduires(List<Heureadeduire> heureADeduires) {
        this.heureADeduires = heureADeduires;
    }
 
    @XmlTransient
    public List<Typereservation> getType() {
        return type;
    }
 
    public void setType(List<Typereservation> type) {
        this.type = type;
    }
 
    public List<Adresse> getAdresses() {
        return adresses;
    }
 
    public void setAdresses(List<Adresse> Adresses) {
        this.adresses = Adresses;
    }
 
    public Boolean getaModifieProfil() {
        return aModifieProfil;
    }
 
    public void setaModifieProfil(Boolean aModifieProfil) {
        this.aModifieProfil = aModifieProfil;
    }
 
    public String getTitre() {
        return titre;
    }
 
    public void setTitre(String titre) {
        this.titre = titre;
    }
 
    @Override
    public int hashCode() {
        int hash = 0;
        hash += (iDPersonne != null ? iDPersonne.hashCode() : 0);
        return hash;
    }
 
    @Override
    public boolean equals(Object object) {
        // TODO: Warning - this method won't work in the case the id fields are not set
        if (!(object instanceof Personne)) {
            return false;
        }
        Personne other = (Personne) object;
        if ((this.iDPersonne == null && other.iDPersonne != null) || (this.iDPersonne != null && !this.iDPersonne.equals(other.iDPersonne))) {
            return false;
        }
        return true;
    }
 
    @Override
    public String toString() {
        return "ch.plugin.sportreserv.entity.Personne[ iDPersonne=" + iDPersonne + " ]";
    }
 
}

et voila pour adresse :

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
 
/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package ch.plugin.sportreserv.entity;
 
import java.io.Serializable;
import javax.persistence.Basic;
import javax.persistence.CascadeType;
import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.JoinColumn;
import javax.persistence.ManyToOne;
import javax.persistence.NamedQueries;
import javax.persistence.NamedQuery;
import javax.persistence.Table;
import javax.xml.bind.annotation.XmlRootElement;
import javax.xml.bind.annotation.XmlTransient;
 
/**
 *
 * @author Chatis
 */
@Entity
@Table(name = "adresse")
@XmlRootElement
@NamedQueries({
    @NamedQuery(name = "AdresseByPersonne", query = "SELECT a FROM Adresse a WHERE a.personne=:IDPersonne"),
    @NamedQuery(name = "AdresseByLocalite", query = "SELECT a FROM Adresse a WHERE a.localite=:IDLocalite")})
public class Adresse implements Serializable {
    private static final long serialVersionUID = 1L;
    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    @Basic(optional = false)
    @Column(name = "IDAdresse")
    private Integer iDAdresse;
    @Column(name = "Adresse1", length = 40)
    private String adresse1;
    @Column(name = "Adresse2", length = 40)
    private String adresse2;
    @Column(name = "TypeAdresse", length = 20)
    private String typeAdresse;
    @Column(name = "isPrincipal")
    private Boolean isPrincipal;
    @ManyToOne
    @JoinColumn(name ="IDPersonne")
    private Personne personne;
    @ManyToOne(cascade= CascadeType.ALL)
    @JoinColumn(name="IDLocalite")
    private Localite localite;
 
    public Adresse() {
    }
 
    public Adresse(Integer iDAdresse) {
        this.iDAdresse = iDAdresse;
    }
 
    public Integer getIDAdresse() {
        return iDAdresse;
    }
 
    public void setIDAdresse(Integer iDAdresse) {
        this.iDAdresse = iDAdresse;
    }
 
    public String getAdresse1() {
        return adresse1;
    }
 
    public void setAdresse1(String adresse1) {
        this.adresse1 = adresse1;
    }
 
    public String getAdresse2() {
        return adresse2;
    }
 
    public void setAdresse2(String adresse2) {
        this.adresse2 = adresse2;
    }
 
    public String getTypeAdresse() {
        return typeAdresse;
    }
 
    public void setTypeAdresse(String typeAdresse) {
        this.typeAdresse = typeAdresse;
    }
 
    public Boolean getIsPrincipal() {
        return isPrincipal;
    }
 
    public void setIsPrincipal(Boolean isPrincipal) {
        this.isPrincipal = isPrincipal;
    }
 
    public Localite getLocalite() {
        return localite;
    }
 
    public void setLocalite(Localite localite) {
        this.localite = localite;
    }
 
    @XmlTransient
    public Personne getPersonne() {
        return personne;
    }
 
    public void setPersonne(Personne personne) {
        this.personne = personne;
    }
 
    @Override
    public int hashCode() {
        int hash = 0;
        hash += (iDAdresse != null ? iDAdresse.hashCode() : 0);
        return hash;
    }
 
    @Override
    public boolean equals(Object object) {
        // TODO: Warning - this method won't work in the case the id fields are not set
        if (!(object instanceof Adresse)) {
            return false;
        }
        Adresse other = (Adresse) object;
        if ((this.iDAdresse == null && other.iDAdresse != null) || (this.iDAdresse != null && !this.iDAdresse.equals(other.iDAdresse))) {
            return false;
        }
        return true;
    }
 
    @Override
    public String toString() {
        return "ch.plugin.sportreserv.entity.Adresse[ iDAdresse=" + iDAdresse + " ]";
    }    
}

14/06/2011, 13h31
-gma-
Bonjour,

est ce que après avoir unmarshallé ton XML tu as bien l'assertion suivante ?
Code:

1 2 3 for(Adresse adresse : personne.getAdresses() { assert adresse.getPersonne() == personne; }
car c'est bien la relation "personne" de "adresse" qui mappe la relation.
14/06/2011, 13h35
chatis
Je n'ai pas très bien compris...

je dois faire quoi avec cette ligne de code ?

quelque-chose comme :
Code:

1 2 3 4 5 6 em.persist(personne) for(Adresse adresse : personne.getAdresses() { assert adresse.getPersonne() == personne; }
c'est ca ? Mais dans ce cas ca ne fait pas "automatiquement" non ?
14/06/2011, 13h41
-gma-
Non en fait je me demande si lorsque le XML

Code:

<personne>...</personne>

est utilisé pour créer les objets, les relations bidirectionnelles sont elles bien renseignées ?

Donc soit tu regardes en debug, soit tu rajoutes une trace comme celle là par exemple:
Code:

1 2 3 4 5 for(Adresse adresse : personne.getAdresses() { System.out.println(adresse.getPersonne()); // ne doit pas afficher "null" } em.persist(personne)
14/06/2011, 13h43
DevServlet

Peut on voir là où tu rajoutes les adresses à l'entité personne avant de persister la personne proprement dite? ton mapping a l'air bon.
14/06/2011, 13h52
chatis
Citation:
Envoyé par -gma-

Donc soit tu regardes en debug, soit tu rajoutes une trace comme celle là par exemple:

Code:

1 2 3 4 5 for(Adresse adresse : personne.getAdresses() { System.out.println(adresse.getPersonne()); // ne doit pas afficher "null" } em.persist(personne)
alors j'ai un null qui ressort. Ca veut dire quoi ?
Est-ce que cela peut provenir de mon annotation @XmlTransient en dessus de getPersonne() dans Adresse ?

EDIT : j'ai enlevé le @XmlTransient et ça ne change rien
14/06/2011, 13h55
chatis

Citation:

Envoyé par DevServlet

Peut on voir là où tu rajoutes les adresses à l'entité personne avant de persister la personne proprement dite? ton mapping a l'air bon.

Mais je ne rajoutes pas les adresses aux personnes. Ma méthode RESTFul récupère une paramètre Personne depuis mon XML qui contient les adresses. Après je lui demande de persister la personne. Et normalement il faite automatiquement le mapping non ?
14/06/2011, 14h02
-gma-

Citation:

Envoyé par chatis

alors j'ai un null qui ressort. Ca veut dire quoi ?

Ca veut dire que ton association bidirectionnelle n'est renseignée que d'un côté (de Personne vers Adresse) et manque de bol JPA se base sur l'attribut "personne" de la classe "Adresse" pour savoir si le lien est fait entre deux instances.
Et même si tu avais l'inverse ça poserait quand même un problème de sémantique puisque si la relation est bidirectionnelle on doit pouvoir la naviguer dans les deux sens.
Tu peux également trouver une explication à ce problème ici (la problématique est la même avec Hibernate et JPA)

Citation:

Envoyé par chatis

Est-ce que cela peut provenir de mon annotation @XmlTransient en dessus de getPersonne() dans Adresse ?

Oui je pense que c'est le cas.

donc ce que tu essayes de me dire en gros c'est que je ne peux rien faire ????

Car j'ai bien du bidirectionnelle et même du protégé selon le lien que tu m'as envoyé...

Personne :
Code:

1 2 3 4 5 6 7 8 9 ... @OneToMany(mappedBy="personne", cascade= CascadeType.ALL) private List<Adresse> adresses; ... public void addAdresse(Adresse adresse){ this.adresses.add(adresse); adresse.setPersonne(this); }
et du côté adresse :
Code:

1 2 3 4 5 @ManyToOne @JoinColumn(name ="IDPersonne") private Personne personne;
et j'ai enlevé le @XmlTransient mais ca ne change rien

14/06/2011, 14h28
-gma-
C'est bien mais lorsque tes objets sont créés à partir de l'XML ce n'est pas la méthode "addAdresse" qui est appelée...

Citation:

Envoyé par chatis

donc ce que tu essayes de me dire en gros c'est que je ne peux rien faire ????

Non en fait tu as plusieurs solutions:
1. tu peux renseigner les relations "personne" à la main
  Code:
  
  1 2 3 4 5 for(Adresse adresse : personne.getAdresses() { adresse.setPersonne(personne)); } em.persist(personne)
  mais ça n'est pas très élégant;
2. tu essayes de trouver une extension qui le fait pour toi (par exemple, si ça correspond aux implémentations que tu utilises)
Mais dans tous les cas c'est plus un problème de déserialisation XML que de JPA.
14/06/2011, 15h10
chatis

Ok je vois ca... Mais par contre j'utilise NetBeans... Connaissez-vous son équivalent pour ca? encore merci
15/06/2011, 22h12
chatis
chui pas très fort peut être.. mais je ne comprends pas trop le truc eclipseLink... comment je peux l'installer sur mon projet ? Car j'ai cru que c'était que pour Eclipse, mais aujourd'hui sur mon netbeans, j'ai vu des fichiers qui était autmatiquement géré avec cette annotation :
Code:

1 2 @Generated(value="EclipseLink-2.2.0.v20110202-r8913", date="2011-06-15T22:03:52")
Alors je ne comprends pas trop.. car je pensais utilisé JPA et Hibernate. Mais pas de EclipseLink...
16/06/2011, 10h54
-gma-

En fait EclipseLink c'est un ORM qui fournit des implémentations de JPA, JAXB et autre. Hibernate fournit aussi une implémentation de JPA et c'est apriori celle que tu utilises dans ton projet.

Mais ton problème se situe plus du côté de JAXB, où tu utilises peut être EclipseLink ? Sur quels fichiers as tu l'annotation @Generated ?

Oui il me semble que j'utilise Hibernate JPA Mais je ne comprend pas pourquoi j'ai EclipseLink également...

l'annotation ce situe dans un package qui est créé lorsque je fait un clean and build de mon projet. Le package s'appelle "Generated sources (ap-sources-output)" et il comprend un sous package avec mes entitées.

Exemple de ce qui est créé :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 package ch.plugin.sportreserv.entity; import ch.plugin.sportreserv.entity.Personne; import javax.annotation.Generated; import javax.persistence.metamodel.SingularAttribute; import javax.persistence.metamodel.StaticMetamodel; @Generated(value="EclipseLink-2.2.0.v20110202-r8913", date="2011-06-16T14:09:29") @StaticMetamodel(Compagnie.class) public class Compagnie_ { public static volatile SingularAttribute<Compagnie, Personne> Personne; public static volatile SingularAttribute<Compagnie, String> imgCompagnie; public static volatile SingularAttribute<Compagnie, String> nom; }
Mais sinon, j'ai enfin réussi à ajouter la libraire EclipseLink dans mon projet... mais quand je rajoute @XmlInverseReference... Il n'y a aucune différence....

Ou est-ce que je dois la mettre dans l'exemple ci-dessous ? :
Code:

1 2 3 4 5 6 7 public class Typereservation implements Serializable { ... @OneToMany(fetch= FetchType.EAGER, mappedBy="possibilitePK.typeReservation") private List<Possibilite> possibilites; ... }
Code:

1 2 3 4 5 6 7 8 9 10 11 @Embeddable public class PossibilitePK implements Serializable { ... @ManyToOne @XmlInverseReference(mappedBy="possibilites") private Typereservation typeReservation; ... }
enfin c'est juste comme j'ai mis la ?

17/06/2011, 12h04
-gma-
Oui, apparemment c'est ça, ou si on reprend le problème initial:
Code:

1 2 3 4 5 6 class Adresse { @ManyToOne @JoinColumn(name ="IDPersonne") @XmlInverseReference(mappedBy="addresses") // ajout private Personne personne; }
Ensuite il faut être sûr que c'est bien le JaXB de EclipseLink (MOXy) qui est utilisé.
17/06/2011, 20h59
chatis

car yen a d'autres?

celui que j'ai il s'appelle eclipselink-2.2.0.jar et eclipselink-javax.persistence-2.0.jar

car ce que j'ai mis ca change strictement à rien...... c'est sensé résoudre le mapping inverse ? et ca marche ?

car dans mon code j'ai beau avoir :
Code:

1 2 3 4 5 @ManyToOne @JoinColumn(name ="IDPersonne") @XmlInverseReference(mappedBy="adresses") private Personne personne;
dans mon manager je suis obligé de faire :
Code:

1 2 3 for(Adresse a : personne.getAdresses()) a.setPersonne(personne);
sinon l'adresse ne se rajoute pas...