Discussion :

[jdev23]

Salut;
j'ai voulu mettre à jour un enregistrement de la table user,le code de la fct modifier :

Code :

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

1
2
3
4
5
6
7
8
 
 org.hibernate.Session session = HibernateUtil.currentSession();
org.hibernate.Query query = session.createQuery("from user as u where u.login='"+d.getLogin()+"'");
User u= (User)query.list().get(0);            
org.hibernate.Transaction tx = session.beginTransaction();
u=d;
session.save(u) ;
tx.commit();

Alors que l'objet "d" est lié aux zones de textes ou normalement l'utilisateur va saisir les nouvelles valeurs de l'enregistrement.

j'ai essayé avec cette fct mais ça ne marche pas,je sais pas ou j'ai commis l'erreur.

[rhani-sama]

Essaie de commencer la transaction avant d'exécuter ta requête HQL

Code :

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

1
2
3
4
5
6
7
org.hibernate.Session session = HibernateUtil.currentSession();
org.hibernate.Transaction tx = session.beginTransaction();
org.hibernate.Query query = session.createQuery("from user as u where u.login='"+d.getLogin()+"'");
User u= (User)query.list().get(0);            
u=d;
session.save(u) ;
tx.commit();

[jdev23]

je l'ai fait mais toujours rien

[aperrin]

Recopie les valeur de d dans u (pas d'affectation de référence). Sinon il y a fort à parier qu'hibernate insert une nouvelle donnée.

[OButterlin]

Qu'est ce qui ne marche pas au juste ?
As-tu une erreur, une log, quelque chose à nous montrer ?

J'ai déjà repéré une erreur dans ta requête, il faut faire

Code :

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

1
2
session.createQuery("from User where login = ...");

Pour bien faire, il serait préférable d'écrire

Code :

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

1
2
session.createQuery("from User where login = :login").setString("login", d.getLogin());

Et à priori, je suppose que login est la clé primaire User, il faudrait utiliser l'accès direct

Code :

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

1
2
 
User u = session.get(User.class, d.login());

A+

[slevy]

jdev23, pourquoi tu fais

Code :

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

1
2
 
u=d;

??

Ca sert à quoi ?

[jdev23]

Merci pour vos réponse,

j'ai pas de messages d'erreur mais lorsque je clique la première fois sur le bouton modifier,j'ai une exception "violation de la clé primaire,objet deja existe dans la BD" alors lors du deuxième clic la moification se fait correctement.


pour la ligne u=d; c'est pour affecter les nouvelles valeur à l'enregistrement u,sachant que l'objet d est lié aux zones de textes ou l'utilisateur va saisir les nouvelles valeurs

[aperrin]

u=d fait une affectation par référence. Pour une recopie de donnée il vaut mieux utiliser la bibliothèque apache bean utils

[jdev23]

il vaut mieux utiliser la bibliothèque apache bean utils


j'ai pas compris !

[slevy]

J'ai l'impression que tu mélanges plusieurs choses.

Code :

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

1
2
 
User u= (User)query.list().get(0);

Là u est de type User qui est une entité Hibernate.

En suite tu fais :

Code :

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

1
2
 
u=d;

Je comprends toujours pas pourquoi tu écrases tout de suite la valeur que tu viens de récupérer de la base pour u ??

Mais bref, si ce User d est un objet détaché qui vient de l'interface utilisateur, il doit être réattaché (et sauvegardé en base) avec la méthode saveOrUpdate().

[jdev23]

j'écrase la valeur que je vien de récupérer de la BD pour modifier l'enregistrement u et mettre à sa place l'enregistrement d,avec les valeurs de d sont des valeurs qui viennent de l'interface utilisateur

[OButterlin]

Ce n'est pas comme ça qu'on procède.
Il faut copier les propriétés de d vers celles de u et persister u.
Il y a plusieurs possibilités pour ça :
- save()
- update()
- merge()
- saveOrUpdate()
(voir la doc pour les différences)

u est une instance de User, d est une instance de quoi ?

A+

[jdev23]

d est aussi une instance de type User,je sais pas mai est ce que le passage par référence ne va copier les prtés de l'bjet d dans u ?

[OButterlin]

Ben non...

Si tu veux, u contient l'adresse d'un objet User et d aussi.
Quand tu fais u = d, tu fais "pointer" (la référence) u vers l'objet "pointé" par d.
En aucun cas tu ne copies les propriétés de l'objet (pointé par) d vers l'objet (pointé par) u
Pour cela, il faut utiliser BeanUtils.copyProperties(objetCible, objetSource)

(org.apache.commons.beanutils.BeanUtils)

A+

[slevy]

Oui, tout ce que dit OButterlin est correct.
Par contre que fait cet objet User d, qui est une entité Hibernate, dans l'interface utilisateur ? Soit c'est une entité qui a été détachée de la session, auquel cas il suffit de la réattacher avec saveOrUpdate() par exemple; soit c'est un DTO (Data Transfert Object) qui contient les mêmes champs que l'entité User, mais qui n'est pas une entité au sens Hibernate du terme, auquel cas tu fais une copie des champs de UserDTO vers User (comme l'a expliqué OButterlin).
Il y a là un choix à faire.
Personnellement je te conseille d'utiliser des Entity détachée, comme ça tu n'as pas a dupliquer toutes les entités de ton application dans des DTO.
Explication sur l'état des objets hibernate et les objets détachés :
http://www.hibernate.org/hib_docs/re...jectstate.html

[jdev23]

Merci les amis c'est résolu