requete SQL différente entre MySQL et php

Bonjour a tous,

J'ai 2 tables dans mysql,
la première comporte une clé primaire "id" dans le seconde table j'ai une clé primaire et une clé étrangère "id". Les 2 "id" étant strictement identique.

Lorsque je fais ma requête sql via phpadmin le retour de ma selection est tout a fait correcte, par contre lorsque je la fait via php je n'ai qu'un seul enregistrement et je ne comprends pas où se situe le problème

ma requête est :

Code:

---

1 2	$sql="SELECT * FROM salarie INNER JOIN enfants ON 'enfants.idSalarie'='$idSalarie'";

---

$idsalarie est obtenu après une recherche dans la table principale

A partir de la requête je m'en sers pour afficher les données dans un tableau

Pour le moment, pour voir ce ce que me retourne la reqête j'utilise un print_r

Merci par avance pour l'aide que vous m'apporterai

Bonne journée a tous

Bonjour,

Comme entre double quote, il y a évaluation des variables, tu peux faire:

Code:

---

1 2	$sql="SELECT * FROM salarie INNER JOIN enfants ON enfants.idSalarie=$idSalarie";

---

Ensuite, je ne comprends pas trop bien

Citation:

---

Pour le moment, pour voir ce ce que me retourne la reqête j'utilise un print_r

---

La requête va te retourner une ressource et non une chaine, tab, structure !

Comment tu exploites cette requête ?

Bonjour,

la jointure marche plutot comme ceci :

Code:

---

$sql="SELECT * FROM salarie INNER JOIN enfants ON 'enfants.idSalarie'='salarie.idSalarie' WHERE salarie.idSalarie=$idSalarie";

---

J'ai mis 'idSalarie' pour la table Salarie, mais note celle de ta clé primaire. Elle peut être 'id' tout simplement (de même dans la clause WHERE).

Citation:

---

Envoyé par Nowwis

Bonjour,

la jointure marche plutot comme ceci :

Code:

---

$sql="SELECT * FROM salarie INNER JOIN enfants ON 'enfants.idSalarie'='salarie.idSalarie' WHERE salarie.idSalarie=$idSalarie";

---

J'ai mis 'idSalarie' pour la table Salarie, mais note celle de ta clé primaire. Elle peut être 'id' tout simplement (de même dans la clause WHERE).

---

Non, non, non 8O
Bien qu'il soit possible de faire une jointure interne avec un WHERE , c'est une ancienne syntaxe et aujourd'hui on recommande plutôt d'utiliser JOIN

La bonne syntaxe serait au final avec les alias et au plus propre:

Code:

---

1 2 3 4 5

"SELECT leNomDesChamps FROM salarie s INNER JOIN enfants e ON e.idSalarie = $idSalarie"

---

Jamais SELECT * (enfin à éviter)

Ou plus exactement, la bonne requête serait celle-ci :

Code:

---

1 2 3 4

SELECT -- les colonnes nécessaires et pas étoile ! FROM salarie s INNER JOIN enfants e ON e.idSalarie = s.idSalarie WHERE s.idSalarie = $idSalarie

---

Cette requête donnera les infos sur le salarié dont l'identifiant est donné par la variable $idSalarie et sur ses enfants.

À noter que s'il n'a pas d'enfants, la requête ne retournera pas de résultat.
Si vous voulez quand même les infos du salarié même s'il n'a pas d'enfants, il faut utiliser une jointure externe.

@cinephil

Citation:

---

INNER JOIN enfants e ON e.idSalarie = s.idSalarie

---

A cause de la variable :roll: tu utilises le Where ...

Je ne suis pas d'accord, le Where, je l'utilise pour ensuite faire une projection de la jointure. Maintenant donnes moi les raisons et je serais d'accord avec toi
;)

Dans le cas de plusieurs jointures aussi, la dessus je suis d'accord.

Oula , je pensais pas ouvrir un débat avec ma question :mouarf:

Quoi qu'il en soit je vais testé ce que vous me dite et je reviens vers vous pour vous donner le résultat

Dans tous les cas, merci a tous pour votre aide :ccool:


Edit : Nickel ça marche à merveille
Merci encore a tous pour votre aide
bonne journée

Fench, la condition de jointure s'opère entre deux colonnes des deux tables jointes. En principe, il s'agit de la clé primaire d'une table et de la clé étrangère de l'autre.

