Discussion :

[péhachepé]

Bonjour,
Je bute sur une requête nécessitant surement une jointure mais je ne trouve pas !

Voila, mon sujet comprend 4 tables, entre [...] le nom de mes champs :
- destinations [id_destination,...]
- regions [id_region, id_destination,...]
- circuits [id_circuit, id_region,id_type_voyage...]
- type_voyage [id_type_voyage,...]

Je souhaite afficher les types de voyage correspondant à des circuits existant dont la destination est envoyé par une variable (via une liste déroulante).

Voici le début de ma requête :

Code :

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

1
2
3
4
5
6
SELECT id_type_voyage 
FROM type_voyage 
WHERE id_type_voyage IN 
(SELECT id_type_voyage
 FROM  circuits
 WHERE ????)

Merci d'avance

[punkoff]

Bonjour,

il suffit de faire des jointures interne, un peu de lecture et vous comprendrez mieux :
http://sqlpro.developpez.com/cours/sqlaz/jointures/

[péhachepé]

Merci pour le lien, je vais m'en imprégner.

Voici la requête avec les jointures internes mais cela ne marche pas encore.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
SELECT id_type_voyage,titre_type_voyage
FROM type_voyage T,destinations D
INNER JOIN circuits C 
                ON (C.id_type_voyage=T.id_type_voyage)
INNER JOIN regions  R
                ON  (C.id_region=R.id_region)
INNER JOIN destinations 
		ON (R.id_destination=D.id_destination)
WHERE D.id_destination=$id_destination

Merci

[Yanika_bzh]

Ne melangez pas les jointures implicites et explicites...

Code :

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

1
2
3
4
5
6
7
8
9
SELECT id_type_voyage,titre_type_voyage
FROM type_voyage T
INNER JOIN circuits C 
         ON (C.id_type_voyage=T.id_type_voyage)
INNER JOIN regions  R
                ON  (C.id_region=R.id_region)
INNER JOIN destinations D 
		ON (R.id_destination=D.id_destination)
WHERE D.id_destination=$id_destination

[zeclad01]

Code :

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

SELECT att1_tab1,att2_tab1,att3_tab1,att1_tab2,att2_tab2,att1_tab3 FROM tab1,tab2,tab3 WHERE tab1.att1_tab1 = tab2.att2_tab2 AND tab1.att1_tab1 = tab3.att1_tab3 AND tab1.att2_tab1 = "kikoolol" AND tab2.att1_tab2 = "bonjour" AND tab1.att3_tab1 = "c'est bon ?"

Je pense qu'il s'agit d'un truc comme ça que tu veux.
Si t'as pas tout compris, donne les tables et les champs qui t'intéresses, je te ferai une requête adaptée à ton code.

[péhachepé]

Merci à vous

J'ai finalement remis les jointures internes à plus tard, j'ai suivi les conseils de zeclad01 en faisant une requête simple, maintenant c'est peut être pas le plus optimisé.

Voici ma requête pour info :

Code :

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

1
2
3
4
SELECT T.id_type_voyage,T.titre_type_voyage,C.id_type_voyage,C.id_region,R.id_region,R.id_destination,D.id_destination
FROM type_voyage T, circuits C, regions  R, destinations D 
WHERE T.id_type_voyage=C.id_type_voyage AND C.id_region=R.id_region AND R.id_destination=D.id_destination AND D.id_destination=$id_destination
GROUP BY T.id_type_voyage

[Yanika_bzh]

Mais votre requete dite "simple" manipule des jointures internes (seulement elles ne sont pas explicites, mais implicites).
Dans votre requete, vous utilisez un GROUP BY, pourquoi ? vous n'avez aucune fonction d'aggrégation.

Je ne saurai quand meme trop vous conseiller de potasser quelques tutoriels (celui ci par exemple)

Bon courage

[CinePhil]

Le besoin exprimé dans le premier message :

Je souhaite afficher les types de voyage correspondant à des circuits existant dont la destination est envoyé par une variable (via une liste déroulante).

Ta requête :

Code :

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

1
2
3
4
SELECT T.id_type_voyage,T.titre_type_voyage,C.id_type_voyage,C.id_region,R.id_region,R.id_destination,D.id_destination
FROM type_voyage T, circuits C, regions  R, destinations D 
WHERE T.id_type_voyage=C.id_type_voyage AND C.id_region=R.id_region AND R.id_destination=D.id_destination AND D.id_destination=$id_destination
GROUP BY T.id_type_voyage

Que viennent faire la région et la destination dans le SELECT si tu ne cherches que les types ?
Et je suis bien entendu du même avis que Yanika_bzh sur les jointures mal écrites et le GROUP BY inutile !

Voici la requête qui je pense répond à ton besoin :

Code :

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

1
2
3
4
5
SELECT DISTINCT T.id_type_voyage,T.titre_type_voyage
FROM type_voyage T
INNER JOIN circuits C ON T.id_type_voyage = C.id_type_voyage
    INNER JOIN regions  R ON C.id_region = R.id_region
WHERE R.id_destination = $id_destination

Remarques :
1) Puisque ta variable est id_destination qui figure dans la table regions, pas besoin de jointure avec la table destinations.
2) Tu devrais nommer tes tables au singulier.
3) Prend la bonne habitude d'indenter et d'aérer ton code, c'est plus facile à lire et donc à débugguer.
J'ai lu récemment dans une signature une phrase amusante qui va dans ce sens :

Codez en pensant que celui qui maintiendra votre code est un psychopathe qui connait votre adresse.