Problemas para actualizar la base de datos

[carroviejo]

Buenas a todos.

Estoy intentando actualizar la base de datos desde un formulario, pero no consigo que esto suceda. Ya he comprobado que los valores de $_POST se pasan correctamente con "echo". Por mucho que lo reviso no encuentro el fallo.

Este es el código:

Código PHP:

$name = $_POST["user"];
$email = $_POST["email"];
$password = $_POST["password"];
$age = $_POST["age"];
$gender = $_POST["gender"];
 
 
$mysql_id = mysql_connect('localhost', 'root', '');
mysql_select_db('database', $mysql_id);
mysql_query("INSERT INTO user (name, email, gender, age, password)
VALUES (".$name.",".$email.", ".$gender.", ".$age.", ".$password.")");
 

Gracias a todos por adelantado. Saludos.

[gjx2]

Hola carroviejo,
Tienes que darle el trato que se merecen los campos de tipo String, encierra la asignación en comillas.

Ej.
VALUES ('".$name."');

Saludos.

[oscartt67]

como te dice gjx2, es un problema de comillas.
Una sintaxis bastente cómoda es

Código PHP:

  $sql="INSERT INTO ...... VALUES ('$name','$email','$gender','$age','$password')";  
   

siempre y cuando, todo el texto esté entre comillas dobles, tal y como lo tienes.

lo de abajo, seria incorrecto.

Código PHP:

  //MAL!!!!
$sql='INSERT INTO ...... VALUES ('$name','$email','$gender','$age','$password')';
$sql='INSERT INTO ...... VALUES ("$name","$email","$gender","$age","$password")';  
   

SALUDOS.

[gjx2]

Iniciado por oscartt67

Una sintaxis bastente cómoda es

Código PHP:

  $sql="INSERT INTO ...... VALUES ('$name','$email','$gender','$age','$password')";  
   

Hola oscartt67, Lo cómodo no es lo mejor, lo mejor es la forma correcta aunque conseguirla nos lleve mas tiempo.

Hacerlo de esa forma '$name' funciona pero forza al interprete por que primero tiene que reconocer que el valor que esta encerrado es un string o una variable.
Lo correcto es delimitar el contenido así conseguimos un código mas limpio y fácil en la depuración.

Saludos.

[oscartt67]

gjx2.
De acuerdo contigo:lo cómodo no es lo mejor. Pero lo más rápido tampoco tiene porque serlo. Pienso que lo mejor tiene que contemplar la velocidad, la legibilidad y otras cosas.

Cierto que le damos mas trabajo al intérprete, pero, a mi modo de ver, y esto es muy personal, el código me resulta mas legible y limpio. Cuando digo limpio, digo eso, con menos comas, comillas y puntos.

Seria bueno hacer una prueba de en que modo afecta al rendimiento dicha sobrecarga. Creo que en una petición donde aparezca dicha sintaxis 10 o 20 veces, por ejemplo, la diferencia va a ser insignificante.

Pero como la suposición es la madre de todos los errores, cuando tenga tiempo, voy a realizar un test y comento los resultados. Y si alguien ya hizo la prueba y quiere postear resultados, magnífico, y salimos de dudas.

Saludos!

[carroviejo]

Os agradezco a los dos las soluciones que me habéis propuesto, ya lo he solucionado y funciona perfectamente. Muchas gracias¡¡¡