Sujet Précédent Sujet Suivant

Erreur mysql update

Résolu/Fermé
bartimus37 Messages postés 193 Date d'inscription samedi 19 novembre 2011 Statut Membre Dernière intervention 8 octobre 2014 - Modifié par bartimus37 le 23/04/2013 à 11:49
bartimus37 Messages postés 193 Date d'inscription samedi 19 novembre 2011 Statut Membre Dernière intervention 8 octobre 2014 - 25 avril 2013 à 10:47
Bonjours, j'ai un problème avec mysql update :
Erreur SQL !mysql_query("UPDATE news SET reponse=réponse de test WHERE id=\'.$nid.\'");
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 'mysql_query("UPDATE news SET reponse=réponse de test WHERE id=\'.$nid.\'")' at line 1
Voici le code : 
$nid= $_GET['id']; 
$rep = $_POST['reponse'];                      
    $sql = 'mysql_query("UPDATE news SET reponse='.$rep.' WHERE id='.$nid.'");';   
    mysql_query($sql) or die('Erreur SQL !'.$sql.'<br />'.mysql_error()); 
    mysql_close();


page de code complète : https://pastebin.com/DMwgZPDy
Le fichier config.php en include_once contient la connexion a la base de donné .
Merci de vos réponse.
A voir également:

6 réponses

Réponse 1 / 6
Guiver Messages postés 262 Date d'inscription mardi 30 novembre 2010 Statut Membre Dernière intervention 20 mai 2021 87
23 avril 2013 à 13:43
Bizarre ton code, essaye ça : 

 
$sql = "UPDATE news SET reponse='".$rep."' WHERE id=".$nid."";

ou si ton champs ID n'est pas de type nombre : 
 
$sql = "UPDATE news SET reponse='".$rep."' WHERE id='".$nid."'";

Comme le dis "gardiendelanuit", ce n'est pas bon de concaténer les variables directement dans les requêtes.
De plus, le mysql est obsolète depuis quelques années. privilégie mysqli ou PDO dorénavant ;)
2
Réponse 2 / 6
gardiendelanuit Messages postés 1769 Date d'inscription jeudi 20 décembre 2007 Statut Membre Dernière intervention 19 novembre 2016 264
Modifié par gardiendelanuit le 23/04/2013 à 12:13
Bonjour,

Il faut rajouter des magiques quotes pour les values de la requête surtout pour les chaînes de caractères: 
$sql = mysql_query("UPDATE news SET reponse=\''.$rep.\'' WHERE id=\''.$nid.'\'");';


EDIT: mysql query est une fonction et tu la met dans ta string??
Merci de mettre "Résolu" quand le problème est réglé!
La connaissance c'est comme la confiture: moins on en a et plus on l'étale.
0
bartimus37 Messages postés 193 Date d'inscription samedi 19 novembre 2011 Statut Membre Dernière intervention 8 octobre 2014 14
23 avril 2013 à 12:25
ne fonctionne pas : Parse error: syntax error, unexpected T_STRING in /beta/reponse.php on line 30
0
gardiendelanuit Messages postés 1769 Date d'inscription jeudi 20 décembre 2007 Statut Membre Dernière intervention 19 novembre 2016 264
Modifié par gardiendelanuit le 23/04/2013 à 12:51
Renseigne toi sur les " et '
Pour ton cas 
$sql = mysql_query("UPDATE news SET reponse='$rep' WHERE id='$nid';");

