Discussion :

[ddaweb]

bonsoir,

Je suis de retour pour un problème de jointure qui me donne une erreur Mysql :

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'AS type ON type.cours_type_id=classe.cours_classe_type_id INNER JOIN d_cours' at line 4

Voici le code de ma fonction :

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
21
22
23
24
25
26
27
28
function RechercheCoursSection ($section) {
	$reponse=mysql_query("SELECT classe.*, section.*, type.*, couleur.*, horaire.*, jour.* 
		FROM d_cours_classe AS classe 
		INNER JOIN d_cours_section AS section ON section.cours_section_id=classe.cours_classe_id 
		INNER JOIN d_classe cours_type AS type ON type.cours_type_id=classe.cours_classe_type_id 
		INNER JOIN d_cours_classe_couleur AS couleur ON couleur.cours_classe_couleur_id=classe.cours_classe_couleur_id 
		INNER JOIN d_cours_horaire AS horaire ON horaire.cours_horaire_classe_id=classe.cours_classe_id 
		INNER JOIN d_cours_jours AS jour ON jour.cours_jour_id=horaire.cours_horaire_jours_id 
		WHERE cours_section=$section && cours_classe_actif='oui' 
		ORDER BY jours.cours_jours_numero, horaire.cours_horaire_debut, classe.cours_classe_nom ASC") or die(mysql_error()); // -------- récupèrer les codes postaux de la table des codes postaux
	while ($in= mysql_fetch_array($reponse)) {
		$i++;
		$cours_jour_nom=$in['cours_jours_nom'];
		$cours_horaire_debut=$in['cours_horaire_debut'];
		$cours_horaire_fin=$in['cours_horaire_fin'];
		$cours_type=$in['cours_type_nom'];
		$cours_classe=$in['cours_classe_nom'];
		$cours_classe_description=$in['cours_classe_site'];
 
		$cours_array[$i][jour]=$cours_jour_nom;
		$cours_array[$i][debut]=$cours_horaire_debut;
		$cours_array[$i][fin]=$cours_horaire_fin;
		$cours_array[$i][type]=$cours_type;
		$cours_array[$i][classe]=$cours_classe;
		$cours_array[$i][description]=$cours_classe_description;
	}
	return ($cours_array);
}

Je cherche sur la toile, mais rien ne m'a encore permis de trouver une solution.

Si quelqu'un a une idée, je prends

@+
ddaweb

[sabotage]

Tu as un espace

Code :

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

d_classe cours_type

La première chose à faire quand tu debugues c'est de circonscrire ton problème.
Si tu l'avais fait tu aurais vu que le problème se trouvait sur la 4ème ligne.

[ddaweb]

Oui je l'avais remarqué et corrigé, mais cela ne change rien à mon problème (pardon de ne pas être venu corrigé, mais j'étais absorbé par une erreur avec le code ci-dessous ; il y a encore d'autres erreurs que j'ai aussi rectifiées sans effet ... mieux vaut partir du code ci-dessous).

J'ai tenté la vielle syntaxe et cela fonctionne :

Code sql :

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

1
2
3
4
5
6
7
8
9
SELECT * 
FROM d_cours_classe,  d_cours_section, d_cours_type, d_cours_classe_couleur, d_cours_horaire, d_cours_jours 
WHERE (d_cours_section.cours_section_id=d_cours_classe.cours_classe_section_id) 
&& (d_cours_type.cours_type_id=d_cours_classe.cours_classe_type_id) 
&& (d_cours_classe_couleur.cours_classe_couleur_id=d_cours_classe.cours_classe_couleurs_id) 
&& (d_cours_horaire.cours_horaire_classe_id=d_cours_classe.cours_classe_id) 
&& (d_cours_jours.cours_jours_id=d_cours_horaire.cours_horaire_jours_id) 
&& cours_section='4C' && cours_classe_actif='oui' 
ORDER BY cours_jours_numero, cours_horaire_debut, cours_classe_nom ASC

Petite infos : je vais chercher des infos dans toutes les tables ... voilà quand on veut faire quelque chose d'hyper paramétrable (développement pour mon club de chien, mais devrait aussi servir dans un autre club avec d'autres paramètres)

[ddaweb]

Je voulais tenter les jointures avec "JOIN" ... cela ne m'a pas trop réussit.
D'habitude je l'écrivais avec le "WHERE", cette tentative pour apprendre une autre manière peut-être plus propre ?

[rawsrc]

Salut,

essaie avec ceci

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
15
SELECT * 
FROM 
    d_cours_classe
        INNER JOIN d_cours_section        ON d_cours_classe.cours_classe_section_id  = d_cours_section.cours_section_id
        INNER JOIN d_cours_type           ON d_cours_classe.cours_classe_type_id     = d_cours_type.cours_type_id
        INNER JOIN d_cours_classe_couleur ON d_cours_classe.cours_classe_couleurs_id = d_cours_classe_couleur.cours_classe_couleur_id
        INNER JOIN d_cours_horaire        ON d_cours_classe.cours_classe_id          = d_cours_horaire.cours_horaire_classe_id
            INNER JOIN d_cours_jours      ON d_cours_horaire.cours_horaire_jours_id  = d_cours_jours.cours_jours_id
WHERE
   cours_section          = '4C' 
   AND cours_classe_actif = 'oui'  
ORDER BY 
    cours_jours_numero, 
    cours_horaire_debut, 
    cours_classe_nom

[ddaweb]

La réponse dans phpmyadmin donne la même réponse que mon code
Merci pour ta réponse.

