0

Muy buenas, tengo un problema con unas pruebas de ajax que estoy realizando, es sobre un input que por un metodo keyup va enviando consultas a la base de datos, esto con ajax, me deberia retornar en consola un array con los datos de la consulta, y lo hace... el problema viene cuando coloco una letra que no coincide en la base de datos, me devuelve etiquetas html como las siguientes, deberia devolverme un array vacio (lo que esta en rojo hacia arriba es el error, lo de azul es como debería devolver los datos):

introducir la descripción de la imagen aquí

Aquí les dejo el codigo del php y el js, haganme saber si necesitan el html:

  AJAX.PHP  
<?php
    include('conect.php');
    $search = $_POST['search'];

    if (!empty($search)) {
        $query = "SELECt * FROM ajax WHERE name LIKE '$search%'";
        $process = mysqli_query($conect, $query);
        if (!$process) {
            die('query error' . mysqli_error($conect));
        }
        while ($register = mysqli_fetch_array($process)) {
            $json[] = array(
                'name' => $register['name'],
                'description' => $register['description'],
                'id' => $register['id']
            );
        }
        $jsonstring = json_encode($json);
        echo $jsonstring;
    }
    ?>

AJAX.JS

 $(document).ready(function () {
    $('#search').keyup(()=>{ 
        let search = $('#search').val()
        $.ajax({
            type: "POST",
            url: "ajax.php",
            data: { search },
            success: function (response) {
                // let tasks = JSON.parse(response)
                // console.log(tasks)
                console.log(response)
            }
        });
    });
});

Por cierto, en el js la la funcion JSON.parse() me devuelve error cuando me devuelven el monton de etiquetas html así que esta ignorado aquí

4
  • Porque te devuelve HTML que no hace parte del error??, estas acaso devolviendo HTML entero del lado del servidor??, debería devolverte solo el json...
    – Layer
    el 7 may. 2020 a las 20:18
  • todo funciona... hasta que la query a la db no coincide, me devuelve eso (si te fijas por ahí decia json no definido)
    – Flav
    el 7 may. 2020 a las 23:27
  • Muchas gracias Riven, pero hay un detalle, me marca error cuando borro todo y el input esta vacio, es normal?
    – Flav
    el 7 may. 2020 a las 23:45
  • 1
    En tu caso es normal eso, pero es debido a que solo estas retornando un json si la busqueda no esta vacia, pero en caso de que este vacia, desde el lado de php no estarias retornando nada, tendras que retornar siempre algo, de lo contrario el error que te marca debe ser jquery tratando de parsear algo que no es un json.
    – Layer
    el 7 may. 2020 a las 23:53

1 respuesta 1

2

Tu variable json debe estar declarada fuera de tu ciclo, por que si lo haces dentro de tu ciclo, esta solo existirá dentro de tu ciclo (lo mismo como si usaras let en js), así que declárala fuera de tu ciclo:

<?php
    include('conect.php');
    $search = $_POST['search'];

    if (!empty($search)) {

        //Aqui debes declarar a tu variable $json.
        $json = [];
        $query = "SELECt * FROM ajax WHERE name LIKE '$search%'";
        $process = mysqli_query($conect, $query);
        if (!$process) {
            die('query error' . mysqli_error($conect));
        }
        while ($register = mysqli_fetch_array($process)) {
            array_push(
               $json,
               array(
                'name' => $register['name'],
                'description' => $register['description'],
                'id' => $register['id']
               )
            );
        }
        $jsonstring = json_encode($json);
        echo $jsonstring;
    }
?>

Y bueno... tambien en esta linea para no confundirnos y tampoco confundir a la variable, lo que tu tenias era:

$json[] = array(
   'name' => $register['name'],
   'description' => $register['description'],
   'id' => $register['id']
);

Y lo cambie a:

$json = array(
   'name' => $register['name'],
   'description' => $register['description'],
   'id' => $register['id']
);

Notese que no hay corchetes en $json.

También obsérvese que tu tenias un ciclo en el cual estabas cargando información, para poder que obtengas toda tu información y tu información no se pierda al reasignar la información, deberás hacer de tu json un array directamente y hacer un push a tu json de la información que estas obteniendo cada vez en el ciclo, de lo contrario la información traída al final por el json será la ultima recibida en el ciclo, entonces afuera del ciclo declaramos tu json como un array:

//Aqui debes declarar a tu variable $json.
$json = [];

Y dentro del ciclo hacemos esto:

array_push(
   $json,
   array(
     'name' => $register['name'],
     'description' => $register['description'],
     'id' => $register['id']
    )
);

De esta manera tendrás tu colección entera de información traída por el ciclo en un solo json, la cual podrás luego consultar en el frontend.

15
  • Solo dos correcciones: inicializa $json = []; y, dentro del ciclo se van a estar agregando elementos, entonces ahí sí estaba bien la asignación $json[] = array(...);
    – Triby
    el 7 may. 2020 a las 19:33
  • No es necesario, puedes dejarlo sin inicializar y posteriormente inicializarlo no es necesario especificar que tenga un valor de inicios. aunque si te refieres a esa inicializacion que hace nuestro OP, realmente no sabia que se podia declarar un arreglo tambien asi... pero a el declararlo de esa manera parece funcionarle... asi que decidi omitir ese detalle...
    – Layer
    el 7 may. 2020 a las 19:35
  • Es raro, estoy probando y funciona hasta sin declarar en PHP 7.2.14. Pero si dejas solo $json = array(...); se va a sobreescribir en cada iteración del ciclo.
    – Triby
    el 7 may. 2020 a las 19:42
  • Por supuesto, en eso tienes razon por que el OP, no tuvo en cuenta ese pequeno detalle de que esta sobrescribiendo el mismo json, pero bueno... eso hace parte del codigo del propio OP... no se si depronto el quiera que solucionemos ese detalle... ya que la pregunta es sobre resolver esos errores por pantalla que se ven...
    – Layer
    el 7 may. 2020 a las 19:44
  • 1
    Al final lo hiciste con array_push() que es lo mismo que $json[] = array('Elemento1', 'elemento2', 'etc'); pero con menor rendimiento. Lo bueno es que ahora sí va a funcionar. +1
    – Triby
    el 7 may. 2020 a las 20:19

Tu Respuesta

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

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