[SOLUCIONADO] You have an error in your SQL syntax; check the manual that corresponds to your MySQL

[Rag_Doll]

Hola a Todos!!!

tengo un error al intentar actualizar los datos de una tabla clientes

esta es la consulta:

Editado: Código de programación no permitido en foros de Bases de Datos.
Leer las reglas del foro, por favor.

este es el error que muestra:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL

leyendo otros foros me fije en poner un echo con la consulta para ver algun error y me imprime esto:

Código SQL:

UPDATE clientes SET tipo = 'Cedula', numero = '100000', nombre = 'fulanito', 
apellido = 'pino', telefono = '3521587', celular = '124587944', correo= '[email protected]', 
nacimiento = '1991-11-04', ocupacion= 'estudiante', convenio ='Ninguno' 
WHERE idcliente = 57)
 

disculpen de antemano si hay algùn error (de esos que todo novato comete), muchas gracias por la ayuda que me puedan brindar.

[Libras]

todos tus datos son de tipo entero?? ya probaste esa consulta desde la consola de MYsql?

[gnzsoloyo]

Lo único que a mi no me cierra del todo es ese paréntesis que te ha quedado al final de la consulta. No estoy seguro si es un Copy+Paste mal hecho, o es que realmente está quedando allí en el SQL.
Si es lo segundo, quítalo y vuelve a probar.

[Rag_Doll]

Hola a Todos!!!

libras la mayoria son varchar, la fecha de nacimiento si es tipo date y una adicional que es automatica y solo se guarda una vez, aqui esta la estructura de la tabla:

Código SQL:

CREATE TABLE IF NOT EXISTS `gym`.`clientes` (
  `idcliente` INT NOT NULL AUTO_INCREMENT,
  `tipo` VARCHAR(50) NOT NULL,
  `numero` VARCHAR(50) NOT NULL,
  `nombre` VARCHAR(50) NOT NULL,
  `apellido` VARCHAR(50) NOT NULL,
  `telefono` VARCHAR(45) NULL,
  `celular` VARCHAR(45) NULL,
  `correo` VARCHAR(45) NULL,
  `nacimiento` DATE NOT NULL,
  `ocupacion` VARCHAR(45) NOT NULL,
  `convenio` VARCHAR(50) NULL,
  `inscripcion` TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP,
  PRIMARY KEY (`idcliente`))
ENGINE = InnoDB;
 

gnzsoloyo tienes razòn, es un mal copy+paste, la consulta es esta:

Código SQL:

UPDATE clientes SET tipo='$tipoid' , numero='$numero' , 
nombre ='$nombre' , apellido='$apellido' , telefono='$telefono' , celular='$celular' 
, correo='$correo' , nacimiento='$nacimiento' , ocupacion='$ocupacion' , 
convenio='$convenio' WHERE idcliente = ".$id
 

Editado: Codigo de programacion no permitido en foros de BBDD

està dentro de una funciòn

de antemano muchas gracias por sus aportes y ayuda!!!

[gnzsoloyo]

