0

Tengo un error el cual no he podido solucionar, da los resultados esperados pero al final me saca un error.

Este es mi código:

<?php

require('datos.php');

$conexion=mysqli_connect($db_host,$db_usuario,$db_contra,$db_nombre);

if($conexion) {
    if($_POST) {    

        $Cedula=$_POST['cedula'];
        $consulta_existe ="SELECT * FROM clientes WHERE cedula='$Cedula'";
        $resultados_existe=mysqli_query($conexion,$consulta_existe);

        if($resultados_existe==1) {
            echo "<script type='text/javascript'> alert('No existe.'); </script>";
        } else {
            echo "<script type='text/javascript'> alert('El cliente ya esta registrado.'); </script>";  
        }

    }
}

?>

introducir la descripción de la imagen aquí

4
  • NO PUEDO VER LA IMAGEN, PUEDES DECIR CUAL ES EL ERROR ?
    – srJJ
    Commented el 9 ago. 2018 a las 0:37
  • mysqli_query devuelve un objeto que es un conjunto de resultados. Para poder leerlo tienes que aplicar algún método fetch. No puedes obtener los datos directamente de lo que te devuelve mysqli_query. No te sugiero una respuesta porque tampoco tu código está claro. Haces un SELECT * con un WHERE pero no especificas si te interesa una sola fila o todas las filas que cumplan esa condición, tampoco se sabe cuál es la columna que te interesa obtener... Además, tu consulta es vulnerable a ataques de Inyección SQL.
    – A. Cedano
    Commented el 9 ago. 2018 a las 0:45
  • @A.Cedano tan claro como siempre... me voy a permitir darle una posible respuesta en base a su codigo
    – srJJ
    Commented el 9 ago. 2018 a las 0:50
  • @srJJ claro. Pero convendría quizá indicarle el uso de consultas preparadas explicando el por qué.
    – A. Cedano
    Commented el 9 ago. 2018 a las 0:51

1 respuesta 1

0

Siguiendo lo que necesitas y las instrucciones del Maestro A. Cedano, te dejo una posible solucion a tu problema utilizando sentencias preparadas para evitar la injeccion de sql:

<?php

require('datos.php');

//creamos una instancia
$mysqli = new mysqli($db_host,$db_usuario,$db_contra,$db_nombre);
//verificamos que no hayan errores en la conexion
if(!$mysqli->connect_errno){
  //validamos que los datos llegue y que no esten vacios
  if(isset($_POST['cedula']) && !empty($_POST['cedula']))  {   

    //capturamos la variable del post
    $Cedula=$_POST['cedula'];
    //preparamos la sentencia con el simbolo "?"
    $sql = $mysqli->prepare("SELECT * FROM clientes WHERE cedula=$Cedula");
    //enviamos el parametro que preparamos 
    $sql->bind_param("i", $Cedula)
    //ejecutamos la sentencia
    $sql->execute();
    //obtenemos el resultado
    $resultado = $sql->get_result();
    //sacamos el numero de filas que trae
    $fila = $resultado->fetch_assoc();
    //validamos que si haya traido por lo menos una fila
    if($fila>0) {
      echo "<script type='text/javascript'> alert('El cliente ya esta registrado.'); </script>";  
    }
    else    {
      echo "<script type='text/javascript'> alert('No existe.'); </script>";
    }



  }
  else{
    //validamos si no llegaron los datos
    echo "<script type='text/javascript'> alert('No llegaron los Datos'); </script>"; 
  }
}
else{
  //mostramos alerta por si fallo la conexion
  echo "<script type='text/javascript'> alert('Fallo la conexion a la BD'); </script>"; 
 }


?>

Espero te sirva y te apoyes en esto aqui te dejo el enlace del Manual de sentencias Preparadas exitos bro. ReNiceCode...

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.