PHP OO UPDATE SET : you have an error in your SQL syntax; check the manual that cor.

[ydmejia87]

Hola a todos, me pueden ayudar con este error:

"you have an error in your SQL syntax; check the manual that corresponsd to your MYSQL server version for the right syntax to use ' set proceso estado=HECHA CON PODER at line 1"

Codigo

Código PHP:

 if( $_REQUEST["estproc"]=="" or $_REQUEST["estjuzg"]=="" or  $_REQUEST["observaciones"]=="" or $_REQUEST["valor"]=="" or $_REQUEST["porcentaje"]=="" ){ 
     
                    cuadro_error("Debe llenar todos los campos"); 
        }else{ 
              
            //donde se llevan los datos a la BD 
             
             
            $sql1 .="update set proceso "; 
            $sql1 .="estado=".$_REQUEST["estproc"]; 
            $sql1 .="estadoj=".$_REQUEST["estjuzg"]; 
            $sql1 .="observaciones=".$_REQUEST["observaciones"]; 
            $sql1 .="valor=".$_REQUEST["valor"]; 
            $sql1 .="porcentaje=".$_REQUEST["porcentaje"]; 
            $sql1 .="select from proceso where nproceso =" .$_REQUEST["numbproceso"]; 
             
                     
             
            if(mysql_query($sql1))    { 
                mensaje("Caso Actualizado Correctamente..."); 
                echo "<br><br><br><br><br>"; 
                ?> 
                    <script type="text/javascript"> 
                    window.location = "../mod_casos/reg_prc.php"; 
                    </script> 
                <?php 
            }else{ 
                cuadro_error(mysql_error());//emite un mensaje de error de la BD sino se realizo la operacio 
            }   

[pateketrueke]

Tu problema es bastante sencillo: lee una referencia de SQL primero y aprende su sintaxis antes de usarlo.

[ydmejia87]

pateketrueke

Que pena, no entendi lo que me quisiste decir

[gnzsoloyo]

A que para poder crear consultas en SQL dentro de un script PHP tienes que tener claras algunas cosas:
1) PHP no es SQL, y viceversa.
2) Cuando creas código SQL dentro de un script PHP, el código SQL resultante debe respetar la sintaxis y lógica del SQL.
3) Que no contenga errores sintácticos en PHP, no quiere decir que no estés metiendo la pata en SQL. Los analizadores de PHP no analizan el SQL.
4) Cada DBMS (motor de base de datos), tiene diferentes sintaxis en muchas cosas. La que usas para SQL Server no necesariamente sirve en Postgre, MySQL u Oracle.
5) Diseñar y administrar BBDD no es lo mismo que programar PHP. Son áreas diferentes.

Dicho esto, uno de los problemas es que no estás respetando tres condiciones que la sintaxis SQL requiere:
1) Las cadenas de texto van entre apóstrofos ('), por tanto cualquier valor en una variable que sea cadena de texto, debe quedar encerrada así en la query una vez creada.
2) Los números van sin apostrofos, y los decimales usan punto decimal, y no coma.
3) Las fechas se colocan entre apóstrofos, respetndo el formato de cada DBMS.

En tu caso tienes al menos cinco errores:
1) No estás indicando qué tabla estás actualizando, o no lo estás haciendo correctamente.
2) Estás poniendo las cadenas de texto sin apostrofos, por lo que el DBMS las tomará como nombres de campos, y al no encontrarlos se generarán errores.
3) Estás colocando un UPDATE pegado a un SELECT, siendo que un SELECT no es parte del UPDATE, y síolo peude ser usado con el si es una subconsulta para una validación en el WHERE, que no es el caso.
4) Como no dejas espacios entre el final de un campo y el comienzo del otro, el valor de la variable se está pegando al siguiente campo, generando errores de sintaxis.
5) No estás poniendo el separador de campos modificados (la coma), y eso generará errores de sintaxis.

Código PHP:

$sql1 .="UPDATE proceso SET ";
$sql1 .=" estado= '".$_REQUEST["estproc"]."', ";
$sql1 .=" estadoj= '".$_REQUEST["estjuzg"]."', ";
$sql1 .=" observaciones= '".$_REQUEST["observaciones"]."', ";
$sql1 .=" valor= '".$_REQUEST["valor"]."', ";
$sql1 .=" porcentaje= '".$_REQUEST["porcentaje"];
$sql1 .=" WHERE nproceso =" .$_REQUEST["numbproceso"];
 

Por otro lado, como no se ve que estés controlando el tipo de caracteres que tienen las variables de cadena, puedes estar metiendo caracteres que rompan la sintaxis. Eso ya es un defecto de programación, no de SQL, pero tiene sus consecuencias en la consulta.

[ydmejia87]

OK, gracias, voy a validar. Y les cuento.