Discussion :

[marseillx3]

Bonjour à tous et merci d'avance pour votre aide,

Je vous expose mon problème.

J'ai 3 tables dans une base de données :

• constructeur (id_constructeur, nom_constructeur)
• software_editeur (id_editeur, editeur)
• contact (id_contact, nom, prenom, id_type_marque, id_marque, autre_marque, telephone_fixe, telephone_portable, mail)

Je cherche à créer une requête qui, si id_type_marque est égal à 1, cherchera id_marque = id_constructeur dans la table constructeur et si id_type_marque est égal à 2 cherchera id_marque = id_software dans la table software_editeur.

Voici la requête que j'ai créé pour le moment :

Code :

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

1
2
3
4
5
6
7
SELECT contact.id_contact, contact.nom, contact.prenom, contact.id_type_marque, contact.id_marque, contact.autre_marque, contact.telephone_fixe, contact.telephone_portable, contact.mail, GROUP_CONCAT(region.region SEPARATOR ", ") as regions, 
CASE 
  WHEN contact.id_type_marque = 1 THEN (SELECT constructeur.nom_constructeur FROM constructeur, contact WHERE constructeur.id_constructeur = contact.id_marque LIMIT 1) 
  ELSE (SELECT software_editeur.editeur FROM software_editeur, contact WHERE software_editeur.id_editeur = contact.id_marque LIMIT 1) 
END AS marque 
FROM contact, region, liaison_contact_region 
WHERE region.id_region = liaison_contact_region.id_region AND contact.id_contact = liaison_contact_region.id_contact

Comme vous vous en doutez, cette requête ne fonctionne pas car le contact.id_marque dans les requêtes CASE ne correspond pas à celui du SELECT de la requête principale.

Ma question est donc la suivante : est-il possible de faire en sorte que le contact.id_marque corresponde à celui de la requête principale ?

Si non, existe t-il un moyen différent pour obtenir ce que je souhaite ?

Merci d'avance pour votre aide.

[CinePhil]

Bonjour,

1) Un code indenté et aéré est plus agréable à lire et à déboguer.

2) L'utilisation d'alias pour les tables est fortement recommandé dès qu'il y a plus d'une table dans la requête. Cela facilite l'écriture et la lecture de celle-ci. Il faut ensuite utiliser systématiquement ces alias devant chaque colonne nommée pour savoir facilement de quelle table elle vient.

3) Les fonctions de groupage comme GROUP_CONCAT doivent être accompagnées d'un GROUP BY.

4) Les jointures s'écrivent depuis 1992 avec l'opérateur JOIN ; il serait temps de s'y mettre !

5) Le moyen pour faire ce que vous souhaitez est de faire deux jointures externes vers les tables constructeur et software_editeur.

6) Par contre, cela montre une erreur de conception de votre base de données dans laquelle vous auriez dû faire un héritage plutôt qu'une sorte de fausse clé étrangère pointant vers deux tables différentes !

Voici votre requête corrigée :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
SELECT c.id_contact, c.nom, c.prenom, c.id_type_marque, c.id_marque, c.autre_marque, c.telephone_fixe, c.telephone_portable, c.mail, 
	GROUP_CONCAT(r.region SEPARATOR ', ') as regions, 
	CASE 
		WHEN c.id_type_marque = 1 THEN co.nom_constructeur
		ELSE se.editeur
	END AS marque 
FROM liaison_contact_region lcr
INNER JOIN region r ON r.id_region = lcr.id_region 
INNER JOIN contact c ON c.id_contact = lcr.id_contact
	LEFT OUTER JOIN constructeur co ON co.id_constructeur = c.id_marque
	LEFT OUTER JOIN software_editeur se ON se.id_editeur = c.id_marque
GROUP BY c.id_contact, c.nom, c.prenom, c.id_type_marque, c.id_marque, c.autre_marque, c.telephone_fixe, c.telephone_portable, c.mail, marque

[aieeeuuuuu]

Voici deux alternatives sur la base de la requete proposée par Cinephil

Code :

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

1
2
3
4
5
6
7
8
9
10
 
SELECT c.id_contact, c.nom, c.prenom, c.id_type_marque, c.id_marque, c.autre_marque, c.telephone_fixe, c.telephone_portable, c.mail, 
	GROUP_CONCAT(r.region SEPARATOR ', ') as regions, 
	COALESCE(co.nom_constructeur, se.editeur) AS marque 
FROM liaison_contact_region lcr
INNER JOIN region r ON r.id_region = lcr.id_region 
INNER JOIN contact c ON c.id_contact = lcr.id_contact
	LEFT OUTER JOIN constructeur co ON co.id_constructeur = c.id_marque AND c.id_type_marque = 1
	LEFT OUTER JOIN software_editeur se ON se.id_editeur = c.id_marque AND  c.id_type_marque = 2
GROUP BY c.id_contact, c.nom, c.prenom, c.id_type_marque, c.id_marque, c.autre_marque, c.telephone_fixe, c.telephone_portable, c.mail, marque

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
 
SELECT c.id_contact, c.nom, c.prenom, c.id_type_marque, c.id_marque, c.autre_marque, c.telephone_fixe, c.telephone_portable, c.mail, 
	GROUP_CONCAT(r.region SEPARATOR ', ') as regions, 
	Mrq.marque 
FROM liaison_contact_region lcr
INNER JOIN region r ON r.id_region = lcr.id_region 
INNER JOIN contact c ON c.id_contact = lcr.id_contact
	INNER JOIN (
	   SELECT id_constructeur as id_marque, nom_constructeur as marque, 1 as id_type_marque
	   FROM constructeur 
	   UNION ALL
	   SELECT id_constructeur as id_marque, editeur as marque, 2 as id_type_marque
	   FROM software_editeur se 
	) AS Mrq
	ON Mrq.id_marque = c.id_marque 
	AND Mrq.id_type_marque =  c.id_type_marque 
GROUP BY c.id_contact, c.nom, c.prenom, c.id_type_marque, c.id_marque, c.autre_marque, c.telephone_fixe, c.telephone_portable, c.mail, marque

[marseillx3]

Merci à tous pour votre aide, les requêtes fonctionnent parfaitement.

Et merci CinePhil pour tes conseils, j'en prend bonnes notes.

Bonne journée