Discussion :

[abdess6600]

Bonjour le forum ,

J'ai une requête sql avec jointure (elle marche bien et me retourne le résultat souhaité).

Cependant je voudrais savoir si cette syntaxe est la mieux adaptée ou bien si elle peut être optimisée.

Votre aide me sera précieuse, je ne touche que rarement aux jointures. Merci.

Code :

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

1
2
3
4
5
SELECT u.* FROM user u
left join affectation_projet a
on a.id_user=u.id_user
where a.id_projet 
in(select id_projet from affectation_projet where id_user =1)

[abdess6600]

en gros je souhaite retourner les users qui sont dans les mêmes projets que le user dont le id = 1

[CinePhil]

Comme je l'explique sur mon blog, mettre une condition de restriction (WHERE) sur la table de droite d'une jointure externe gauche revient à faire une jointure interne.

Par ailleurs, évite la guerre des étoiles !

Enfin, ta condition de restriction est équivalente à une nouvelle jointure sur la table affectation_projet et la jointure devrait être plus performante qu'une requête corrélée.

Code :

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

1
2
3
4
5
SELECT -- les colonnes nécessaires et pas étoile !
FROM user u
INNER JOIN affectation_projet a ON a.id_user = u.id_user
    INNER JOIN affectation_projet a2 ON a2.id_user = a.id_user
WHERE a2.id_user = 1

[abdess6600]

Merci bcp de ton aide , ta réponse m'a bcp aidé .

il fallait changer la colonne de jointure du deuxième inner join pour retourner les collègues du user, la requête devient alors comme suit

Code :

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

1
2
3
4
5
6
 
SELECT u.*
FROM user u
INNER JOIN affectation_projet a ON a.id_user = u.id_user
    INNER JOIN affectation_projet a2 ON a2.id_projet = a.id_projet
 WHERE a2.id_user = 1

et pour ne pas retourner le collaborateur en question il sufit d'ajouter le contrôle suivant

Code :

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

1
2
3
4
5
6
7
SELECT u.*
FROM user u
INNER JOIN affectation_projet a ON a.id_user = u.id_user
    INNER JOIN affectation_projet a2 ON a2.id_projet = a.id_projet
          and a.id_user <> 1
WHERE a2.id_user = 1

Merci encore de ta reponse , je note comme résolu

[Yanika_bzh]

Enfin, ta condition de restriction est équivalente à une nouvelle jointure sur la table affectation_projet et la jointure devrait être plus performante qu'une requête corrélée.

Code :

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

1
2
3
4
5
SELECT -- les colonnes nécessaires et pas étoile !
FROM user u
INNER JOIN affectation_projet a ON a.id_user = u.id_user
    INNER JOIN affectation_projet a2 ON a2.id_user = a.id_user
WHERE a2.id_user = 1

Peut etre plus performante mais surement pas iso fonctionnelle ...