0

Quiero realizar una varias consultas en un ciclo teniendo como condicional el array[$correos] y guardar los resultados en otro array, pero solo se me guarda el último valor encontrado y concretamente son dos resultados en este caso. ¿Cómo podría hacer lo anterior descrito?

for($i=0; $i<sizeof($correos)-1; $i++){
    $documento_destinatarios = $link->query("SELECT usu_docume FROM usuario WHERE usu_email = '$correos[$i]'");
}


$row = mysqli_fetch_array($documento_destinatarios, MYSQLI_NUM);
printf ("%s/ %s\n", $row[0], $row[1]);

1 respuesta 1

2

Actualización

Gracias a los comentarios de @A.Cedano también se puede replantear todo de este modo:

$correos_destinatarios = '"'.rtrim(implode('","',$correos), ',').'"';
$documentos_destinatarios = $link->query("SELECT usu_docume FROM usuario WHERE usu_email IN($correos_destinatarios)");

while ($row = mysqli_fetch_array($documentos_destinatarios)) {
    echo $row['usu_docume']." / ";
}
echo "\n";

Con esta solución se evita sobrecargar MySQL con demasiadas consultas consecutivas y pasar el resultado de la única consulta a PHP para que haga el resto mediante el bucle while recorriendo todas las filas de resultados de la consulta.

Por otra parte, segun comenta el OP, el array $correos puede contener un registro vacio al final de su array, por lo tanto al hacer el implode() de sus elementos nos va a generar una coma de más, que la eliminamos con rtrim() de este modo:

$correos_destinatarios = '"'.rtrim(implode('","',$correos), ',').'"';

a la vez que agregamos comillas a todos los elementos del array para que queden entrecomillados y no se produzcan errores al usar la claúsula IN en SQL.

Repuesta inicial

Debes ponerlo todo dentro del bucle, sino vas sobreescribiendo la variable $documento_destinatarios hasta que sale del bucle, donde solo conserva la asignación de la consulta del último ciclo.

for($i=0; $i<sizeof($correos)-1; $i++){
    $documento_destinatarios = $link->query("SELECT usu_docume FROM usuario WHERE usu_email = '$correos[$i]'");
    $row = mysqli_fetch_array($documento_destinatarios, MYSQLI_NUM);
    printf ("%s/ %s\n", $row[0], $row[1]);
}
11
  • Muchas gracias, me ayudaste mucho.
    – Acris
    el 28 abr. 2022 a las 15:15
  • Lo propio sería escribir la consulta así: WHERE usu_email IN(implode(',',$correos)) de eso modo, ejecutarías una sola consulta, no N consultas, ganando en rendimiento. Evidentemente, al hacer eso, sobra el for, luego, para leer los resultados sí usas un for o mejor un while.
    – A. Cedano
    el 28 abr. 2022 a las 15:16
  • Puede ser, pero por el momento sólo veo que quiere obtener el valor de usu_docume de un conjunto de correos, que pueden ser 2 o más ... en ese caso, en vez de mandar a ejecutar query() N veces es mejor una sola ejecución, de lo contrario el código no estará optimizado. Cada ejecución de query() tiene su precio.
    – A. Cedano
    el 28 abr. 2022 a las 15:22
  • @A.Cedano Sí sí, no leí lo que dijistes despues y entendi mejor la consulta, que genera varios registros que se pueden volver a recorrer para lo que sea, por eso acabé borrando mi comentario
    – masterguru
    el 28 abr. 2022 a las 15:24
  • 1
    Uf, al fin! :-) Me alegro que ya te haya servido! Saludos!
    – masterguru
    el 28 abr. 2022 a las 16:25

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.