Discussion :

[sadkat]

Bonjour,
J'ai eu quelques cours de sql au lycée mais on ne m'a jamais vraiment parlé des jointures type "left join" ou "right join".

Pourriez vous me dire si il y a plus optimisé que ce que j'ai fais (qui fonctionne).

Voici une image de ma BBD :

Et voici ma requête :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 SELECT*
        FROM enseigne JOIN formule 
        ON formule_idformule = idformule 
        JOIN enseigne_has_activite
        ON enseigne_idenseigne = idenseigne
        AND principale = "Y"
        JOIN activite
        ON enseigne_has_activite.activite_idactivite = idactivite
        JOIN secteur
        ON secteur_idsecteur = idsecteur
        WHERE idenseigne=\''.$id.'\' 
        AND statut_idstatut NOT IN(3,6)

Sachant qu'il y a beaucoup plus de champs que ce que j'affiche là....

Merci d'avance !!

[CinePhil]

À première vue, ce n'est pas trop mal mais :
1) Inutile de mettre les valeurs numériques entre apostrophes.

2) Les valeurs textuelles s'écrivent entre apostrophes et non entre guillemets.

3) Évitez la guerre des étoiles !
Notamment, rien que sur le morceau de schéma que vous donnez, vous aurez deux fois la colonne "libelle" provenant des tables "secteur" et "activite".

4) Mettez des alias pour les tables et utilisez ces alias pour toutes les colonnes utilisées dans la requête.
Ça rend la requête plus facile à lire.

5) Comme vos jointures sont toutes des jointures internes, il est d'usage de séparer les conditions de jointure et les conditions de restriction.
La restriction sur principale devrait donc être dans le WHERE.

Ça donnerait donc ça :

Code :

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

1
2
3
4
5
6
7
8
9
SELECT les_colonnes_necessaires_et_pas_etoile
FROM enseigne e
INNER JOIN formule f ON e.formule_idformule = f.idformule 
INNER JOIN enseigne_has_activite ea ON ea.enseigne_idenseigne = e.idenseigne
INNER JOIN activite a ON ea.activite_idactivite = a.idactivite
INNER JOIN secteur s ON e.secteur_idsecteur = s.idsecteur
WHERE idenseigne = $id
    AND statut_idstatut NOT IN(3,6) -- manque l'alias de table
    AND ea.principale = 'Y'

Une dernière chose : Les champs sont à la campagne, dans les blocs opératoires, dans les formulaires mais pas dans les tables SQL qui ne sont composées que de lignes et de colonnes !

[sadkat]

Ok ! Merci beaucoup pour tous ces conseils.

En fait j'ai mis une * pour vous éviter l'énumération de ch... heuu de colonne. Donc j'ai bien réglé le problème des multi-libellés avec des alias.

En tout cas je prends note de tes remarques pour les guillemets.

Encore merci !