affectation de variables dans requete sql

Version imprimable

05/09/2020, 19h28
marjan

affectation de variables dans requete sql

Bonjour,

Malgré de longues heures de recherche, je n'arrive pas à trouver la syntaxe d'un DELETE ou d'un SELECT.
JE METS LES VARIABLES SUIVANTES :
$c = table de la bdd
$n = nom
$p = prenom

Je voudrais écrire la ligne suivante :

Code:

$reponse = $bdd->query(DELETE FROM '.$c.' WHERE Nom ='.$n.' AND Prenom = '.$p.');

ou éventuellement par une requête préparée.

Code:

$reponse = $bdd -> prepare (......................)

Quelqu'un peut-il me dépanner svp?
D'avance MERCI.
06/09/2020, 05h52
Dendrite

Ceci devrait te convenir.

https://www.developpez.net/forums/bl...pdo-soupe-lit/
06/09/2020, 16h15
marjan

merci de l'adresse.
Je vais approfondir les subtiités sql.
les quotes, double cotes etc n'auront plus de secret pour moi

Je n'arrive vraiment pas à percer le mystère des guillemets ou apostrophes malgré la lecture que l'on m'a conseillée.
Requêtes qui ne marchent pas :
Code:

1 2 3 $bdd->exec(DELETE FROM ".$_POST['$c']." WHERE PrénomMembre = ".$_POST['$p']." AND NomMembre = ".$_POST['$n']." ); $bdd->exec('DELETE FROM ".$_POST['$c']." WHERE PrénomMembre = ".$_POST['$p']." AND NomMembre = ".$_POST['$n']." '); $bdd->exec('DELETE FROM $_POST['$c'] WHERE PrénomMembre = $_POST['$p'] AND NomMembre = $_POST['$n'] ');
S'il vous plait, j'aurai vraiment besoin d'un coup de mains.

07/09/2020, 07h31
cavo789

Bonjour

Citation:

Envoyé par marjan

Code:

$bdd->exec('DELETE FROM ".$_POST['$c']." WHERE PrénomMembre = ".$_POST['$p']." AND NomMembre = ".$_POST['$n']." ');

Quand tu commences avec un ' il faut fermer avec un ' (même règle pour le guillemet).

Et donc

Code:

$bdd->exec("DELETE FROM ".$_POST['$c']." WHERE PrénomMembre = '".$_POST['$p']."' AND NomMembre = '".$_POST['$n']."'");

Tu définis en fait des sous-chaînes qui seront concaténées.

Code:

$bdd->exec("SOUS-CHAINE 1" . "SOUS-CHAINE 2" . "SOUS-CHAINE 3");

Ceci compris, tu dois penser à tes critères : c'est du texte et donc quelque chose comme WHERE NomMember='NESTOR' ==> "WHERE NomMember='".$_POST['$n']."'"Note: je programme toujous en anglais aussi je suis étonné de voir que tu as un nom de champ avec un accent. Est-ce que PrénomMembre est réellement une colonne ? (je n'ai jamais testé; préférant ici FirstName)

Note bis (en gras) : Ton code comme ci-dessus est extrêmement dangereux car pas sécurisé. Si tu as réellement quelque chose comme ça, jete un oeil à la technique dite SQL Injection. Là, en principe, avec ton code, je pourrais attaquer ton script PHP et p.ex. supprimer l'intégralité de ta base de données en injectant un DROP DATABASE comme ... nom de famille.

Bonne journée.
07/09/2020, 16h31
marjan

Merci à tous pour les conseils.

Lorsque je mets textuellement (couper-coller) l'écriture proposée, j'ai l'erreur suivante :

Citation:

Fatal error: Uncaught PDOException: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'WHERE PrénomMembre = '' AND NomMembre = ''' at line 1 in /customers/5/4/0/lXXXXXXX.fr/httpd.www/effacer.php:18 Stack trace: #0 /customers/5/4/0/XXXXXXXXXX.fr/httpd.www/effacer.php(18): PDO->exec('DELETE FROM WH...') #1 {main} thrown in /customers/5/4/0/XXXXXXXXXX.fr/httpd.www/effacer.php on line 18

D'autre part, les accents dans le nom des colonnes ne m'ont jamais posé problème.
J'ai un tas de scripts qui utilisent prénomMembre et ça fonctionne.
Ça m'embête de changer le nom de la colonne, car j'ai vraiment un tas de scripts qui se servent de PrénomMembre.

Enfin, lorsque la syntaxe sera comprise, je changerai bien évidemment la requête en requête préparée pour plus de sécurité.
07/09/2020, 20h09
cavo789

Salut

Peux-tu faire un echo de ton query complet ?

Pour le nom de la colonne ne change rien, je faisais juste part de mon étonnement car cela ne me serait jamais venu à l'esprit d'utiliser un accent (je code en anglais). Si cela fonctionne, ne change rien 😉

Une petite question :
Qu'entends-tu par echo du query ?

Si je remplace les variables $c,$p,$n par des valeurs en dur tout marche bien.

Code:

1
2
3
4
5
6
7
 
$bdd = new PDO('mysql:host=XXXXXX.mysql;dbname=XXXXXX', 'XXXXXX, 'YYYYYY', array (PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES \'UTF8\''));
$bdd->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
$c = "conferences"
$p = René
$n = DUPONT"
$bdd->exec("DELETE FROM ".$_POST['$c']." WHERE PrénomMembre = '".$_POST['$p']."' AND NomMembre = '".$_POST['$n']."'");

07/09/2020, 20h41
cavo789
Ta question me surprend... Par echo je pensais ... echo ;)
Code:

1 2 3 4 5 6 7 8 <?php $c = "conferences"; $p = "René"; $n = "DUPONT"; echo "DELETE FROM ".$c." WHERE PrénomMembre = '".$p."' AND NomMembre = '".$n."'";
Ce qui me donne DELETE FROM conferences WHERE PrénomMembre = 'René' AND NomMembre = 'DUPONT'Et cela me semble être un SQL correct toutefois; dans ton code, en lignes 5 et 6, ton code PHP est incorrect (il manque un guillemet d'ouverture en ligne 6, tu n'as pas de guillemet en ligne 5 et tu as oublié tes points virgules à la fin de deux lignes); ton code est donc non fonctionnel pour cette raison.
07/09/2020, 20h44
cavo789

Note aussi, à relire ton SQL, je ne comprends pas ce que font les champs PrénomMembre et NomMembre dans la table conference.

J'ai l'impression (à toi de valider) que tu devrais avoir une table Membre avec un champ ID, un champ Nom, un champ Prénom, etc. et dans la table Conference, juste le MembreID.

Conference et Membre sont deux objets totalement distincts, deux tables donc.
07/09/2020, 22h00
marjan

Merci beaucoup. J'ai testé, ça marche.
Pour les erreurs PHP, c'est dû au copier-coller que j'ai fait entre mon script et la réponse que je t'ai faite.
J'avais bien les guillemets, les points-virgule dans mon script.
En ce qui concerne la table conférences j'aurai pu faire comme tu me le suggère -> table membres -->champs NomMembre, PrénomMembre,ID.
je ferai probablement la transformation plus tard.

Il ne me reste plus qu'à faire la requête préparée.

Je te remercie, j'ai passé un temps fou avec ces guillemets et quote, j'ai une grosse lacune dans les concaténations. (je ne pensais pas..!)

Encore merci.
07/09/2020, 22h08
cavo789

Citation:

Envoyé par marjan

je ferai probablement la transformation plus tard.

Au plus tard tu le feras, au plus de boulot de refactoring tu seras confronté ;)
08/09/2020, 10h16
Dendrite

Citation:

Envoyé par marjan

j'ai passé un temps fou avec ces guillemets et quote, j'ai une grosse lacune dans les concaténations. (je ne pensais pas..!)

Oui, ce truc fait un peu mal à la tête au début.
Un des avantages des requêtes préparées (voir mon premier lien), c'est que tu ne t'occupes pas de cet aspect (gérer les guillemets SQL selon le type de données).
Mais bon, comme tu devais travailler tes concaténations, tu n'as pas perdu ton temps. :mrgreen:

Sinon, oui, dans une table SQL, la bonne pratique dit "pas d'accents et que des minuscules".
Si tu veux séparer les mots, le underscore est ton ami (ton_meilleur_ami). Mais bon, tu le sauras pour ton prochain développement.