Faire comme tu le fais revient à mélanger la condition de jointure et la condition de restriction. Ça marche (sous MySQL, les autres SGBD je ne sais pas) mais ce n'est pas propre.

D'ailleurs, à la réflexion, je me demande si ne pas faire correctement la condition de jointure n'entraîne pas un produit cartésien des deux tables.

Donnons une valeur à $idSalarie dans cette requête :

Code:

---

1 2 3

SELECT -- les colonnes nécessaires et pas étoile ! FROM salarie s INNER JOIN enfants e ON e.idSalarie = 12

---

Ne serait-elle pas équivalente à celle-ci ?

Code:

---

1 2 3

SELECT -- les colonnes nécessaires et pas étoile ! FROM salarie s, enfants e WHERE e.idSalarie = 12

---

Citation:

---

Envoyé par Fench

Bonjour,

Comme entre double quote, il y a évaluation des variables, tu peux faire:

Code:

---

1 2	$sql="SELECT * FROM salarie INNER JOIN enfants ON enfants.idSalarie=$idSalarie";

---

Ensuite, je ne comprends pas trop bien

La requête va te retourner une ressource et non une chaine, tab, structure !

Comment tu exploites cette requête ?

---

j'assigne à une variable la fonction mysql_fetch_assoc($sql)

Citation:

---

Fench, la condition de jointure s'opère entre deux colonnes des deux tables jointes. En principe, il s'agit de la clé primaire d'une table et de la clé étrangère de l'autre.

---

Oui je suis d'accord :P

Citation:

---

Faire comme tu le fais revient à mélanger la condition de jointure et la condition de restriction. Ça marche (sous MySQL, les autres SGBD je ne sais pas) mais ce n'est pas propre.

---

Bein non car la variable est un id, donc c une jointure simple (le produit cartésien comme tu dis), le where sera mis ensuite pour réduire cet ensemble.

Citation:

---

Donnons une valeur à $idSalarie dans cette requête :

Code SQL :Sélectionner tout - Visualiser dans une fenêtre à part123 SELECT -- les colonnes nécessaires et pas étoile !
FROM salarie s
INNER JOIN enfants e ON e.idSalarie = 12
Ne serait-elle pas équivalente à celle-ci ?

Code SQL :Sélectionner tout - Visualiser dans une fenêtre à part123 SELECT -- les colonnes nécessaires et pas étoile !
FROM salarie s, enfants e
WHERE e.idSalarie = 12

---

Ici, tu fais nouvelle méthode (la recommandée) et ancienne méthode qui marche aussi.

Maintenant, différences sur les deux :roll:

Citation:

---

Envoyé par stefano

j'assigne à une variable la fonction mysql_fetch_assoc($sql)

---

Oui c'est Ok, j'avais juste peur qd tu disais

Citation:

---

Pour le moment, pour voir ce ce que me retourne la reqête j'utilise un print_r

---

Donc rien à dire ;)

Je viens d'essayer les requêtes avec et sans le WHERE

Avec le WHERE, seul les enregistrements concerner par l'égaliter sont retournés
Si je prends la version sans le WHERE, c'est l'ensemble des données des 2 table qui me sont retournés

Oui,

Si tu fais:

Code:

---

1 2 3 4

SELECT -- les colonnes nécessaires et pas étoile ! FROM salarie s INNER JOIN enfants e ON e.idSalarie = s.idSalarie

---

tu auras l'ensemble des deux tables :)

Et

Code:

---

1 2 3 4 5

SELECT -- les colonnes nécessaires et pas étoile ! FROM salarie s INNER JOIN enfants e ON e.idSalarie = s.idSalarie WHERE s.idSalarie = $idSalarie

---

tu auras une restriction à ceux qui correspondent à l'égalité :)

MAIS

Code:

---

1 2 3 4

SELECT -- les colonnes nécessaires et pas étoile ! FROM salarie s INNER JOIN enfants e ON e.idSalarie = $idSalarie

---

à tester ;)

[edit] oui bon la meilleur méthode est la deux :mouarf: désolé à tous :oops:
car pour la mienne je pensais trop Doctrine qui lors d'une jointure prend en considération les liens et donc pas besoin du ON, bon ça coute rien de tester qd même.