Ton code est sujet aux injections SQL aussi.
0
bartimus37 Messages postés 193 Date d'inscription samedi 19 novembre 2011 Statut Membre Dernière intervention 8 octobre 2014 14
23 avril 2013 à 13:31
Toujours une erreur ...
Erreur SQL !1
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 '1' at line 1
0
bartimus37 Messages postés 193 Date d'inscription samedi 19 novembre 2011 Statut Membre Dernière intervention 8 octobre 2014 14
23 avril 2013 à 13:53
je crois que la variable $nid pose problème car elle est appelé en $_GET
$nid = $_GET['id'];
on arrive sur la page reponse.php?id=2
ensuite on remplit le formulaire et on reviens sur reponse.php qui dit que
$nid = $_GET['id'];
et $nid est vide
Je ne vois pas comment résoudre ça ...?
0
gardiendelanuit Messages postés 1769 Date d'inscription jeudi 20 décembre 2007 Statut Membre Dernière intervention 19 novembre 2016 264
23 avril 2013 à 14:28
Si tu utilises un form, c'est pas mieux de mettre un input type="hidden" ?
0
Réponse 3 / 6
Guiver Messages postés 262 Date d'inscription mardi 30 novembre 2010 Statut Membre Dernière intervention 20 mai 2021 87
24 avril 2013 à 09:12
Salut,

C'est une bonne idée de mette le code sur pastebin.
Mais tu peux l'indenter correctement stp la prochaine fois ?
Je viens de passer 5 minutes à le faire pour le lire correctement.

Donc 
<INPUT TYPE=HIDDEN NAME=id VALUE=<?$_GET['id']?>>

Inutile de le mettre dans un <TD>, ça fait un espace vide dedans, mets le juste en dessous de ton <form>. Et écris le plutôt comme ça : 
<INPUT TYPE="HIDDEN" NAME="id" VALUE="<?php $_GET['id'] ?>" />

Ensuite, tu as une balise <td> en trop juste avant <td align="right">
Pour finir, sur ton dernier if, tu ferme l'accolade après $erreur; mais tu ne l'ouvre pas (accolade que tu ne mets pas dans ton premier if d'ailleurs).
0
Réponse 4 / 6
bartimus37 Messages postés 193 Date d'inscription samedi 19 novembre 2011 Statut Membre Dernière intervention 8 octobre 2014 14
24 avril 2013 à 17:10
j'ai fait tout ce que tu m'a dit : https://pastebin.com/HQxYgF1C
mais ça ne fonctionne pas : Parse error: syntax error, unexpected $end
in reponse.php on line 38 .
C'est le dernier if ...
0

Vous n’avez pas trouvé la réponse que vous recherchez ?

Posez votre question
Réponse 5 / 6
Guiver Messages postés 262 Date d'inscription mardi 30 novembre 2010 Statut Membre Dernière intervention 20 mai 2021 87
24 avril 2013 à 17:44
L'indentation n'est pas ton fort on dirait ^_^
Mais c'est déjà beaucoup mieux, merci pour l'effort.

J'ai pas le temps de regarder en détail se soir, mais je vois déjà que ton 
if ( $pseudo =='admin')
{

n'est pas fermé.
Ensuite, tu as oublié un double quote dans ton form :
<form action="reponse.php method="post">
au lieu de
<form action="reponse.php" method="post">
0
Réponse 6 / 6
bartimus37 Messages postés 193 Date d'inscription samedi 19 novembre 2011 Statut Membre Dernière intervention 8 octobre 2014 14
24 avril 2013 à 18:09
Il n'y a plus aucune erreur mais l'id ne s'affiche pas ...
https://pastebin.com/qyjUzZ8e
0
gardiendelanuit Messages postés 1769 Date d'inscription jeudi 20 décembre 2007 Statut Membre Dernière intervention 19 novembre 2016 264
Modifié par gardiendelanuit le 24/04/2013 à 21:50 
<?php echo $_GET['id']; ?>

Sûrement non? :D
0
bartimus37 Messages postés 193 Date d'inscription samedi 19 novembre 2011 Statut Membre Dernière intervention 8 octobre 2014 14
25 avril 2013 à 10:47
Merci !!!!!!!!
0

Discussions similaires

Instagram "Désolé, une erreur s'est produite"
bananamilk -
cedric.masdeb -

60 réponses
Erreur vidéo inconnue sur le Freebox Player
mouhaaV2.0 -
LH -

59 réponses
Free erreur 38 : impossible d'envoyer des SMS
warnawak -
marcellou -

58 réponses
hp bios update bloqué
almg -
Tkofficial -

9 réponses
Erreur de lecture sur iptv smarters
Bogs -
bazfile -

1 réponse