Discussion :

[kennely]

Bonjour,

Suite à la migration d'une base ACCESS vers SQL Server, je suis un peu perdu avec le langage SQL (appris il y a une dizaine d'année, mais pas ou peu utilisé depuis), et les jointures multiples.

J'ai créé la requête ACCESS suivante :

Code :

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

1
2
3
4
SELECT OCEAN_ADM_CELLULE.CELL_ID, OCEAN_ADM_MODRADIO.MRAD_REF
FROM (OCEAN_ADM_SITE INNER JOIN OCEAN_ADM_BAIE ON OCEAN_ADM_SITE.SIT_ID = OCEAN_ADM_BAIE.SIT_ID) INNER JOIN (OCEAN_ADM_CELLULE INNER JOIN (OCEAN_ADM_CONFBAIE INNER JOIN OCEAN_ADM_MODRADIO ON (OCEAN_ADM_CONFBAIE.MRAD_ID = OCEAN_ADM_MODRADIO.MRAD_ID) AND (OCEAN_ADM_CONFBAIE.SYS_ID = OCEAN_ADM_MODRADIO.SYS_ID)) ON (OCEAN_ADM_CELLULE.SYS_ID = OCEAN_ADM_MODRADIO.SYS_ID) AND (OCEAN_ADM_CELLULE.CTR_ID = OCEAN_ADM_MODRADIO.CTR_ID)) ON OCEAN_ADM_BAIE.BAI_ID = OCEAN_ADM_CONFBAIE.BAI_ID
WHERE (((OCEAN_ADM_CELLULE.CTR_ID)=6))
GROUP BY OCEAN_ADM_CELLULE.CELL_ID, OCEAN_ADM_MODRADIO.MRAD_REF;

Comment traduire cela en SQL pour SQL Server ?

En relisant mes cours sur le language SQL, je comprend bien comment créer une jointure entre 2 tables.
Mais lorsque il y a plusieurs jointures vers la même table qui elle même comporte plusieurs jointures vers une autre table, là je suis perdu.
Y a-t-il une astuce pour faciliter l'écriture du code ?


Merci pour votre aide

[aieeeuuuuu]

Bonjour,

globalement, les "astuces" sont toujours les même et somme toute assez basiques :
1/ indentation
2/ utilisation des alias
3/ suppression du superflu (comme les parenthéses dans votre cas)

ce qui donne avec votre requete :

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
19
20
 
SELECT 
		Cel.CELL_ID
	,	Mo.MRAD_REF
FROM		OCEAN_ADM_SITE sit
INNER JOIN	OCEAN_ADM_BAIE bai
		ON		sit.SIT_ID = bai.SIT_ID
INNER JOIN	OCEAN_ADM_CELLULE Cel
			INNER JOIN OCEAN_ADM_CONFBAIE conf
						INNER JOIN OCEAN_ADM_MODRADIO Mo
								ON			conf.MRAD_ID = Mo.MRAD_ID 
								AND			conf.SYS_ID = Mo.SYS_ID 
					ON		Cel.SYS_ID = Mo.SYS_ID 
					AND		Cel.CTR_ID = Mo.CTR_ID 
		ON		bai.BAI_ID = conf.BAI_ID
WHERE	Cel.CTR_ID=6
GROUP BY 
			Cel.CELL_ID
		,	Mo.MRAD_REF
;

[SQLpro]

Access rajoute des tonnes de parenthèses totalement farfelues et pour certaines (notamment jointures) totalement incompréhensible. Débarrassez vous de l'écriture automatique des requêtes via le gadget visuel d'accès et écrivez vos propres requêtes sous forme textuelle !

A +

[kennely]

Merci pour vos réponses.
Je suis d'accord, ACCESS rajoute des tonnes de parenthèses et utilise son propre code SQL.

OK pour les alias.
J'ai aussi remarqué que l'on peux utiliser le JOIN sans le INNER
Ce qui me perturbe dans vos réponses c'est que aieeeuuuuu imbrique les INNER JOIN et met ensuite les condition ON, alors que ce n'est pas le cas de hmira.
Les 2 codes sont équivalents ?

[aieeeuuuuu]

