Skip to main content
cuerpo editado
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257
  • El código tiene algunos problemas. Entre ellos que no es coherente. Si Ajax espera un json, que fue lo que indicaste en el dataType, no es buena práctica hacer un simple echo si hay un error. Debes recoger en una variable el resultado del código e imprimirlo al final como json, pues eso es lo que Ajax espera.
  • En PDO tienesEn PDO tienes que usar fetchAll y no fetch si quieres todos los resultados en un arreglo, de golpe. fetch es para ir iterando en un bucle. Ignora esto. El comentario de @aldanux me ha hecho ver que usar fetchAll y nocon fetch si quieres todos los resultadospuedes en efecto almacenar una fila completa en un arreglo,array de golperesultados. Si fuera más de una fila (y no son muchas filas), puedes entonces usar fetchfetchAll es para ir iterando en un bucle.
  • No le veo mucha lógicológica al uso de dos consultas, primero una para contar y luego otra para obtener los datos. Creo que se podría hacer con una sola, pero eso queda a opción tuya.
public function buscarCliente($ced){

    $buscar = "SELECT COUNT(*) FROM registro_clientes WHERE rc_cedu=:a";
    $resultado = $this->db_conexion->prepare($buscar);
    $resultado->execute(array(':a'=>$ced));

    $filas = $resultado->fetchColumn();

    if($filas>0){

        $extraer = "SELECT rc_cedu,rc_nomb,rc_aped,rc_telf,rc_dire FROM registro_clientes WHERE rc_cedu=:a";
        $resultado2 = $this->db_conexion->prepare($extraer);
        $resultado2->execute(array(':a'=>$ced));

        $arrResultado["datos"][] = $resultado2->fetchAll>fetch(PDO::FETCH_ASSOC);

        $resultado2->closeCursor();
    }else{
        $arrResultado=array("error"=>"No se encontraron datos");
    }

    $resultado->closeCursor();
    $this->db_conexion = null;

        header('Content-Type: application/json');
        echo json_encode($arrResultado);

 }
  • El código tiene algunos problemas. Entre ellos que no es coherente. Si Ajax espera un json, que fue lo que indicaste en el dataType, no es buena práctica hacer un simple echo si hay un error. Debes recoger en una variable el resultado del código e imprimirlo al final como json, pues eso es lo que Ajax espera.
  • En PDO tienes que usar fetchAll y no fetch si quieres todos los resultados en un arreglo, de golpe. fetch es para ir iterando en un bucle.
  • No le veo mucha lógico al uso de dos consultas, primero una para contar y luego otra para obtener los datos. Creo que se podría hacer con una sola, pero eso queda a opción tuya.
public function buscarCliente($ced){

    $buscar = "SELECT COUNT(*) FROM registro_clientes WHERE rc_cedu=:a";
    $resultado = $this->db_conexion->prepare($buscar);
    $resultado->execute(array(':a'=>$ced));

    $filas = $resultado->fetchColumn();

    if($filas>0){

        $extraer = "SELECT rc_cedu,rc_nomb,rc_aped,rc_telf,rc_dire FROM registro_clientes WHERE rc_cedu=:a";
        $resultado2 = $this->db_conexion->prepare($extraer);
        $resultado2->execute(array(':a'=>$ced));

        $arrResultado["datos"][] = $resultado2->fetchAll(PDO::FETCH_ASSOC);

        $resultado2->closeCursor();
    }else{
        $arrResultado=array("error"=>"No se encontraron datos");
    }

    $resultado->closeCursor();
    $this->db_conexion = null;

        header('Content-Type: application/json');
        echo json_encode($arrResultado);

 }
  • El código tiene algunos problemas. Entre ellos que no es coherente. Si Ajax espera un json, que fue lo que indicaste en el dataType, no es buena práctica hacer un simple echo si hay un error. Debes recoger en una variable el resultado del código e imprimirlo al final como json, pues eso es lo que Ajax espera.
  • En PDO tienes que usar fetchAll y no fetch si quieres todos los resultados en un arreglo, de golpe. fetch es para ir iterando en un bucle. Ignora esto. El comentario de @aldanux me ha hecho ver que con fetch puedes en efecto almacenar una fila completa en un array de resultados. Si fuera más de una fila (y no son muchas filas), puedes entonces usar fetchAll.
  • No le veo mucha lógica al uso de dos consultas, primero una para contar y luego otra para obtener los datos. Creo que se podría hacer con una sola, pero eso queda a opción tuya.
public function buscarCliente($ced){

    $buscar = "SELECT COUNT(*) FROM registro_clientes WHERE rc_cedu=:a";
    $resultado = $this->db_conexion->prepare($buscar);
    $resultado->execute(array(':a'=>$ced));

    $filas = $resultado->fetchColumn();

    if($filas>0){

        $extraer = "SELECT rc_cedu,rc_nomb,rc_aped,rc_telf,rc_dire FROM registro_clientes WHERE rc_cedu=:a";
        $resultado2 = $this->db_conexion->prepare($extraer);
        $resultado2->execute(array(':a'=>$ced));

        $arrResultado["datos"][] = $resultado2->fetch(PDO::FETCH_ASSOC);

        $resultado2->closeCursor();
    }else{
        $arrResultado=array("error"=>"No se encontraron datos");
    }

    $resultado->closeCursor();
    $this->db_conexion = null;

        header('Content-Type: application/json');
        echo json_encode($arrResultado);

 }
