Variable de type INT dans un SELECT WHERE

Version imprimable

Voir 40 message(s) de cette discussion en une page

23/09/2019, 12h05
gdelarminat

Variable de type INT dans un SELECT WHERE
Bonsoir à tous,

Je m'arrache les cheveux sur un problème apparemment basique, mais pas tant que ça...

Pour avoir dans un formulaire une liste déroulante, ça fonctionne parfaitement avec cet extrait de code (pour avoir une liste déroulante qui n'affiche que les fournisseurs dont l'id_echoppe =2
Code :

Code:

$reponse = $bdd->query("SELECT * FROM fournisseur WHERE id_echoppe='2'");

Or je veux récupérer l'id_echoppe via une variable POST, mais cela ne fonctionne pas
Code:

1 2 $id_echoppe = $_POST['id_echoppe']; $reponse = $bdd->query("SELECT * FROM fournisseur WHERE id_echoppe='$id_echoppe'");
Je pensais que le problème venait de mon post, mais en faisant un printr de $id_echoppe il me renvoie bien la valeur 2.

Et si je code ainsi pour "forcer" la valeur 2 dans ma variable (pour comprendre) cela ne fonctionne pas plus :
Code:

1 2 $id_echoppe =2; $reponse = $bdd->query("SELECT * FROM fournisseur WHERE id_echoppe='$id_echoppe'");
J'ai essayé toutes les solutions : variable encadrée ou non par les points, quotes précédées de \... etc... rien n'y fait...

D'où cela peut-il venir ?

D'avance merci.
23/09/2019, 12h16
rawsrc

Salut,

et comme ça t'as essayé :

Code:

$reponse = $bdd->query('SELECT * FROM fournisseur WHERE id_echoppe = '.(int)$id_echoppe);
23/09/2019, 12h40
gdelarminat
Top !! Effectivement ça fonctionne maintenant parfaitement ! Même sans le (int) cela fonctionne. De ce que je comprend mon erreur était donc une erreur de syntaxe. Je croyais qu'il fallait toujours commence et finir la requête par une simple quote comme ici ? :
Code:

1 2 $id_echoppe = $_POST['id_echoppe']; $reponse = $bdd->query('SELECT * FROM fournisseur WHERE id_echoppe='$id_echoppe'');
Là où ce que tu m'indiques, et qui fonctionne devient :
Code:

1 2 $id_echoppe = $_POST['id_echoppeF']; $reponse = $bdd->query('SELECT * FROM fournisseur where id_echoppe='.$id_echoppe);
C'est donc bien ça ? La référence à une variable doit toujours sortir des quotes ? Et être précéder d'un point (.) ?

Merci pour tout en tout cas...
23/09/2019, 12h57
rawsrc

attention, t'emballe pas.
Vu que tu ne prépares pas la requête, tu ne vires surtout pas le cast (le (int) devant la variable). On a la certitude que la valeur qui sera passée au sql sera un entier.
Un entier doit être passé au sql tel quel, sans artifice.
Une chaîne de caractère doit être entourée de guillemets et surtout échappée (ses caractères dangereux rendus inoffensifs)
Ou mieux, bascule sur PDO : PDO, une soupe et au lit
23/09/2019, 14h04
Celira

Citation:

Envoyé par gdelarminat

C'est donc bien ça ? La référence à une variable doit toujours sortir des quotes ? Et être précéder d'un point (.) ?

ça dépend de la syntaxe que tu utilise pour construire la chaine :arrow: Apostrophes ou guillemets : lesquels choisir ?

Effectivement tu as raison ;-). Ce que je voulais dire c'est qu’au-delà du (int) j'avais aussi un problème de syntaxe. D'ailleurs je rencontre maintenant un autre souci avec le reste du code.

En gros, j'ai 2 tables, une table produit (id, nom, id_fournisseur), et une table fournisseur (id, nom, id_echoppe). Je souhaite afficher la liste des produits dont les fournisseurs appartiennent à un échoppe (sorte de magasin) particulière (que je récupère dans une variable via un $_POST issue d'une précédente page .php.