French,

effectivement ça donne la même chose :( j'avais tu me faire une faute de frappe tout à l'heure )

Si ça fonctionne tant mieux pour toi mais sûrement pas tel que tu le présentes !

Citation:

---

Code:

---

1 2	$sql="SELECT * FROM salarie INNER JOIN enfants ON enfants.idSalarie=$idSalarie";

---

---

$sql est ici une simple chaîne de caractères.

Citation:

---

j'assigne à une variable la fonction mysql_fetch_assoc($sql)

---

Faire un mysql_fetch_assoc sur une chaîne de caractères devrait te renvoyer une belle erreur !

Il manque un mysql_query entre les deux !

Donc le bon code devrait être celui-ci :

Code:

---

1 2 3 4 5 6 7 8 9 10

$sql = " SELECT -- les colonnes nécessaires et pas étoile ! FROM salarie s INNER JOIN enfants e ON e.idSalarie = s.idSalarie WHERE s.idSalarie = $idSalarie ";   $result = mysql_query($sql);   $row = mysql_fetch_assoc($result);

---

Citation:

---

Envoyé par Fench

je pensais trop Doctrine

---

Eh oui ! L'enDoctrinement a souvent fait des ravages dans l'Histoire humaine et il continue d'en faire ! :roll:

salut,

@fench, là je soutiens cinephil sans problème... va falloir que tu relises mieux tes cours de sgbd...

ou mieux, autres que ceux de la fac ou des écoles d'ingénieurs qui utilisent encore des notations et présentations des choses comme avant 1992... (vu le nombre d'étudiants qui ne connaissent pas les opérateurs de jointure modernes, enfin si depuis la norme de 1992 on peut appeler ça moderne...)

attention, comme tu le vois le formalisme, parfois mal appliqué, ça tue :mouarf:

