PHP + MySQL Help!

[valtimore]

Hola gente del foro; disculpen si nunca me había presentado, yo soy nuevo aquí. Estoy haciendo un curso de PHP mezclado con MySQL básico, y a decir verdad estoy demasiado perdido... Quisiera saber si ustedes me podrían dar una mano.


este vendría a ser el código en cuestión:


$titulo=$_POST['titulo'];
$autores=$_POST['autores'];
$generos=$_POST['generos'];
$editoriales=$_POST['editoriales'];
$id_libro=$_POST['id_libro']
$sql=update libros set titulo='$titulo', id_autor='$autores', id_editorial='$editoriales', id_genero='$generos' where id_libro='$id_libro';

Me salta este error:

Parse error: syntax error, unexpected T_VARIABLE in C:\xampp\htdocs\curso\editar.php on line 21

Quería saber qué opinan ustedes, si es un error de ortografía, o si le estoy errando en algo yo... Desde ya muchas gracias.

Acá les dejo el archivo completo, junto con la base de datos http://www.mediafire.com/?m2qm5nlq41rz38b

Desde ya muchísimas gracias

[Patriarka]

esto esta mal:

$sql=update libros set titulo='$titulo', id_autor='$autores', id_editorial='$editoriales', id_genero='$generos' where id_libro='$id_libro';

cambialo por

$sql="update libros set titulo='$titulo', id_autor='$autores', id_editorial='$editoriales', id_genero='$generos' where id_libro='$id_libro' ";

[valtimore]

Muchísimas gracias, ya lo arreglé; pero sigue tirándome el mismo error en la misma línea... Debo estar fallando en otro lado pero no sé en dónde... Fijate si podés encontrar el error, y gracias de nuevo por lo de las comillas

[JaimeSavines]

Iniciado por valtimore

Fijate si podés encontrar el error, y gracias de nuevo por lo de las comillas

Espero que sea broma, ya te dijeron que el error es de sintaxis asi que revisa con calma

[valtimore]

Iniciado por JaimeSavines

Espero que sea broma, ya te dijeron que el error es de sintaxis asi que revisa con calma

Ya corregí lo que me dijo Patriarka, y sigo teniendo ese error... Ya le puse las comillas, es más hice Copy Paste de la corrección que me dijo Patriarka, así que ese no es solamente el error mi amigo, por eso les pido de buena manera si me pueden ayudar

[Patriarka]

aunque pensandolo bien...lo haria asi

$sql="update libros set titulo=".$titulo.", id_autor=".$autores.", id_editorial=".$editoriales.", id_genero=".$generos." where id_libro=".$id_libro;

[valtimore]

Gracias, ya pude corregir eso, me faltaban algunos ; que había abajo

[valtimore]

Patriarka, por qué concatenado?

[Patriarka]

por que $sql no es mas que una CADENA o STRING como quieras llamarlo.
que lo vas a ejecutar con mysql_query($sql);

[valtimore]

Ajam, y en qué varía que lo hagas sin concatenar a que lo hagas concatenado?

[Patriarka]

hay muchas formas de hacer la cadena sql, hace tu propia experiencia y fijate cual te gusta mas.

pero es una cadena al fin y listo

[valtimore]

Ahh buenísimo.. Gracias, te dejé Karma :)

[JaimeSavines]

Iniciado por valtimore

Ajam, y en qué varía que lo hagas sin concatenar a que lo hagas concatenado?

Hay varias formas de representar lo que estas haciendo, todo de pende de las necesidades, si lo concatenas puedes crear una forma de armar el query de manera mas dinámica. Pero si tu update es siempre como lo muestras en tu ejemplo lo puedes dejar como esta, cambiarlo a la forma de concatenación o también a esta otra:

Código PHP:

$sql="update libros set titulo = '$titulo', id_autor = $autores, id_editorial = $editoriales, id_genero = $generos where id_libro = $id_libro";
 

tomando en cuenta que id_autor, id_editorial, id_genero y id_libro son valores numéricos. Todo depende del gusto.

[GabrielPCS]

Te recomiendo ampliamente validar primero los datos recibidos por $_POST:

El tipo de dato y los rangos de valores validos y no esta de mas algo para prevenir la Inyección SQL

Yo uso vanadium.js y KSES