Voici mon code ... avec PDO, une soupe et au lit ;-)

J'ai l'impression que ma requette sql est bonne : je fais ma jointure, j'identifie la clé commune, et je mets ma condition via ma variable comme vu au début...
Mais j'ai une erreur :
( ! ) Fatal error: Uncaught Error: Call to a member function fetch() on boolean in C:\wamp64\www\les-zechoppes\dev\bdd\formulaire_bdd\commande_client_2ter.php on line 16
( ! ) Error: Call to a member function fetch() on boolean in C:\wamp64\www\les-zechoppes\dev\bdd\formulaire_bdd\commande_client_2ter.php on line 16

Encore un problème de syntaxe ?
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 26 27 <?php try { $bdd = new PDO('mysql:host=localhost;dbname=bdd_test;charset=utf8', 'root', ''); } catch(Exception $e) { die('Erreur : '.$e->getMessage()); } $id_echoppe = $_POST['id_echoppeF']; $reponse = $bdd->query('SELECT * FROM produit INNER JOIN fournisseur ON produit.idfournisseur=fournisseur.id WHERE fournisseur.id_echoppe='.(int)$id_echoppe); while ($donnees = $reponse->fetch()) { ?> liste des produits : <?php echo $donnees['produit.nom']; ?> <?php } $reponse->closeCursor(); ?>

Bonjour,

:alerte: il faut que tu apprennes à préparer les requêtes :

A LIRE : Comprendre PDO

Code:

1
2
3
4
$sql_select = $bdd->prepare("SELECT * FROM produit INNER JOIN fournisseur ON produit.idfournisseur=fournisseur.id WHERE fournisseur.id_echoppe = :id_echoppe ";
$sql_select->execute([
   ':id_echoppe' => $id_echoppe
   ]);

Ou :

Code:

1
2
3
4
$sql_select = $bdd->prepare("SELECT * FROM produit INNER JOIN fournisseur ON produit.idfournisseur=fournisseur.id WHERE fournisseur.id_echoppe = ? ";
$sql_select->execute([
   $id_echoppe
   ]);

Ou :

Code:

1
2
3
$sql_select = $bdd->prepare("SELECT * FROM produit INNER JOIN fournisseur ON produit.idfournisseur=fournisseur.id WHERE fournisseur.id_echoppe = :id_echoppe ";
$sql_select->bindValue(':id_echoppe', $id_echoppe, PDO::PARAM_INT);
$sql_select->execute();

Puis :

Code:

1
2
while ($donnees = $sql_select->fetch())
{

Merci beaucoup du conseil. Pour autant même en préparant la requête (cf ci dessous) j'ai toujours la même erreur suivante au niveau de la fonction fetch():

Citation:

( ! ) Fatal error: Uncaught Error: Call to a member function fetch() on boolean in C:\wamp64\www\les-zechoppes\dev\bdd\formulaire_bdd\commande_client_2ter.php on line 23
( ! ) Error: Call to a member function fetch() on boolean in C:\wamp64\www\les-zechoppes\dev\bdd\formulaire_bdd\commande_client_2ter.php on line 23

Une idée ?
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 26 27 28 29 30 <?php try { $bdd = new PDO('mysql:host=localhost;dbname=les_zechoppes;charset=utf8', 'root', ''); } catch(Exception $e) { die('Erreur : '.$e->getMessage()); } $id_echoppe = $_POST['id_echoppeF']; $sql_prepare = $bdd->prepare("SELECT * FROM produit INNER JOIN fournisseur ON produit.idfournisseur=fournisseur.id WHERE fournisseur.id_echoppe = :id_echoppe "); $reponse = $sql_prepare->execute([ ':id_echoppe' => $id_echoppe ]); while ($donnees = $reponse->fetch()) { ?> liste des produits : <?php echo $donnees['produit.nom']; ?> <?php } $reponse->closeCursor(); ?>

23/09/2019, 18h12
Invité

Et avec un try catch, ça dit quoi ?

Avec le try/catch autour de la requete j'ai toujours le même message d'erreur:

Citation:

( ! ) Fatal error: Uncaught Error: Call to a member function fetch() on boolean in C:\wamp64\www\les-zechoppes\dev\bdd\formulaire_bdd\commande_client_2ter.php on line 23
( ! ) Error: Call to a member function fetch() on boolean in C:\wamp64\www\les-zechoppes\dev\bdd\formulaire_bdd\commande_client_2ter.php on line 23

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
26
27
28
29
30
31
32
33
34
35
36
<?php
try{$bdd = new PDO('mysql:host=localhost;dbname=les_zechoppes;charset=utf8', 'root', '');}
catch(Exception $e){die('Erreur : '.$e->getMessage());}
 
/****************************************************************************/
 
$id_echoppe = $_POST['id_echoppeF'];
 
/****************************************************************************/
 
try
{
$sql_prepare = $bdd->prepare("SELECT * FROM produit INNER JOIN fournisseur ON produit.idfournisseur=fournisseur.id WHERE fournisseur.id_echoppe = :id_echoppe ");
$reponse = $sql_prepare->execute([':id_echoppe' => $id_echoppe]);
}
catch(Exception $e)
{
        die('Erreur : '.$e->getMessage());
}
 
/****************************************************************************/
 
while ($donnees = $reponse->fetch())
{
?>
    <p>
    <strong>liste des produits</strong> : <?php echo $donnees['produit.nom']; ?>
   </p>
<?php
}
 
/****************************************************************************/
 
$reponse->closeCursor();
 
?>

Idem, même erreur en mettant le try/catch autour du while... Même intitulé d'erreur à la ligne 25...

Citation:

( ! ) Fatal error: Uncaught Error: Call to a member function fetch() on boolean in C:\wamp64\www\les-zechoppes\dev\bdd\formulaire_bdd\commande_client_2ter.php on line 25
( ! ) Error: Call to a member function fetch() on boolean in C:\wamp64\www\les-zechoppes\dev\bdd\formulaire_bdd\commande_client_2ter.php on line 25

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
<?php
try{$bdd = new PDO('mysql:host=localhost;dbname=les_zechoppes;charset=utf8', 'root', '');}
catch(Exception $e){die('Erreur : '.$e->getMessage());}
 
/****************************************************************************/
 
$id_echoppe = $_POST['id_echoppeF'];
 
/****************************************************************************/
 
try
{
$sql_prepare = $bdd->prepare("SELECT * FROM produit INNER JOIN fournisseur ON produit.idfournisseur=fournisseur.id WHERE fournisseur.id_echoppe = :id_echoppe ");
$reponse = $sql_prepare->execute([':id_echoppe' => $id_echoppe]);
}
catch(Exception $e)
{
        die('Erreur : '.$e->getMessage());
}
 
/****************************************************************************/
 
try
{
while ($donnees = $reponse->fetch())
{
?>
    <p>
    <strong>liste des produits</strong> : <?php echo $donnees['produit.nom']; ?>
   </p>
<?php
}
}
catch(Exception $e)
{
        die('Erreur : '.$e->getMessage());
}
 
/****************************************************************************/
 
$reponse->closeCursor();
 
?>

OUPS ! :oops:
Je me suis (complètement) planté dans la syntaxe ! (habituellement, je n'utilise pas les même noms de variables...)
Code:

1 2 3 4 5 $sql_select = $bdd->prepare("SELECT * FROM produit INNER JOIN fournisseur ON produit.idfournisseur=fournisseur.id WHERE fournisseur.id_echoppe = :id_echoppe "); $sql_select->execute([':id_echoppe' => $id_echoppe]); while ($donnees = $sql_select->fetch()) {
(j'ai aussi corrigé mon précédent message...)

J'ai essayé avec ta bonne syntaxe pour "préparer" la requête et je n'ai plus de message d'erreur. Par contre la page est vide !
Sachant que pour info sous phpmyadim la requête fonctionne très bien (quand j’écris la requête sous PMA j'indique comme valeur d'$id_echoppe = 1 et cela me renvoie bien le bon résultat (12 lignes pour info).
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 <?php try{$bdd = new PDO('mysql:host=localhost;dbname=les_zechoppes;charset=utf8', 'root', '');} catch(Exception $e){die('Erreur : '.$e->getMessage());} /****************************************************************************/ $id_echoppe = $_POST['id_echoppeF']; /****************************************************************************/ $sql_select = $bdd->prepare("SELECT * FROM produit INNER JOIN fournisseur ON produit.idfournisseur=fournisseur.id WHERE fournisseur.id_echoppe = :id_echoppe "); $sql_select->execute([':id_echoppe' => $id_echoppe]); while ($donnees = $sql_select->fetch()) { ?> liste des produits : <?php echo $donnees['produit.nom']; ?> <?php } $sql_select->closeCursor();

en complément de mon precedent message,

Si je ne passe pas par ma variable $_POST mais que je force la valeur de mon "Where" à 1 avec le code ci dessous, j'ai alors les 12 erreurs suivantes (qui correspondent au 12 enregistrements que je m'attend à avoir). Visiblement je suis donc bien rentré dans la boucle mais avec une erreur malgré tout. N'est ce pas une piste ? Si on trouve la solution sans passé par la variable peut être pourra t on la rajouter dans un second temps ? Mais j'avoue ne pas comprendre comment corriger ce code en forçant la valeur du WHERE fournisseur.id_echoppe=1 : Probleme de syntaxe la aussi ?

Voici les erreurs (x12)

Citation:

( ! ) Notice: Undefined index: produit.nom in C:\wamp64\www\les-zechoppes\dev\bdd\formulaire_bdd\commande_client_2ter.php on line 16

Et le code :
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 <?php try{$bdd = new PDO('mysql:host=localhost;dbname=les_zechoppes;charset=utf8', 'root', '');} catch(Exception $e){die('Erreur : '.$e->getMessage());} /****************************************************************************/ $reponse = $bdd->query("SELECT * FROM produit INNER JOIN fournisseur ON produit.id_fournisseur=fournisseur.id WHERE fournisseur.id_echoppe=1"); while ($donnees = $reponse->fetch()) { ?> liste des produits : <?php echo $donnees['produit.nom']; ?> <?php } $reponse->closeCursor(); /****************************************************************************/ ?>

23/09/2019, 19h31
gdelarminat

A noter que sur mon dernier message en forçant la valeur de mon WHERE à fournisseur.id_echoppe =1 j'avais donc 12 message d'erreur.
Le souci vient de mon echo j'ai l'impression : echo $donnees['produit.nom'];.

Or si je modifie mon echo ainsi : echo $donnees['nom'] alors je n'ai plus de message d'erreur et j'ai la liste de 12 noms qui s'affiche. Mais pas les bons ! Il affiche le champs nom de la table "fournisseur". Or je veux celui de la table "produit". Raison pour laquelle dans le echo j'avais mis 'produit.nom'... Syntaxe la encore ?
23/09/2019, 20h07
Invité
Ah ben non...
Tu ne peux pas écrire $donnees['produit.nom'], mais $donnees['nom'], sauf que "nom" peut être ambigu si tu as une colonne "nom" dans les 2 tables.

La bonne méthode est de :
- récupérer, via la requête, uniquement les colonnes qu'on veut afficher (pas de "SELECT * ...")
- éventuellement, mettre des ALIAS (AS...)
Par exemple ici :

Code:

"SELECT produit.nom AS produit_nom, ..."

Par conséquent, à l'affichage :

Code:

echo htmlspecialchars($donnees['produit_nom'])

IMPORTANT : on utilise TOUJOURS htmlspecialchars() à l'AFFICHAGE.

Yesssssss !

J'obtiens bien le résultat escompté. Le problème venait bien du champs "nom" présent dans les deux tables...
Ce qui donne ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
$reponse = $bdd->query("SELECT produit.nom as nom_produit, fournisseur.nom as nom_fournisseur FROM produit INNER JOIN fournisseur ON produit.id_fournisseur=fournisseur.id WHERE fournisseur.id_echoppe=1");
 
while ($donnees = $reponse->fetch())
{
?>
    <p>
    <?php echo 'Produit '.$donnees['nom_produit'].' du fournisseur '.$donnees['nom_fournisseur']; ?>
   </p>
<?php
}
 
$reponse->closeCursor();

Maintenant j'ai toujours mon problème quand je veux passer mon fournisseur.id_echoppe avec une variable. Si je résume nos échanges et tes conseils je devrais l'écrire ainsi. Or ma page est vierge ! Rien ne s'affiche !

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<?php
try{$bdd = new PDO('mysql:host=localhost;dbname=les_zechoppes;charset=utf8', 'root', '');}
catch(Exception $e){die('Erreur : '.$e->getMessage());}
 
/****************************************************************************/
 
$id_echoppe = $_POST['id_echoppeF'];
 
$sql_select = $bdd->prepare("SELECT produit.nom as nom_produit, fournisseur.nom as nom_fournisseur FROM produit INNER JOIN fournisseur ON produit.idfournisseur=fournisseur.id WHERE fournisseur.id_echoppe = :id_echoppe ");
$sql_select->execute([':id_echoppe' => $id_echoppe]);
 
while ($donnees = $sql_select->fetch())
{
?>
    <p>
    <?php echo 'Produit '.$donnees['nom_produit'].' du fournisseur '.$donnees['nom_fournisseur']; ?>
   </p>
<?php
}
 
$sql_select->closeCursor();
?>

En complément de mon précédent message, le problème semble venir de la manière de coder la requete preparé. Car sans requete preparé, ça marche nickel ainsi ! Ce qui solutionne donc mon problème mais j'aurais aimé comprendre pourquoi ça fonctionne pas en "préparé"...

Voici le code qui fonctionne :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 $id_echoppe = $_POST['id_echoppeF']; $reponse = $bdd->query("SELECT produit.nom as nom_produit, fournisseur.nom as nom_fournisseur FROM produit INNER JOIN fournisseur ON produit.id_fournisseur=fournisseur.id WHERE fournisseur.id_echoppe=$id_echoppe"); while ($donnees = $reponse->fetch()) { ?> <?php echo 'Produit '.$donnees['nom_produit'].' du fournisseur '.$donnees['nom_fournisseur']; ?> <?php } $reponse->closeCursor();

23/09/2019, 20h54
Toufik83
Salut,

Et avec (int), toujours le même problème ? (rawsrc te l'a suggéré dans son post)
Code:

1 2 $sql_select->execute([':id_echoppe' =>(int)$id_echoppe]);

Effectivement j'ai finis par l’oublier lui ;-)
Mais ça ne marche pas plus. Pas d'erreur, mais page vierge...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
$id_echoppe = $_POST['id_echoppeF'];
 
$sql_select = $bdd->prepare("SELECT produit.nom as nom_produit, fournisseur.nom as nom_fournisseur FROM produit INNER JOIN fournisseur ON produit.idfournisseur=fournisseur.id WHERE fournisseur.id_echoppe = :id_echoppe ");
$sql_select->execute([':id_echoppe' => (int)$id_echoppe]);
 
while ($donnees = $sql_select->fetch())
{
?>
    <p>
    <?php echo 'Produit '.$donnees['nom_produit'].' du fournisseur '.$donnees['nom_fournisseur']; ?>
   </p>
<?php
}
 
$sql_select->closeCursor();

la doc de PDO indique que tous les paramètres sont par défaut des string, donc il faut lier ton paramètre entier séparément.

Code:

1
2
3
$sql_select = $bdd->prepare("SELECT produit.nom as nom_produit, fournisseur.nom as nom_fournisseur FROM produit INNER JOIN fournisseur ON produit.idfournisseur=fournisseur.id WHERE fournisseur.id_echoppe = :id_echoppe ");
$sql_select->bindValue(':id_echoppe', (int)$id_echoppe, PDO::PARAM_INT);
$sql_select->execute();

Voir 40 message(s) de cette discussion en une page