une seule erreur de la part de cinephil (mais je pense qu'il a dit ça au vu du cas ici), c'est que la jointure n'est pas restreinte à une condition sur 1 seule colonne commune à chaque table en jeu...

:zoubi:

le principe d'une jointure est de ne pas arrivé au cas dégénéré du produit cartésien... sauf si tu fais du "cross join" qui est le produit cartésien, bien sur...

le principe de la jointure thêta, ancien nom des jointures (pour les diférencier du produit cartésien car tu appliques une condition thêta) est d'obtenir une restriction des jeu de données, grâce à des sélection via les conditions, avant l'application du produit cartésien et non après... ce qui n'induirait aucune optimisation...

et, pour rectifier une autre bêtise, c'est la partie entre select et from qui est la project, car c'est select qui est l'opérateur du projection...

@stefano... passe à pdo pour te connecter à mysql car le jour où ils vont virer mysql et mysqli comme connecteurs, ton application va planter net!!!

et faites gaffe à `` qui sert à encadrer le noms d'entités sql (table, bd, etc...) et '' qui encadre une chaine de caractères... y a eu un méchant flottement à un moment et le sgbd lui ne pardonnera pas la confusion...
:aie:

comme tu as un inner join tu peux faire sauter le where avec:

Code:

---

1 2 3

SELECT -- les colonnes nécessaires et pas étoile ! FROM salarie s INNER JOIN enfants e ON e.idSalarie = s.idSalarie and e.idSalarie = $idSalarie

---

attention si tu utilise "on" la condition de jointure doit être complètement donnée :zoubi:

Salut à tous,

Pour Eric, non je n'ai pas besoin de relire mes cours et je pense que tu as pas tout lu dans les réponses, merci à toi de les relires, vlan, :scarymov:

Bon ça c'est pour le premier réglement de compte :mouarf:

Sinon,

Citation:

---

le principe d'une jointure est de ne pas arrivé au cas dégénéré du produit cartésien... sauf si tu fais du "cross join" qui est le produit cartésien, bien sur...

le principe de la jointure thêta, ancien nom des jointures (pour les diférencier du produit cartésien car tu appliques une condition thêta) est d'obtenir une restriction des jeu de données, grâce à des sélection via les conditions, avant l'application du produit cartésien et non après... ce qui n'induirait aucune optimisation...

---

Pour résumer:

Code:

---

1 2 3 4

Produit cartésien SELECT CLI_NOM, TEL_NUMERO FROM T_CLIENT C, T_TELEPHONE T

---

et

Code:

---

1 2 3 4 5 6

Jointure SELECT CLI_NOM, TEL_NUMERO FROM T_CLIENT C, T_TELEPHONE T WHERE C.CLI_ID = T.CLI_ID OU voir version 2 que je disais bonne avec le inner join + on

---

Tu as vu, je sais donc encore lire des cours :whistle:

Citation:

---

et, pour rectifier une autre bêtise, c'est la partie entre select et from qui est la project, car c'est select qui est l'opérateur du projection...

---

Bon là, je suis d'accord, j'ai confondu la projection et la sélection à cause du select sans doute donc erreur de ma part, 'sélection' :pastaper: si si je reconnais que j'en ai le droit aussi :mouarf:

Et enfin, j'aime pas ta version du AND, ça c parceque tu m'as énervé ...

:chin:

Citation:

---

Et enfin, j'aime pas ta version du AND

---

Moi non plus !
Je préfère séparer la condition de jointure de la condition de restriction, sauf bien sûr en cas de restriction sur la table externe d'une jointure externe.

même pas mal :zoubi:
j'ai un casque casque lourd et un casque de chantier au pire moins bon contre les marteaux mais bon... :mouarf:

j'ai pas dit ça méchamment :kiss:

l'opérateur ',' c'est le mal dans from comme '*' dans select...

lis la doc mysql pour t'en convaincre...

il vaut mieux toujours employer un opérateur de jointure explicite qui garantit que le lecteur humain et l'interpréteur comprennent bien ce que tu veux faire et ne pas compter sur les risques d'un changement d'interprétation d'un sgbd à l'autre...

et entre nous mois non plus j'en suis pas fan car ça enlève la lisibilité de où s'applique la restriction...

mais c'est pas moi qui veut faire des inner join sans where, le but c'est pas de minimiser la longueur, l'optimiseur est la pour ça, mais d'écrire des trucs facilement maintenable et produisant quand même la requête la plus efficace :mouarf:

ici tu peux le faire sans soucis car la restriction s'applique à la condition de jointure directement, ça garde un sens... mais ça reste un cas particulier...

il vaut mieux en effet ne mettre dans le on que ce qui restreint la table externe...:ccool:

Bonjour a tous,

Je reviens vers vous car j'ai toujours un problème avec php et Mysql

Dans ma table "Enfants" j'ai 3 enregistrements pour un même salarié hors lorsque je veux affichié le résultat dans un tableau il me ressort bien 3 enregistrement sauf que c'est le même

Code:

---

1 2 3 4 5 6 7 8 9 10

$sqlEnf="SELECT nomEnfant, prenomEnfant, dateNaissance, nivScolaire FROM enfants INNER JOIN salarie ON enfants.idSalarie=salarie.idSalarie AND salarie.idSalarie=$idSalarie";   $reqEnf=$connexion->requeteSelection($sqlEnf); $infoEnf=$reqEnf->fetch(); $lignesTotalRecu=$reqEnf->rowCount();

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

function requeteSelection($requete){ try { if($this->connexionActive() == false) { return null; } else { $this->donnees = $this->connexion->query($requete); return $this->donnees; } } catch(PDOException $e) { echo "Echec lors de l'execution de la requete : ".$e->getMessage(); return null; } }

---

et le bout de code pour remplir le tableau

Code:

---

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

<tbody> <tr> <?php $donnees=array(); $i=0;   while ($i<$lignesTotalRecu ){   $donnees['nom'][$i]=$infoEnf['nomEnfant']; $donnees['prenom'][$i]=$infoEnf['prenomEnfant']; $donnees['dateNaissance'][$i]=$infoEnf['dateNaissance']; $donnees['nivScolaire'][$i]=$infoEnf['nivScolaire'];   echo '<td>' .$donnees['nom'][$i]. '</td><td>'.$donnees['prenom'][$i]. '</td><td>'.$donnees['dateNaissance'][$i]. '</td><td>'.$donnees['nivScolaire'][$i].'</td>';   $i ++; }   ?> </tr> </tbody>

---

Si vous pouvez me dire ou je me plante car je sèche


Merci par avance

si c'est la méthode query de pdo, ça te renvoie un objet (l'équivalent de ta ressource avec le connecteur mysql)... lis la doc... cet objet est de type PDOStatement...

