UPDATE Nb Param invalide en requête Résolu/Fermé

Question

Bonjour, J'ai connu la programmation séquentielle et il est certain que le programmation objet m'échappe un peu (beaucoup) ! Face à ce type d'erreur, je me sens quelque peu démuni ! Erreur de requÃ¨te : SQLSTATE[HY093]: Invalid parameter number: parameter was not defined Malgré tout, j'ai tenté de corriger le nombre de paramètre communiqué en y ajoutant admail bien que n'ayant pas à le mettre à jour mais l'utilisant en sélection (WHERE). Pour autant, l'erreur persiste !! J'avais buté sur la manière d'exprimer les conditions pour le WHERE mais n'attendait pas la difficulté sous cette forme ! Le problème à résoudre est simple : Je présume initialement que les adhérents voudront recevoir Comptes rendus et Programmes (difCR et difProg). Bien sûr, j'ai modifié la pratique en corrigeant dès la saisie si un adhérent n'a pas communiqué de mail et ils peuvent demander en modification le changement. Pourtant, à la création, j'ai initialisé avec la diffusion pour tous et dois donc supprimer cet excès lorsqu'il n'y a pas d'adresse mail (admail). Noter que l'admail est vérifié à la saisie et donc simplement contôlé < " " ou non. J'indique le code : include_once("ConnectBDutl.php"); // Connexion à MySQL (users enregistrés) /* Création de la requête préparée : Effacement de toutes les lignes de la table adh dont cle > $cle */ $admail = " "; $pdo_options[PDO::ATTR_ERRMODE] = PDO::ERRMODE_EXCEPTION; $req = $connexion->prepare('UPDATE adh SET difCR = :difPR, difProg=:difProg , admail=:admail WHERE (admail <= $admail AND (difPR == 1 OR difProg ==1))'); try { $req->execute(array('difCR' => 0, 'difProg' => 0, 'admail' => $admail )); $count = $sth->rowCount(); echo $count,utf8_decode(" enregistrements modifiés"); } catch( Exception $e ){ echo 'Erreur de requète : ', $e->getMessage(); } Comment faut-il faire avec les paramètres à ne pas mettre à jour ? ici 3 utilisés mais 2 mis à jour. Comment sont-ils comptés ? Merci de tes découvertes parmi mes errances ;Dur dur ! RJL2831 Configuration: Windows 7 / Firefox 15.0

Utilisateur anonyme · Answer

Toujours distrait ?

tu appelles un paramètre :difPR
mais dans ton tableau, tu donnes difCR comme clé...
Alors, difPR ou difCR ?

rjl · Answer

Ca s'est arrangé au plan du nombre de paramètres (sauf si je tentes de ne pas traiter en paramètre admail que je n'exploite en réalité qu'en WHERE sans avoir à le modifier) ! 
Pour le reste, j'ai tenté des ' et des " pour distinguer les champs, j'avais aussi tenté sans ' pour les nombres mais ça n'allait pas car c'difCR et difProg sont en Char sur 1 caractère (défini avec * ou " " à l'origine) ! 
Quelles que soient mes tentatives, ça échoue : J'ai bien remarqué que pour difCR le chiffre de comparaison ou affectation est en rouge amis en noir pour le second paramètre difProgen comparaison et affectation mais sans comprendre le pourquoi et l'origine (même définition en structure de table, ! 

Erreur de requÃ¨te : SQLSTATE[42000]: Syntax error or access violation: 1064 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 'AND (difCR == '1' OR difProg =='1'))' at line 2 
code : $req = $connexion->prepare("UPDATE adh SET difCR = :difCR, difProg= :difProg  WHERE                 (admail <= $admail 
                    AND (difCR == "1" 
                     OR    difProg == "1"))"); 
try { 
  $req->execute(array('difCR'  => "0", 
       'difProg' => "0" 
       )); 
Merci d'avance pour ton eil expert!          RJL2831

Utilisateur anonyme · Answer

La programmation avec le PHP est objet (enfin, un peu) mais cela ne l'empêche pas d'être séquentielle. Les deux ne sont pas incompatibles. Ce qui serait incompatible avec séquentiel serait, par exemple, événementiel.

Trois remarques

1 - j'aurais dû le voir avant. La comparaison (dans les WHERE en particulier) en SQL c'est =, pas ==. J'espère que tu ne confonds pas avec la comparaison du php. 
Une requête SQL est une chaîne qui suit les règles de syntaxe de SQL. Elle est "fabriquée" par php à partir de ton code et en remplaçant éventuellement les paramètres par leurs valeurs. Elle est analysée par le moteur de base de données. Mais le php en lui même n'en a rien à faire du contenu de cette chaîne.
2 - Les paramètres passés à une requête PDO doivent avoir des noms différents. Même si tu passes deux fois la même valeur.  Tu ne peux pas utiliser :difCR deux fois dans une requête.

3- Si je peux me permettre je t'INTERDIS d'essayer des ' ou des ". Partout sur ce forum on voit des gens conseiller d'utiliser l'un plutôt que l'autre : c'est totalement stupide. Il faut apprendre une fois pour toutes la signification de l'un et de l'autre (c'est simple). En suite, on n'"essaye" pas, on écrit. L'un ou l'autre comme on veut, peu importe car les deux sont toujours possibles, seulement plus ou moins 'confortables' selon la situation. À condition de respecter les règles d'écriture qui, je le répète, sont simples.

Regarde "Entourée de guillemets simples" et "Entourée de guillemets doubles"
https://www.php.net/manual/fr/language.types.string.php

Je suis absent jusqu'à 22h

rjl · Answer

Bonsoir, 

Je n'ai pas du trop perdre mon temps en traînant du côté de Définir une classe ¦ Créer un objet ¦ Héritage ¦ Opérateur :: et parent ¦ Durée de vie ¦ Les fonctions magiques, etc... 
Pour autant, si j'ai corrigé les points 1 & 3 évoqués, je suis resté perplexe sur le point 2 : Les paramètres passés à une requête PDO doivent avoir des noms différents. "Même si tu passes deux fois la même valeur. Tu ne peux pas utiliser :difCR deux fois dans une requête". J'ai repris le code pour imaginer ce que ça donnerait et comment MySQL peut s'y retrouver : $req = $connexion->prepare('UPDATE adh SET difCR = :difCR, difProg=:difProg  WHERE  (admail <= $admail AND (difCR = '1' OR    difProg ='1'))'); 
try { $req->execute(array('difCR'  => '0', 'difProg' => '0' ));Tu m'as déjà expliqué que je pouvais mettre :toto (ou n'importe quoi) à la place de :difCR... OK mais si je mets par ailleurs truc à la place de difCR, on est d'accord que j'obtiens : SET truc :toto ! Comment fait-il alors le lien avec difCR : je suis largué. 
J'utilise difCR et difProg en sélection mais aussi en modification ; comment m'en tiere avec des noms modifiés et quel sera le lien ? Le SET ? 
Merci d'une clarification bien nécessaire. 

Autant pour moi, j'avais mal lu : c'est :difCR que je ne peux pas utiliser 2 fois... mais sauf à avoir mal relu mon code (mais j'ai fait une recherche... il n'est présent qu'une fois dans le script), et je sais pourvoir le remplacer ! 
Ca n'arrange pas mon cas car je ne vois pas où est le problème. 
@bientôt      RJL2901

Utilisateur anonyme · Answer

Pour le point 2, c'était une erreur de ma part. Trop pressé avant de partir ce matin, j'avais mal lu et j'avais eu l'impression que tu utilisais deux fois le même nom de paramètre, ce qui n'était pas le cas. Donc ma remarque ne te concernait pas en fait. Par contre, elle te concerne maintenant dans ton message n° 7, tu utilises deux fois :caramail. Dans ta requête $req = $connexion->prepare('UPDATE adh SET difCR = :difCR, difProg=:difProg WHERE ((admail <= $admail) AND ((difCR = 1) OR (difProg = 1)))'); tu cumules plusieurs problèmes - erreur php : mauvaise utilisation de $admail : à l'intérieur d'une chaîne entre simples quotes, $admail n'est rien d'autre que le texte de 7 caractères $-a-d-m-a-i-l et absolument pas une variable php. $admail est une variable en dehors des chaines ou dans les chaînes entre doubles quotes. - erreur mysql si $mysql est un texte. il faut qu'il soit entre quotes sinon sql considère que c'est un nom de champ. Mais comme tu travailles en pdo, tu as intérêt à remplacer toutes les valeurs variables de la requête par des paramètres : $req = $connexion->prepare('UPDATE adh SET difCR = :difCR, difProg=:difProg WHERE ((admail <= :admail) AND ((difCR = 1) OR (difProg = 1)))'); try { $req->execute(array('difCR' => 0, 'difProg' => 0, 'admail'=>$admail )); D'autre part, il est inutile de me parler de tes chiffres qui passent ou rouge ou en noir ou en vert : ce sont là des particularités de ton environnement de travail, qui n'ont rien d'universel et n'évoquent rien pour moi. Tu vas encore me reprocher de faire des essais Faire des essais n'est pas une mauvaise chose en soi... Ce que j'ai voulu dire, c'est que dans la plupart des cas, il suffit de quelques connaissances de base (et en PHP, je pense toujours à cette histoire de simples ou doubles quotes des chaînes) pour écrire quelque chose de correct du premier coup et ne plus jamais avoir besoin d'essayer.

rjl · Answer

Bonjour le père, 

Rassures-toi, ce n'est pas parce que j'oublie de réaborder un sujet que je ne le prends pas en compte ! Concernant les ' et ", j'ai lu, puis édité la doc vers laquelle tu m'avais envoyé, avant de corriger. 
Un changement total car ça exécutait. 
Dans un premier temps, je suis tombé sur une Fatal error car je n'adressais pas le bon objet pour le rowCount ! 
Après correction, seconde inquiétude car tout s'exécutait avec un comptage à zéro ! 
J'ai alors corrigé la logique car ce n'était pas des 1 que j'avais mis initialement mais des * ! j'ai donc corrigé ce détail (avec les " nécessaires : il me faisait une erreur avec les simples quotes... car je ne l'avais pas échappé !). 
Une fois la correction effectuée, les 0 se sont bien mis en place mais, compte tenu de mon erreur avec l'astérisque, j'ai fait le passage complémentaire pour mettre en place les 1 ce qui fait que je n'ai plus aucune * et simplement des 1 et 0. 
Voila ce sujet clos et un excellent exemple pour la suite ! 
Encore un grand merci car j'aurai pu boucler longuement avec mes essais... 
Et dire que dans le boulot il m'arrivait d'interdire aux programmeurs de lancer tel ou tel essai car lorsqu'ils ne pouvaient pas me dire ce qu'ils en tireraient suivant que les résultats seraient x ou y... 
Pour autant avec des sujets qui interfèrent PHP, MySQL, PDO,... il n'est pas toujours évident de tomber sur la bonne doc traitant du cas que l'on pense toujours particulier. 
Bon dimanche        RJL2902

UPDATE Nb Param invalide en requête

6 réponses

Discussions similaires