cuerpo editado
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257
public function buscarCliente($ced){

    $buscar = "SELECT COUNT(*) FROM registro_clientes WHERE rc_cedu=:a";
    $resultado = $this->db_conexion->prepare($buscar);
    $resultado->execute(array(':a'=>$ced));

    $filas = $resultado->fetchColumn();

    if($filas>0){

        $extraer = "SELECT rc_cedu,rc_nomb,rc_aped,rc_telf,rc_dire FROM registro_clientes WHERE rc_cedu=:a";
        $resultado2 = $this->db_conexion->prepare($extraer);
        $resultado2->execute(array(':a'=>$ced));

        $arrResultado["datos"][] = $resultado2->fechtAll>fetchAll(PDO::FETCH_ASSOC);

        $resultado2->closeCursor();
    }else{
        $arrResultado=array("error"=>"No se encontraron datos");
    }

    $resultado->closeCursor();
    $this->db_conexion = null;

        header('Content-Type: application/json');
        echo json_encode($arrResultado);

 }
public function buscarCliente($ced){

    $buscar = "SELECT COUNT(*) FROM registro_clientes WHERE rc_cedu=:a";
    $resultado = $this->db_conexion->prepare($buscar);
    $resultado->execute(array(':a'=>$ced));

    $filas = $resultado->fetchColumn();

    if($filas>0){

        $extraer = "SELECT rc_cedu,rc_nomb,rc_aped,rc_telf,rc_dire FROM registro_clientes WHERE rc_cedu=:a";
        $resultado2 = $this->db_conexion->prepare($extraer);
        $resultado2->execute(array(':a'=>$ced));

        $arrResultado["datos"][] = $resultado2->fechtAll(PDO::FETCH_ASSOC);

        $resultado2->closeCursor();
    }else{
        $arrResultado=array("error"=>"No se encontraron datos");
    }

    $resultado->closeCursor();
    $this->db_conexion = null;

        header('Content-Type: application/json');
        echo json_encode($arrResultado);

 }
public function buscarCliente($ced){

    $buscar = "SELECT COUNT(*) FROM registro_clientes WHERE rc_cedu=:a";
    $resultado = $this->db_conexion->prepare($buscar);
    $resultado->execute(array(':a'=>$ced));

    $filas = $resultado->fetchColumn();

    if($filas>0){

        $extraer = "SELECT rc_cedu,rc_nomb,rc_aped,rc_telf,rc_dire FROM registro_clientes WHERE rc_cedu=:a";
        $resultado2 = $this->db_conexion->prepare($extraer);
        $resultado2->execute(array(':a'=>$ced));

        $arrResultado["datos"][] = $resultado2->fetchAll(PDO::FETCH_ASSOC);

        $resultado2->closeCursor();
    }else{
        $arrResultado=array("error"=>"No se encontraron datos");
    }

    $resultado->closeCursor();
    $this->db_conexion = null;

        header('Content-Type: application/json');
        echo json_encode($arrResultado);

 }
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Prueba de este modo. He aplicado algunas cosas dichas en comentario.

##jQuery

  • He usado código actualizado de jQuery (document.ready está obsoleto).

  • He agregado el fail.

  • Si respondes adecuadamente en el servidor no necesitas parsear el JSON luego.

     $(function() {
    
         // Asigno un evento a un botón de mi formulario
    
         $("[name='buscar_cli']").click(function(e){
    
             e.preventDefault();
    
             var datos_enviados = {
                 'buscar_cli' : $("[name='ced_cli']")
             }
    
    
         var request = $.ajax({
           url: "bd_comun.php",
           method: "POST",
           data: datos_enviados,
           dataType: "json"
         });
    
    
         request.done(function( data ) {
             alert("Todo bien");
             console.log(data); //Si pones el content-type en PHP no necesitas parse         
         });
    
         request.fail(function( jqXHR, textStatus ) {
           alert( "Hubo un error: " + textStatus );
         });
    
    
         })
    
     });
    

##PHP

  • El código tiene algunos problemas. Entre ellos que no es coherente. Si Ajax espera un json, que fue lo que indicaste en el dataType, no es buena práctica hacer un simple echo si hay un error. Debes recoger en una variable el resultado del código e imprimirlo al final como json, pues eso es lo que Ajax espera.
  • En PDO tienes que usar fetchAll y no fetch si quieres todos los resultados en un arreglo, de golpe. fetch es para ir iterando en un bucle.
  • No le veo mucha lógico al uso de dos consultas, primero una para contar y luego otra para obtener los datos. Creo que se podría hacer con una sola, pero eso queda a opción tuya.

Espero te sirva.

public function buscarCliente($ced){

    $buscar = "SELECT COUNT(*) FROM registro_clientes WHERE rc_cedu=:a";
    $resultado = $this->db_conexion->prepare($buscar);
    $resultado->execute(array(':a'=>$ced));

    $filas = $resultado->fetchColumn();

    if($filas>0){

        $extraer = "SELECT rc_cedu,rc_nomb,rc_aped,rc_telf,rc_dire FROM registro_clientes WHERE rc_cedu=:a";
        $resultado2 = $this->db_conexion->prepare($extraer);
        $resultado2->execute(array(':a'=>$ced));

        $arrResultado["datos"][] = $resultado2->fechtAll(PDO::FETCH_ASSOC);

        $resultado2->closeCursor();
    }else{
        $arrResultado=array("error"=>"No se encontraron datos");
    }

    $resultado->closeCursor();
    $this->db_conexion = null;

        header('Content-Type: application/json');
        echo json_encode($arrResultado);

 }