Error sentencia UPDATE mysql

astut · #1 (**permalink**) 04/08/2007, 12:54

Hola, sabrian decirme que hay de malo en este codigo:

Código PHP:

  $editar= mysql_query("UPDATE usuario SET contrasena= {$_POST['contrasena']}, correo= {$_POST['correo']} 
                              , ubicacion= {$_POST['ubicacion']}, cfg_estilo= {$_POST['estilo']} 
                                                    WHERE id= {$_COOKIE['juego_jugador_id']}");

Me devuelve el siguiente error:

Código:

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 '@hotmail.com , ubicacion= Valencia, España, cfg_estilo=' at line 1

Gracias y saludos!

gerson · #2 (**permalink**) 04/08/2007, 13:15

Hola astut pues te falta comillas en los valores:

Código PHP:

  $editar= mysql_query("UPDATE usuario SET contrasena='".$_POST['contrasena']."', correo='".$_POST['correo']."', ubicacion='".$_POST['ubicacion']."', cfg_estilo='".$_POST['estilo']."' WHERE id=".$_COOKIE['juego_jugador_id']);

Saludos

astut · #3 (**permalink**) 04/08/2007, 13:57

La forma en que puse las variables no tiene que ver, ya que comprobe de varias formas, saludos!

GatorV · #4 (**permalink**) 04/08/2007, 14:15

Se refiere a que te faltan las comillas en actualizar tus valores:
Esto te da el error:

Código:

UPDATE tabla SET valor1=un valor

Esto no te da error:

Código:

UPDATE tabla SET valor1='un valor'

Saludos.

astut · #5 (**permalink**) 04/08/2007, 14:35

Se hace de esta forma verdad?:

Código PHP:

  contrasena= '{$_POST[contrasena]}'

Los corchetes van dentro, ¿de esta forma ahorro consumo?

saludos

vb2005 · #6 (**permalink**) 04/08/2007, 15:02

no, se hace como teb dijeron arriba los {} no sirven de nada

astut · #7 (**permalink**) 04/08/2007, 15:25

Estas seguro de lo que dices?, en un post me respondieron que use los {}... alguien sabe mas? saludos

gildus · #8 (**permalink**) 04/08/2007, 15:55

Holas,

Estan bien como te mencionan astut, ademas es sin el {}.

;)

Saludos
Gildus

astut · #9 (**permalink**) 04/08/2007, 18:40

de veras?

Yo lo hago de la forma:

Código PHP:

  nombre= '$_POST[estilo]'

¿Acaso es malo hacerlo asi?, saludos!
Y porque los {} no sirven para nada, ¿para que sirven?

gerson · #10 (**permalink**) 05/08/2007, 13:18

Pues es valida esa forma que utilizas pero vale aclarar que tienes q usarlos dentro de las comillas dobles:

Código PHP:

  $editar= mysql_query("UPDATE usuario SET contrasena='$_POST[contrasena]', correo='$_POST[correo]', ubicacion='$_POST[ubicacion]', cfg_estilo='$_POST[estilo]' WHERE id=$_COOKIE[juego_jugador_id]");

y con respecto a {} pues lo que se, es que se utilizan en variables variables.

Saludos

gildus · #11 (**permalink**) 06/08/2007, 18:56

Holas,

si usas de la forma:

Cita:

$editar= mysql_query("UPDATE usuario SET contrasena='$_POST[contrasena]', correo='$_POST[correo]', ubicacion='$_POST[ubicacion]', cfg_estilo='$_POST[estilo]' WHERE id=$_COOKIE[juego_jugador_id]");

Esta mal porque el PHP no lo interpreta el valor de la variable $_POST[contrasena], por ejemplo, tienes que separar lo que es variable o lo que es valor de una variable. Ademas si usas $_POT[contrasena] te dira el PHP que no se reconoce la variable contrasena, tiene q ser: $_POT['contrasena'] .

Trata de usar comillas simples es mas rapido que las comillas dobles, haces buscar al PHP dentro de las comillas dobles si hay ono una variable, y eso hay que ayudarle al php.

Ahora los {} (corchetes) claro que sirven pero no para lo que quieres hacer.

Saludos
Gildus

bejarano · #12 (**permalink**) 07/08/2007, 03:50

Yo no me liaría y sacaría cada $_POST a una variable. Luego metería las variables con las comillas como te dicen arriba: Algo así...

Código PHP:

  $contrasena=$_POST['contrasena'];
$correo=$_POST['correo'];
.
.
.

$editar= mysql_query("UPDATE usuario SET contrasena='$contrasena', correo='$correo' ............ y así sucesivamente

evitas problemas, además me parece una forma de programar más elegante y clara.

Espero que te sirva de ayuda