J'ai aussi remarqué que l'on peux utiliser le JOIN sans le INNER

En effet, INNER est facultatif (par défaut).

Ce qui me perturbe dans vos réponses c'est que aieeeuuuuu imbrique les INNER JOIN et met ensuite les condition ON, alors que ce n'est pas le cas de hmira.
Les 2 codes sont équivalents ?

J'ai en effet gardé la structure initiale de votre requete.
Dans le cas de jointures internes uniquement, c'est équivalent, et je trouve l'écriture d'hmira bien plus lisible (notamment car la condition de jointure suit directement la table jointe...). Cette écriture est d'ailleurs nettement plus courante, et de surcroit, rien ne vous empêche d'indenter les jointures pour une meilleur lisibilité de l'arborescence, sans pour autant imbriquer les jointures :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
SELECT ...
FROM CLIENT C
INNER JOIN COMMANDE C
    ON C.cli_id = C.cli_id
            INNER JOIN COMMANDE_DETAIL CD
                ON CD.cmd_id = C.cmd_id
INNER JOIN PAIEMENT P
    ON P.cli_id = C.cli_id
            INNER JOIN PAIEMENT_DETAIL PD
                ON PD.pai_id = P.pai_id

L'imbrication des jointures devient cependant utile en cas de jointure externe. Si dans l'exemple ci dessus, vous supposez qu'un client peut ne pas avoir de commande et qu'il peut ne pas avoir de paiement (mais que vous voulez malgré tout garder ces clients dans le résultat de la requête) vous pouvez passer toutes les jointures en jointure externe, ou bien écrire comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
SELECT ...
FROM CLIENT C
LEFT JOIN COMMANDE C
            INNER JOIN COMMANDE_DETAIL CD
                ON CD.cmd_id = C.cmd_id
    ON C.cli_id = C.cli_id
LEFT JOIN PAIEMENT P
            INNER JOIN PAIEMENT_DETAIL PD
                ON PD.pai_id = P.pai_id
    ON P.cli_id = C.cli_id

[kennely]

aieeeuuuuu, merci pour ces précisions.
En effet, je trourve aussi le code d'hmira plus lisible.
C'est ACCESS qui a créé ces imbrications.

Grâce à vos explications (à tout les deux), j'ai pu créer ma requête que j'ai utilisé dans une requête mise à jour avec une instruction CASE.
Pour me remettre dans le bain, pourriez-vous me conseiller un livre de référence sur le code SQL ?

[hmira]

Cette syntaxe de Microsof Access n'a plus cours et, à ma connaissance n'a même jamais eu cours sous SQL Server ! Il faut respecter désormais la norme SQL

Sous SQL Server, votre requête doit être réécrite comme suit :

Code SQL :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
SELECT OCEAN_ADM_CELLULE.CELL_ID, OCEAN_ADM_MODRADIO.MRAD_REF
FROM OCEAN_ADM_SITE 
INNER JOIN OCEAN_ADM_BAIE 
  ON OCEAN_ADM_SITE.SIT_ID = OCEAN_ADM_BAIE.SIT_ID
INNER JOIN OCEAN_ADM_CONFBAIE 
  ON OCEAN_ADM_BAIE.BAI_ID = OCEAN_ADM_CONFBAIE.BAI_ID 	
INNER JOIN OCEAN_ADM_MODRADIO 
  ON  OCEAN_ADM_CONFBAIE.MRAD_ID = OCEAN_ADM_MODRADIO.MRAD_ID 
  AND OCEAN_ADM_CONFBAIE.SYS_ID = OCEAN_ADM_MODRADIO.SYS_ID	
INNER JOIN OCEAN_ADM_CELLULE 
  ON  OCEAN_ADM_CELLULE.SYS_ID = OCEAN_ADM_MODRADIO.SYS_ID
  AND OCEAN_ADM_CELLULE.CTR_ID = OCEAN_ADM_MODRADIO.CTR_ID
WHERE OCEAN_ADM_CELLULE.CTR_ID = 6
GROUP BY OCEAN_ADM_CELLULE.CELL_ID, OCEAN_ADM_MODRADIO.MRAD_REF;

A+

PS : Je n'avais pas vu la réponse de aieeeuuuuu !