ou alors faut que tu montres ce que tu mets dans connexion::query()

or tu utilises fetch() qui ne retourne qu'une seule ligne... ça devrait être fetchAll() si tu veux générer ton tableaux en une seule fois avant ta boucle de lecture...

autre chose, pourquoi repasser par un tableau ($donnees) avant d'afficher????

tu alloues de la mémoire inutilement et allourdis le temps d'exécution et la relecture pour un humain pour rien... $infoEnf est largement suffisant et parlant...

:aie:

quitte à encapsuler autan y aller franco...

Code:

---

1 2 3

$reqEnf=$connexion->requeteSelection($sqlEnf); $infoEnf=$reqEnf->fetchAll(); $lignesTotalRecu=$reqEnf->rowCount();

---

devrait être dans requeteSelection()
de même, que de mettre un echo dans cette méthode qui va s'afficher en vrac dans ta page... il faut apprendre à mieux gérer ton code et les erreurs...

et pourquoi stocker un retour de requête dans $this qui est ta connexion?

l'idée est de faire un tableau avec 3 parties qui sera renvoyé (moins lourd qu'un objet vu ce qu'on a besoin de faire derrière):

• ['count'] qui contient le nombre de ligne
• ['error'] qui contient le texte de l'erreur ou null
• ['data'] qui contient les données ou null

ça donne pour :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

function requeteSelection($requete) { try { if($this->connexionActive() == false) return array('count'=>0,'error'=>'Connexion inactive','data'=>null); else { $donnees = $this->connexion->query($requete); return array('count'=>$donnees->rowCount(),'error'=>null,'data'=>$donnees->fetchAll(PDO::FETCH_ASSOC)); } } catch(PDOException $e) { echo "Echec lors de l'exécution de la requête : ".; return array('count'=>0,'error'=>$e->getMessage(),'data'=>null); } }

---

il vaut mieux fixer le type de retour attendu dans fetchAll (ou fetch)... ici on retourne avec PDO::FETCH_ASSOC seulement le tableau associatif au lieu d'avoir à la fois la version associative et numérique pour l'accès des colonnes (ce qui démultiplie le temps de récupération de la requête et la taille en mémoire)

comme ça c'est plus propre et simple... tu as toujours un tableau avec toujours la même structure facile à analyser...

ensuite ton tableau html ne se justifie que si il n'y a pas d'erreur d'où parfois la mauvaise idée de ne pas générer ça en pure php plutôt que de trop mixer html/php

la génération du tableau des enfants donnerait alors:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

<?php $sqlEnf="SELECT nomEnfant, prenomEnfant, dateNaissance, nivScolaire FROM enfants INNER JOIN salarie ON enfants.idSalarie=salarie.idSalarie AND salarie.idSalarie=$idSalarie"; $lignes=connexion->requeteSelection($sqlEnf); if(empty($lignes[error]) { echo'<table><!-- insere ton entete de tableau à la place de ce commentaire--><tbody>'; for($i=0;$i<$ligne['count'];$i++) echo'<tr><td>'.$lignes['data'][$i]['nom']. '</td><td>'.$lignes['data'][$i]['prenom']. '</td><td>'.$lignes['data'][$i]['dateNaissance']. '</td><td>'.$lignes['data'][$i]['nivScolaire'].'</td></tr>'; echo'</tbody></table>'; } else echo"<div>Une erreur c'est produite:{$lignes[error]}</div>"; ?>

---

j'ai l'impression que tu confonds aussi colonne et lignes dans ton tableau dans ton code, vu là où tu as mis le [$i]

voilà je pense que ça devrait résoudre ton problème les les morceau de code que tu as donné...

peut-être que cette méthodologie te fera mieux voir comment tu peux gérer un affichage de tableau ou d'erreur... à adapter à ce que tu veux très facilement ensuite

:ccool:

Merci pour toutes ces explications, C'est vrai que je suis pas encore très à l'aise avec le php et encore moins avec le PDO.
J'ai eu quelques cours de php et comme ce sont les vacances j'ai pas envie de perdre le peu que j'ai appris.

Quoi qu'il en soit je te remercie pour tes explications et ta patience

Merci

EDIT : c'est bon ça fonctionne nickel chrome :ccool: