[SOLUCIONADO] Cual es mi error?

[G33N]

Tengo un error y no se cual es!! aver si alguien lo puede ver..

mensaje de error:

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 'WHERE id=''' at line 1

Codigo:

<?php
include ("dconexion.php");
include ("formmodificaraa.php");

$con=mysql_connect($host,$user,$pw)or die("Problemas al conectar con Mysql");

mysql_select_db($db,$con)or die("Problemas con DB");

mysql_query("UPDATE auto SET cliente_cliente='$_POST[cliente_cliente]', patente='$_POST[patente]', modelo='$_POST[modelo]', kilometro='$_POST[kilometro]', doperacion='$_POST[doperacion]', strabajo='$_POST[strabajo]', efecha='$_POST[efecha]', sfecha='$_POST[sfecha]', WHERE id='$s'",$con)or die (mysql_error());

echo "Modificacion correcta";
?>
----------------------

[pateketrueke]

¿Es mi imaginación o hay una coma antes del WHERE?

Por favor, revisa una referencia de SQL, tu error es de sintaxis de SQL, nada que ver con PHP.

[Alexis88]

Quita la coma que está antes de WHERE, está de más.

Saludos

Edito: Se me adelantaron por unos segundos.

[alejo_ferguson]

Quiero añadir a lo que pusieron los amigos arribar que me parece que tiene mucha razon, que al momento de hacer uso del la variable que recibes en el post debe ser escrita asi:
....set columna = ' " . $_POST['variable_post']. " ' , columna2 = '$variable' ";
ten cuiado porque veo que en tu query al momento de hacer uso de la variable post solo la escribes asi: ......$_POST[variable_post]' , columan2='$variale'.....
Eso estoy CASI seguro que dara un error porque no tiene comillas simples al nombre de la variable que esperas en el POST.


Suerte me avisas!

[pateketrueke]

Iniciado por alejo_ferguson

[...] Eso estoy CASI seguro que dara un error porque no tiene comillas simples al nombre de la variable que esperas en el POST.

De hecho esa forma de colocar variables es válida, lee lo siguiente:

http://www.forosdelweb.com/f18/como-...s-bien-588701/

[Artificium]

Te recomiendo que te facilites a ti mismo la lectura e interpretación de tus códigos, ahorrarte unas líneas no justifica que escribas expresiones difíciles de entender como la que has mostrador y peor aún si eres novato. En este caso es preferible coger los valores del post en variables y luego contruir tu consulta. Además no es pecado usar las comillas dobles y las variables dentro porque el rendimiento no disminuye mucho, es justificable por la legibilidad que se obtiene.

[G33N]

Iniciado por Artificium

Te recomiendo que te facilites a ti mismo la lectura e interpretación de tus códigos, ahorrarte unas líneas no justifica que escribas expresiones difíciles de entender como la que has mostrador y peor aún si eres novato. En este caso es preferible coger los valores del post en variables y luego contruir tu consulta. Además no es pecado usar las comillas dobles y las variables dentro porque el rendimiento no disminuye mucho, es justificable por la legibilidad que se obtiene.

Antes que nada muchas gracias a todos por ayudar :D .. y especificamente, con lo que vos me decis, trate de hacerlo pero el problema que me surgio era que para sacar los datos de mi <form> y inviarlo a un archivo para pasar los $_POST a variables, se me complico ya que tengo un onChange en el form y cre oque deberia poder hacer que el <form> exporte los datos a 2 archivos creo que es mas complicado en ese caso :/,