Erreur de syntaxe

[mloolm]

salut
j ai cette erreur
Erreur de syntaxe près de 'WHERE id = '1'' à la ligne 1

mon code est

Code PHP :

<?php 

$pa= 233;

$id = 1;

mysql_select_db($database_wardog, $wardog);

$query_panouveau = "INSERT INTO pdf_membre(pa) VALUES('".$pa."' ) WHERE id = '$id' ";

$panouveau = mysql_query($query_panouveau, $wardog) or die(mysql_error()); 

[Plume]

Déjà, je pense que `id` est un entier donc à ne pas mettre entre apostrophes.

Voilà pour toi :

Code PHP :

<?php 

$query_panouveau = 'INSERT INTO pdf_membre(pa) VALUES(\''.$pa.'\' ) WHERE id = '.$id; 

[icien]

il n'y a pas de WHERE dans les requêtes INSERT.

sauf dans ce cas :

INSERT INTO Nom_Table
(champ, ..., champN)
SELECT nom_champ, ..., Nom_champN
FROM nom_table
WHERE Condition

[Plume]

C'est pas faux non plus ^^ La prochaine fois je regarderais plus loin qu'autour de l'erreur signalée :oops:

[grouik5962]

Salut mloolm,

L'insert ne sert souvent qu'une seule fois (à l'inscription du joueur pour initialiser les tables), et il n'y a pas de where. En revanche, si tu veux remettre à 1 le nombre de PA du joueur alors c'est plutot un UPDATE qu'il te faudra.

Dans le cas de l'insert, précise tous les champs qui vont être renseignés, dans ton cas, à priori :
"Insert into pdf_membre(id, pa) VALUES(".$pa.", ".$id.")";
// Evite les $var entre les "" pour des questions de lisibilité vaut mieux les sortir des ""

Pour ton update se sera plutôt
"Update pdf_membre set pa = ".$pa." where id = ".$id;

Aux erreurs de saisie prêt :-)

A+ et bon courage

[Plume]

Comme tu dis - aux erreurs de saisies près.
mloolm, on te fait confiance tu ne copies pas la requète comme telle. Elle contient une erreur, mais pas de syntaxe - la requète est correcte.
Ensuite, j'aimerais savoir à quoi ça vous sert d'utiliser les guillemets pour encadrer vos chaînes de caractères si vous les concaténer quand vous rencontrez une variable. C'est la forme la plus lourde que vous puissiez trouver

'fin bref ^^

[grouik5962]

PErso je fais comme çà car la plupart du temps il y a déjà des ' dans la requête pour les champs texte. Donc je garde les " pour la délimitation mais effectivement d'après ce que j'ai pu lire à droite à gauche dans certains cas ce n'est pas l'écriture la plus rapide en terme d'interprétation.

[Plume]

Et bien certes, même si pour l'optimisation {infime, certes, mais tout de même}, je conseille l'utilisation des apostrophes - qui soit dit en passant, peuvent être échappé grâce au caractère `\` dans des chaînes de caractères - il est évidemment possible d'utiliser les guillements. Dans ce cas, la concaténation n'est pas nécessaire, les variables sont repérées & interprétées comme il se doit, donc à bas la concaténation ^^

[Sephi-Chan]

Cherchez pas, sprintf() forever !

Avant j'utilisais beaucoup la concaténation.

Code PHP :

<?php 

$var = 'Ce cheval est ',$couleur,' et rose.'; 

[Plume]

Je maintiens ce que je dis. Je préfère de loin l'affichage de la variable avec la coloration syntaxique ... ^^