Discussion :

[ddxxgg]

Bonjour,

La suppression de mon entité persistante ne marche pas. Le code est :

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
 
EntityManager em = EntityManagerUtil.getEntityManagerUtil().getEntityManager();
EntityTransaction tr = em.getTransaction();
tr.begin();			
 
Contact newContact = em.find(Contact.class, contactVO.getID());
 
if (newContact == null)		    
      throw new IllegalArgumentException("Suppression impossible. Ce contact n'existe pas.");		  
else 
{
     em.remove(newContact);
}
tr.commit();
EntityManagerUtil.closeEntityManager();

"EntityManagerUtil" est une classe utilitaire qui me permet d'instancier EntityManagerFactory et EntityManager. Elle fonctionne bien (je l'utilise de la même façon dans mes ajouts, recherches et modifications sans problème).

ContactVO est une Value Object (image de l'entité Contact). Et son ID est bien renseigné !

Le problème est qu'avec ce code (qui est sensé effectuer une suppression), rien n'est supprimé de la BD. Au contraire, lors d'un check dans la BD, je me rend compte que la ligne correspondante a été mise à jour (champs NOM, PRENOM à vides).

une idée ?

Merci.

[longbeach]

Faut que tu flush l'EntityManager :

em.flush();

[ddxxgg]

Faut que tu flush l'EntityManager :

em.flush();

Salut,

Désolé de n'avoir pas indiqué quelle était mon erreur.
Au fait ce n'est pas à cause de l'absence du flush() mais plutôt une mauvaise transmission de la clé de l'entité Contact à la méthode find() :

J'avais écrit :

Code :

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

Contact newContact = em.find(Contact.class, contactVO.getID());

Alors que je devais faire :

Code :

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

Contact newContact = em.find(Contact.class, new Long(contactVO.getID()));

car la clé primaire est de type classe Long et contactVO.getID() ne retourne d'un entier.

Maintenant, on me dira mais si c'était ça alors une exception serait levée (car l'objet ne serait pas trouvé) :

Code :

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

1
2
3
 
   if (newContact == null)		    
      throw new IllegalArgumentException("Suppression impossible. Ce contact n'existe pas.");

Et je répondrais que je n'en ai aucune idée. Mais en tout cas, dès que j'ai apporté cette modification, la suppression marche (sans effectuer de flush !).

Finalement, le code fonctionnel est :

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
 
   EntityManager em = EntityManagerUtil.getEntityManagerUtil().getEntityManager();
   EntityTransaction tr = em.getTransaction();
tr.begin();			
 
Contact newContact = em.find(Contact.class, new Long(contactVO.getID()));
 
if (newContact == null)		    
      throw new IllegalArgumentException("Suppression impossible. Ce contact n'existe pas.");		  
else 
{
     em.remove(newContact);
}
tr.commit();
EntityManagerUtil.closeEntityManager();

Cordialement.

[ohanny]

Merci pour ton retour, cette astuce mérite d'être notée. Pour info, quelle implémentation de JPA utilises tu ?