Discussion :

[Asterius]

Bonjour,

J'ai depuis toujours utilisé les Criteria hibernate et je trouve cela plus clair que le HQL (mais bon, chacun ses gouts). Voici un exemple de code que j'ai toujours utilisé :

Code :

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

1
2
3
4
5
6
7
8
9
public class InstitutionDAOImpl extends HibernateDaoSupport implements InstitutionDAO {
 
    /** {@inheritDoc} */
    @SuppressWarnings("unchecked")
    public List<Institution> getAllSortedInstitution() {
        Criteria criteria = getSession().createCriteria(Institution.class);
        criteria.addOrder(Order.asc("name"));
        return criteria.list();
    }

Je me suis récemment laissé dire que c'était une mauvaise manière de coder, notamment parce qu'on est pas certain que la session retournée par getSession() soit bien fermée après. A la place, on me suggère d'utiliser :

Code :

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

1
2
3
4
5
6
7
8
9
    public List<Institution> getAllSortedInstitution() {
        return (List<Institution>) getHibernateTemplate().execute(new HibernateCallback() {
            public Object doInHibernate(Session session) throws HibernateException, SQLException {
                Criteria criteria = session.createCriteria(Institution.class);
                criteria.addOrder(Order.asc("name"));
                return criteria.list();
            }
        });
    }

Je dois avouer ne pas vraiment bien saisir pourquoi on doit passer par une telle complexité de code, alors qu'en HQL, HibernateTemplate nous offre la très belle méthode "find". Votre méthode DAO prend une seule ligne, ici il en faut maintenant 9...

N'y a-t-il rien de plus simple qui permette d'utiliser les Criteria sans prendre de risques?
Merci pour vos avis!

[_skip]

Voici un exemple sorti de mon code :

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
18
public  List<Campaign>  getCampaigns(  Boolean filterValidated, boolean prefetchUser, boolean prefetchItems)
    {
 
        DetachedCriteria criteria = DetachedCriteria.forClass(Campaign.class);
 
        //set filters and fetch modes...
        if( filterValidated != null)
            criteria.add( Restrictions.isNotNull( Campaign.PROP_VALIDATION_DATE ) );
 
        if(prefetchItems )
            criteria.setFetchMode(Campaign.PROP_ITEM, FetchMode.JOIN);
 
        if( prefetchUser )
            criteria.setFetchMode(Campaign.PROP_USER, FetchMode.JOIN);
 
        return getHibernateTemplate().findByCriteria(criteria);
 
    }

Comme tu peux le voir, je profite de l'API criteria.

Dans ton premier exemple, Il est en effet contre-indiqué de bosser avec l'API hibernate directement, pour des raisons de sessions et il me semble (sans avoir vérifié, prend le comme tel) d'encapsulation des exceptions hibernate en exception Spring.

Généralement on utilise la fonctionnalité de callback de hibernateTemplate seulement quand on a vraiment besoin d'une fonctionnalité bas niveau d'hibernate qu'on ne peut obtenir avec les méthodes wrapper de hibernateTemplate.

[Asterius]

Merci, c'est ce que j'avais finalement trouvé aussi!

Un seul détail : dommage que findByCriteria retourne une List et qu'il n'existe pas un équivalent de unique result, mais c'est un embêtement mineur.

[_skip]

Si ca tenait qu'à moi, j'aimerai aussi un deleteByCriteria puis plein d'autres options. C'est très commode à lire et à utiliser.

[fr1man]

Dans ton premier exemple, Il est en effet contre-indiqué de bosser avec l'API hibernate directement, pour des raisons de sessions

Pourquoi donc ? Peux tu expliquer ?

Je ne suis pas pour l'utilisation du callback, tout du moins dans ton cas et pour la raison que tu invoques.
La fermeture de la session ne doit pas être gérée au niveau du dao.
En générale on regroupe plusieurs accès à la base, (un service qui appelle plusieurs DAO), sous la même session.

[_skip]

En fait j'ai mal lu son code, je n'ai pas vu qu'il employait getSession() fourni par hibernateTemplate et donc qu'il devenait responsable de la gestion de session.

Mais le reste sur l'encapsulation d'exception reste vrai.