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:

Erreur mysql update
Erreur 0x80070643 - Accueil - Windows
Windows update bloqué - Guide
Erreur 0x80070643 Windows 10 : comment résoudre le problème de la mise à jour KB5001716 - Accueil - Windows
Asus live update - Télécharger - Utilitaires
Mysql community server - Télécharger - Bases de données

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 ;)

Réponse 2 / 6

gardiendelanuit Messages postés 1770 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.

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

gardiendelanuit Messages postés 1770 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.

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

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 ...?

gardiendelanuit Messages postés 1770 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" ?

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).

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 ...

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">

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

gardiendelanuit Messages postés 1770 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

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 !!!!!!!!

Discussions similaires

hp bios update bloqué

Code erreur 3000

Erreur vidéo inconnue sur le Freebox Player

Free erreur 38 : impossible d'envoyer des SMS