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