Discussion :

[shawty]

Bonjour,

Faut-il placer le code hibernate dans la méthode "ActionForward execute" de la classe java de struts ?

exemple de code Hibernate :

Code :

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

1
2
3
4
 
session = HibernateUtil.currentSession();
tx = session.beginTransaction();
Query query = session.createQuery("from User u");

merci

[djo.mos]

Ca peut se faire, mais ce serait contraire aux bonnes pratiques du pattern MVC !
En effet, faire ceci revient à mélanger le Model (code Hibernate) avec le Control (dans Action.execute).
Le mieux serait de Crér une classe avec une méthode qui contient le code Hibernate, et d'appeler cette méthode depuis Action.execute().

exemple:

Dans la nouvelle classe:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public class OpUser {
    public static List users(){
        List u = ....createQuery("From User u").list();
        return u;
    }
 
    public void add(String login, String pwd){
        ...
    }
 
    ...
}

et dans ton Action

Code :

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

1
2
3
4
5
6
7
8
9
 
public ActionMapping execute(...){
    ...
    List l = OpUser.users();//  <== un seul appel à une méthode du couche métier
    ...
    //par exemple, mettre la liste d'utilisateurs dans la session
    request.getSession.setAttribute("users", l);
    ...
}

C'est un peu plus long et emmerdant à coder, mais ton code est beaucoup plus propre et tu adhères strictement au MVC !

[shawty]

d'accord je comprends mieux,
j'ai 2 questions :
- on mettrait la classe 'OpUser' dans le package contenant les Dao ?
- admettons qu'on veuille récupérer un user en fonction d'un prédicat (exemple clause WHERE), comment s'y prendrait-on pour faire une méthode dans la classe 'OpUser' ?

[djo.mos]

Q1: Oui, OpUser fait partie du package model, le même qui contient les DAOs.

Q2: Apparament, tu veux coder une méthode qui te permet de valider un couple (login, mdp) lorsqu'un un utilisateur les fournit dans une page de connexion (corrige moi si je me trompe), ceci peut se faire comme ceci :

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
16
17
 
public class OpUser {
    public static List users(){
        List u = ....createQuery("From User u").list();
        return u;
    }
 
    public User validate(String login, String pwd){
        List l = ....createQuery("From User u Where u.login=? AND u.pwd=?").setString(0, login).setString(1, pwd).list();
        if(l.size()==0)
                return null;
        else
                return (User)l.get(0);
    }
 
    ...
}

Ainsi, dans execute() de ton Action, tu appeles OpUser.validate() pour vérifier si un compte existe !

[shawty]

d'accord c'est bien ce que j'imaginais, merci pour ton aide