Dans l'absolu, quelle est le meilleur code, voir le plus efficace ?

[rawsrc]

Dans tous les cas : les jointures.
Ne serait-ce que parce qu'il est possible d'avoir un contrôle sur la jonction INNER, LEFT, RIGHT, OUTER tandis qu'avec le WHERE nada.

[ddaweb]

J'ai déjà pu remarquer l'avantage du JOIN LEFT ou RIGHT ... avant je faisais 2 requêtes pour arriver à la même chose.

Dans le cas de ce post, en fin de compte je n'y vois pas trop d'intérêt directement (peut-être du point de vue des ressources utilisées par la fonction JOIN mieux interprétée par le serveur ?).
J'avoue avoir découvert sur ce forum cette manière de coder les jointures il y a peu ... du coup j'ai voulu tenter de coder ainsi, p'te trop de tables en une fois

Dans le but d'apprendre, si tu peux développer un peu, d'une manière pas trop technique car j'ai appris à coder de manière autodidacte, mais suivant mes besoins du moment, pas dans son ensemble malheureusement ... maintenant je creuse un peu plus.

[Celira]

Faire des jointures et pas des liaisons par WHERE permet d'éviter de mélanger les conditions de jointure et les conditions de recherche. Cela peut être très pratique lorsque tu écris ta requête de façon dynamique à partir de champs renseignés ou non dans un formulaire, histoire d'éviter les produits cartésiens (remonter 350 000 lignes au lieu de 15, c'est pas terrible)

A lire sur le sujet : Le SQL de A à Z - les Jointures

[CinePhil]

J'ai tenté la vielle syntaxe et cela fonctionne :

Code sql :

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

1
2
3
4
5
6
7
8
9
SELECT * 
FROM d_cours_classe,  d_cours_section, d_cours_type, d_cours_classe_couleur, d_cours_horaire, d_cours_jours 
WHERE (d_cours_section.cours_section_id=d_cours_classe.cours_classe_section_id) 
&& (d_cours_type.cours_type_id=d_cours_classe.cours_classe_type_id) 
&& (d_cours_classe_couleur.cours_classe_couleur_id=d_cours_classe.cours_classe_couleurs_id) 
&& (d_cours_horaire.cours_horaire_classe_id=d_cours_classe.cours_classe_id) 
&& (d_cours_jours.cours_jours_id=d_cours_horaire.cours_horaire_jours_id) 
&& cours_section='4C' && cours_classe_actif='oui' 
ORDER BY cours_jours_numero, cours_horaire_debut, cours_classe_nom ASC

1) Il vaut mieux éviter la guerre des étoiles !

2) Les jointures s'écrivent depuis plus de 20 ans avec l'opérateur JOIN ; il serait temps de s'y mettre !

3) Les parenthèses dans les conditions de jointure ou de restriction sont le plus souvent inutiles, sauf en cas de mélange de AND et de OR.

4) 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.

5) En SQL, l'opérateur logique "et" s'écrit AND et non pas &&.

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

7) Le tri ascendant est le tri par défaut. Il est donc inutile de préciser ASC dans la partie ORDER BY quand vous n'avez que des colonnes triées en ascendant.

Si la colonne "cours_section" fait bien partie de la table "d_cours_section" et si la colonne "cours_classe_actif" fait bien partie de la table "d_cours_classe" alors cette requête devrait être équivalente à la votre et donc fonctionner de la même manière. Je vous laisse cependant compléter les alias des colonnes pour celles qui figurent dans la partie ORDER BY et remplacer l'étoile par les colonnes dont vous avez besoin dans la partie SELECT.

Code :

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

1
2
3
4
5
6
7
8
9
10
SELECT les_colonnes_necessaires_et_pas_etoile
FROM d_cours_classe c 
INNER JOIN d_cours_section s ON s.cours_section_id = c.cours_classe_section_id
INNER JOIN d_cours_type t ON t.cours_type_id = c.cours_classe_type_id
INNER JOIN d_cours_classe_couleur cl ON cl.cours_classe_couleur_id = c.cours_classe_couleurs_id
INNER JOIN d_cours_horaire h ON h.cours_horaire_classe_id = c.cours_classe_id
	INNER JOIN d_cours_jours j ON j.cours_jours_id = h.cours_horaire_jours_id
WHERE s.cours_section = '4C' 
	AND c.cours_classe_actif = 'oui'
ORDER BY cours_jours_numero, cours_horaire_debut, cours_classe_nom

[ddaweb]

Merci CinePhil pour toute ses infos, certaines chose je savais, mais codais comme j'avais vu lors de mes recherches : je fais devoir changer cela absolument.
Comme je veux aller plus loin qu'avant et coder au mieux et surtout plus efficace, c'est très intéressant.

Je constate que tu as bien compris ma logique concernant les tables et nom des champs ... le 'd_' devant le nom de la table correspond à DATA et les champs reprennent le nom de la table avec le complément du nom du champs : il devient ainsi presque impossible d'obtenir le même nom de champs en cas de jointure.

Concernant le AND et && ... je vais dire que c'est par habitude et surtout fainéantise (et cela fonctionne toujours ), le OR je l'indique par contre au lieu de ||.

Les alias sont nouveaux pour moi et j'avoue encore avoir du mal avec à la lecture, je mets d'ailleurs les parenthèse pour cela ... habitudes quand tu nous tient

Je suis en fait occupé de faire des choses qui auraient du être faites depuis le début ... pas facile de normaliser quelque chose et adapter tous les scripts qui vont avec.

[CinePhil]

Et donc, est-ce ?