Mira, en términos generales no hay errores en la sintaxis de la query, al menos a nivel de SQL: Pero como la construyes dinámicamente, es altamente probable que tengas problemas con el contenido de las variables, y alli la cosa no es visible desde el lado de la BBDD.
Lo que veo con más posibilidades son dos cosas: !) En los valores de cadena hay algo que está entrando que rompe la cadena y genera el error. Eso suele suceder cuando en un campo dado entra un apóstrofo('), dado que ese tipo de caracteres deben ser escapados para meterlos en el SQL.
La otra posibilidad es quew el valor del ID esté llegando vacío, ya que sería el unico valor cuya ausencia genraría un error sintáctico.
Desde ya, es correcto que pongas un valor numerico sin apostrofos porque so permite detectar esrrores de programación que disparan errores de sintaxis.
tendrás que debuggear tu función y verificar si los valores de las variables están pasando correctamente. Pero eso es tema de PHP, no de MySQL.

Revisa, debuggea y veamos por donde falla la cosa.

[Rag_Doll]

Hola a Todos!!!

De antemano disculpen la tardanza, en fin; revise el codigo y siguiendo el consejo de gnzsoloyo pasa lo siguiente:

si dejo los apostrofes surge el mismo error, ahora si quito todos menos a los que imprimen nùmeros, me arroja esto:

Se produjo un error. Intente nuevamenteYou have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'alonso , apellido=ramirez perez , telefono='545' , celular='324564' ' at line 1

en la variable nombre debia salir 'jorge alonso' y el resto de las variables no las imprime. ahora si quito todos los apostrofes sale lo siguiente:

Se produjo un error. Intente nuevamenteYou have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'alonso , apellido=ramirez perez , telefono=3386404 , celular=3165885901 , corr'

ese 'corr' hace referencia a correo, que es como deberia seguir la cadena, no se si el error esta dentro de lo que imprime, o de lo que no (disculparàn la poca deducciòn que pueda tener), muchas gracias por sus aportes!!!

[gnzsoloyo]

si dejo los apostrofes surge el mismo error, ahora si quito todos menos a los que imprimen nùmeros, me arroja esto:

Por lo pronto, no estás entendiendo lo que te quise decir.
Supongamos que en tu campo apellido, entrase un nombre como Oscar apellido como O'Hara. Eso haría que la query quedase así:

Código MySQL:

..., apellido = 'O'Hara', nombre = 'Oscar'
 

¿Notas como se ha roto la contención de la cadena de texto, y parte del "Hara" y el siguiente campo aparecen como cadenas, pero no el "Oscar"?.
Bueno, eso pasa cuando en una variable entra un apostrofo, y el mismo no se procesa para ser escapado.

A eso me refería, porque es un problema posible al crearse las consultas en forma programática. Y no es el único caso que las puede hacer fallar.

Por otro lado, según devuelve el mensae, parte de la query dice así:

Código MySQL:

'alonso , apellido=ramirez perez , telefono='545' , celular='324564' '
 

Como se puede ver, tanto el "alonso", como "ramirez perez", no han quedado encerrados entre apostrofos como las cadenas de texto que son, y sí han quedado los valores numéricos, al revés de lo que dices.

Eso debería estar viéndose:

Código MySQL:

'alonso' , apellido='ramirez perez' , telefono=545 , celular=324564
 

Ene se contexto, es evidente, para mi, que estás cometiendo errores en la creacion de la sentencia a nivel prograático. Pero como el tema de programación es OFF TOPIC en los foros de BBDD, voy a mover tu posts al foro de PHP para que podamos seguir viéndolo desde ese lado.

Movido a PHP para continuar el tema desde programación en PHP.

Postea nuevamente el código PHP donde creas la query, y también la parte donde lo ejecutas.

[pateketrueke]

¿Qué librería usas para ejecutar la consulta?

El detalle es que debes "escapar" los valores apropiadamente, y no sólo agregar apostrofes a diestra y siniestra.

Por ejemplo, vamos a imaginar que usas PostgreSQL así que la función sería pg_escape_string():

Que básicamente agregará los apostrofes requeridos sin romper nada.

Tienes que buscar el equivalente para lo que usas.

[Rag_Doll]

Hola a Todos!!!

Ya voy entendiendo un poco, sin embargo, antes ya habia creado CRUD y desde que aprendi a actualizar por ejemplo todo ha funcionado hasta ahora claramente, pero enfin, acontinuaciòn el codigo referente al caso:

Tabla Mysql:

Código PHP:

CREATE TABLE IF NOT EXISTS `gym`.`clientes` (
  `idcliente` INT NOT NULL AUTO_INCREMENT,
  `tipo` VARCHAR(50) NOT NULL,
  `numero` VARCHAR(50) NOT NULL,
  `nombre` VARCHAR(50) NOT NULL,
  `apellido` VARCHAR(50) NOT NULL,
  `telefono` VARCHAR(45) NULL,
  `celular` VARCHAR(45) NULL,
  `correo` VARCHAR(45) NULL,
  `nacimiento` DATE NOT NULL,
  `ocupacion` VARCHAR(45) NOT NULL,
  `convenio` VARCHAR(50) NULL,
  `inscripcion` TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP,
  PRIMARY KEY (`idcliente`))
ENGINE = InnoDB;
 

Clase Clientes Actualizar:

Código PHP:

function actualizar
($tipoid,$numero,$nombre,$apellido,$telefono,$celular,$correo,$nacimiento,$ocupacion,$convenio,$inscripcion){
 
if($this->con->conectar()==true){
 
return mysql_query("UPDATE clientes SET tipo=$tipoid , numero=$numero , nombre=$nombre , apellido=$apellido , telefono=$telefono , celular=$celular , correo=$correo, nacimiento=$nacimiento , ocupacion=$ocupacion , convenio=$convenio , inscripcion=$inscripcion WHERE idcliente =".$id);
        }
    }
 

Este es el còdigo donde se utiliza la clase:

Código PHP:

<?php
$id=$_GET['id'];
$idinscripcion=$_GET['idinscripcion'];
$idnumero=$_GET['idnumero'];
 
  
if(isset($_POST['formulario'])){
  include('clases/clientes.class.php');
  $objCliente=new Cliente;
  
  $idcliente=$id;
  $tipoid= htmlspecialchars(trim($_POST['inptipo']));
  $numero = $_POST['inpnumero'];
  $nombre = htmlspecialchars(trim($_POST['inpnombre']));
  $apellido = htmlspecialchars(trim($_POST['inpapellido']));
  $telefono=$_POST['inptelefono'];
  $celular= $_POST['inpcelular'];
  $correo=htmlspecialchars(trim($_POST['inpcorreo']));
  $nacimiento=$_POST['inpnacimiento'];
  $ocupacion = htmlspecialchars(trim($_POST['inpocupacion']));
  $convenio = htmlspecialchars(trim($_POST['inpconvenio']));
  $inscripcion=$idinscripcion;
  
  
  
  
if ( $objCliente->actualizar($tipoid,$numero,$nombre,$apellido,$telefono,$celular,$correo,$nacimiento,$ocupacion,$convenio,$inscripcion) == true){
    //echo 'Datos guardados';
    //header('location:indexcoordinador.php');
    ?>
        <script>
        //un alert
        window.alert("Datos Actualizados Correctamente")
          
        location.href = 'infocliente.php?id=<?php echo $id ?>&idnumero=<?php echo $idnumero ?>'; 
        
        </script>
  <?php
  }else{
    
    echo 'Se produjo un error. Intente nuevamente' .mysql_error();
    ?>
        <script>
        //location.href="verconvenios.php"
        </script>
  <?php
  } 
}else{
  if(isset($_GET['id'])){
    
    include('clases/clientes.class.php');
    $objCliente2  = new Cliente;
    $consulta2 = $objCliente2->mostrar_cliente($id);
    $convenio = mysql_fetch_array($consulta2);
    
 
?>
 

De antemano muchas gracias por sus aportes y sugerencias!!!

[gnzsoloyo]

Pues aqui mismo tienes el error, a simple vista: Ninguna de las variables con datos que sean cadenas de texto está correctamente puesta entre apostrofos.

MySQL NO ENTIENDE PHP, ni tampoco ve las variables de PHP. Son entornos de existencia diferente.
MySQL recibe en realidad una cadena de texto que contiene una sentencia SQL, pero la sentencia SQL debe estar perfectamente formateada para MySQL, ya que la base jamás dialoga en PHP.

¿Queda claro.

Eso que haces debería tener este formato (te lo pongo en una variable para que se vea mejor la construcción de la query):

Código PHP:

$qry = "UPDATE clientes SET tipo='$tipoid', 
    numero=$numero , 
    nombre='$nombre', 
    apellido='$apellido', 
    telefono='$telefono', 
    celular='$celular', 
    correo='$correo', 
    nacimiento='$nacimiento', 
    ocupacion='$ocupacion', 
    convenio='$convenio', 
    inscripcion='$inscripcion'
WHERE idcliente = $id";
 
return mysql_query($qry);
 

Recuerda siempre: La base de datos NO entiende PHP, y NO RECIBE variables de PHP.

Nota: Esto que te paso no te asegura que ande completamente bien, porque si, por ejemplo, las fechas no están formateadas como "AAAA-MM-DD HH:MM:SS", entonces pude darte error de formato de fecha, o bien truncarte el dato o almacenarlo incorrectamente.

[Rag_Doll]

Hola a Todos!!!

Ya entiendo mejor la importancia del apòstrofo y la relaciòn php-bd, aplique el codigo de gnzsoloyo y me aparece lo siguiente:

Se produjo un error. Intente nuevamenteYou have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 12

la linea 12 del codigo seria esta:

WHERE idcliente = $id";

luego encerrè a $id en apòstrofes y dice que los datos se actualizaron correctamente, el problema esque no actualiza, tambien coloque a $id de otras formas, por ejemplo como lo tenia: WHERE idcliente =" .$id ; y tampoco.

De antemano muchas gracias por sus aportes y la atenciòn prestada.

[chronos682]

Yo tengo una duda, la variable $id si llega hasta el método actualizar de tu clase? No deberías pasarla como argumento de la función, puede ser que por el ámbito de las variables dentro de la clase, la variable $id sea una cadena vacía. Intenta antes de hacer el update imprimir el valor de $id con var_dump a ver si está llegando el valor del id.

[gnzsoloyo]

Como te dice @chronos682, el tema pasa por el valor de esa variable.
Si la variable entra vacía, dispara un error de sintaxis, lo cual es lo mejor que te puede pasar, ya que te muestra que estás programando algo MAL.
Al poner esa variable (numérica) entre apóstrofos estás escondiendo un error de programación grave. Eso no se debe hacer.
Lo que tienes que mirar ahora es:

1) Quita los apostrofos que le pusiste indebidamente a esa variable.

2) Verifica dónde se está perdiendo el valor de $id. Algún error estas cometiendo por el que la variable entra nula.

[Rag_Doll]

Hola a Todos!!!

bueno, tanto chronos682 como gnzsoloyo tenian razòn, y era que la variable $id era nula, el metodo que hice no se que tan poco ortodoxo y viable pueda ser, acontinuaciòn pongo el codigo:

Código PHP:

function actualizar($idcliente,$tipoid,$numero,$nombre,$apellido,$telefono,$celular,$correo,$nacimiento,$ocupacion,$convenio,$inscripcion){
        if($this->con->conectar()==true){
    $qry = "UPDATE clientes SET idcliente=$idcliente ,
    tipo='$tipoid', 
    numero='$numero' , 
    nombre='$nombre', 
    apellido='$apellido', 
    telefono='$telefono', 
    celular='$celular', 
    correo='$correo', 
    nacimiento='$nacimiento', 
    ocupacion='$ocupacion', 
    convenio='$convenio', 
    inscripcion='$inscripcion'
WHERE idcliente = $idcliente ";
return mysql_query($qry);
        }
    }
 

lo ùnico que cambie fue una variable màs con el id, ahora, puede que sea un metodo rudimentario, si es asi, resivo criticas constructivas y consejos que me puedan brindar para mejorar, no siendo mas muchas gracias por todos sus importantes aportes y hasta la pròxima!!!

[chronos682]

Si, así está bien, la variable id cliente tiene el mismo valor que el id pasado por get.