Discussion :

[Asterius]

Bonjour,

je vous présente une petit problème d'optimisation :

J'ai une table User et une table Game. Entre les deux, une many-to-many, appellée Player, définie comme suit :

Code :

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

1
2
3
4
5
6
7
8
9
<class name="Player" table="Player">
        <id name="id" column="id">
            <generator class="native"/>
        </id>
        <property name="status"/>    
        <many-to-one name="user" column="login" class="User"/>
        <many-to-one name="game" column="gameId" class="Game"/>
        (...)
    </class>

Donc, l'objectif est maintenant d'aller chercher dans la base toutes les parties auxquelles un User est associé via cette table Player. Voici mon code DAO non optimisé :

Code :

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

1
2
3
4
5
6
7
8
public List<Game> getUserGames(String login) {
        /* SELECT * FROM GAME g, PLAYER p WHERE p.gameId = g.gameId and p.login = login */
        Criteria c1 = getSession().createCriteria(Game.class);
        Criteria c2 = c1.createCriteria("players");
        Criteria c3 = c2.createCriteria("user");
        c3.add(Restrictions.eq("login", login));
        return c1.list();
    }

Ce code fonctionne, mais me crée deux jointures : une Game-Player, et une seconde Player-User. C'est une de trop puisqu'on connait l'identifiant de User (=login) qui est la clé étrangère de la table Player. Il suffirait alors de pouvoir mettre la restriction sur c2 plutôt que sur c3. Mais c'est là qu'est le soucis : le mapping de Player ne définit pas de "propriété" login...

Une idée?

[Gardyen]

alors si j'ai bien suivi ce que tu as expliqué, la clé primaire de user c'est le varchar login, que tu réutilises dans player en tant que clé étrangère ?

dans ce cas as-tu essayé quelque chose comme ça:

Code :

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

1
2
3
4
 
        Criteria c1 = getSession().createCriteria(Game.class);
        Criteria c2 = c1.createCriteria("players");
        c2.add(Restrictions.eq("user.login", login));

voire même

Code :

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

1
2
 
        Criteria c1 = getSession().createCriteria(Game.class).createCriteria("players").add(Restrictions.eq("user.login", login));

[Asterius]

C'est tout à fait ça!
Merci pour ton aide!

Peux-tu m'expliquer exactement ce que ça fait?
Quelle est donc la sémantique de ce "user.login"?
Car si je fais "user.email" par exemple, ça me lance une exception.
Que veut dire le point dans cette expression?

[Gardyen]

le point sert à accéder à une propriété: dans mon expression "user.login" j'accède par exemple à la propriété login de l'objet user existant dans players.
tu t'en sers par exemple avec les alias:

Code :

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

1
2
3
4
5
6
 
Criteria c1 = getSession().createCriteria(Game.class)
        .createCriteria("players", "pl") // je crée un alias
        .add(Restrictions.eq("user.login", login));
// Supposons que tu aies une propriété Name dans player
c1.add(Restrictions.eq("pl.Name", login));

mais dans ce cas précis, tu ne peux accéder qu'aux clés primaires/étrangères puisque la clé est le seul élément de user contenu dans player.
Du coup, Hibernate cherche la valeur donnée (ton String login ici) dans la colonne login de player, sans joindre la table user, ce que tu voulais.

Donc forcément user.email lance une erreur puisque qu'il n'y a pas de colonne email dans player, tu as besoin de user pour ça

j'espère avoir été clair

[Asterius]

Oui nickel, j'ai tout compris

Merci beaucoup!