[SOLUCIONADO] PHP OO array_map() no detecta 2º parametro como array

[guardarmicorreo]

Estoy creando una clase para gestionar los emails relacionados con mi blog desde el propio blog.

Para poder presentar los mensajes en los buzones necesito recoger toda la información posible de sus cabeceras. Esto se consigue con imap_headerinfo o con imap_header (imap_headers devuelve muy poca información de cabecera).

imap_headerinfo o su alias imap_header están pensadas para recoger la cabecera de un mensaje referenciado por su número en los parámetros de dichas funciones.

Ante esta limitación, decidí que quería lograr dos cosas con el método de extraer todas las cabeceras de todos los mensajes:

1) código optimizado a ser posible en una sola línea, clara y concisa.

2) a ser posible evitar foreach o cualquier otro loop o condicional.

así que concluí en que debía hacerlo así

Código PHP:

public function getAllHeaders(){
        return $this->all_headers=  array_map("\imap_headerinfo", $this->imap_stream, self::search("ALL"));
    }
 

La idea es sencilla y aparentemente me pareció posible.

self::search() es otro método que utiliza imap_search() para recoger todos los números de los correos de un buzón. El parámetro que le paso "ALL" indica que quiero que recoja todos los los correos en la bandeja; devuelve un array con la siguiente estructura

array(2) {
[0]=>
int(1)
[1]=>
int(2)
}

Es un array asociativo y bien sencillo.

El problema viene que cuando solicito todas las cabeceras a través de este método (getAllHeaders) me devuelve el siguiente error en array_map()

Warning: array_map(): Argument #2 should be an array in C:\xampp\htdocs\xampp\web\pruebas\imap.php on line 114

La línea 114 es la misma del return del método getAllHeaders.

¿Por qué me devuelve ese error si realmente le estoy pasando un array asociativo tal y como especifica la documentación de PHP?

[Italico76]

Si el array esta vacio, hay muchas funciones y estructuras de control incluso de PHP que no lo reconocen como array sino directamente NULL

Si is_null o empty() .. no deberias hacer nada o si quieres casting a array

Código PHP:

  public function getAllHeaders(){
        return $this->all_headers=  array_map("\imap_headerinfo", (array) $this->imap_stream, self::search("ALL"));
    }  
   

No le veo sentido al casting porque esta vacio... pero al menos no explotaria el array_map(), mejor chequea que no este vacio

[guardarmicorreo]

Hola Italico76! gracias por tu ayuda.

Acabo de comprobar otra vez que sea array, no imprimiendo sino que provisionalmente le he añadido un condicional

Código PHP:

public function getAllHeaders(){
        $numbers=self::search("ALL");
        if(is_array($numbers))
        return $this->all_headers=  array_map("\imap_headerinfo", $this->imap_stream, $numbers);
    }
 

Y me sigue devolviendo el mismo error en array_map().

Es decir, sí hay array, y es exactamente el que he mostrado porque también lo he imprimido y me devuelve el mismo error :S

[hhs]

ya verificaste que $this->imap_stream es un array ?

[guardarmicorreo]

Hola hhs, $this->imap_stream no es un array, es un stream a un servidor y buzón.

Si le hago un var_dump() me devuelve esto

resource(2) of type (imap)

[guardarmicorreo]

Vale, acabo de darme cuenta de dónde está mi error. Creía que el argumento 2 se refería al segundo argumento de imap_headerinfo, un error mío, cuando en realidad se refiere al segundo argumento de array_map que en realidad es el primer argumento de headerinfo, es decir, $this->imap_stream.

¿Podría solucionarlo creando un array momentáneo con una clave y como valor $this->imap_stream?

Gracias!

[guardarmicorreo]

Finalmente pude solucionar el problema así.

Código PHP:

return $this->all_headers=  array_map("\imap_headerinfo", array_fill(0, count($num=self::search("ALL")), $this->imap_stream), self::search("ALL"));
 

¿Os parece una manera correcta de traer todas las cabeceras de un buzón?

¿Y sino cómo lo haríais?

Gracias por vuestra orientación, a veces mi cabeza se lía sola :D

[hhs]

Algo que hace ruido es que estas usando un arreglo adicional solo para pasar el resource que dicho sea de paso es tantas veces como mensajes tengas. Que sucede si imap_search te devuelve 100 mensajes o mas. Te parece adecuado pasar un arreglo con 100 posiciones para pasar el resource ?

[guardarmicorreo]

No me parece óptimo, tienes razón en ello, pero no se me ocurre otra manera para que array_map funcione y pueda tener el código limpio a la vista, o por lo menos esa es la intención :D

[hhs]

Si lo quieres dejar asi, necesitas pensar como pasar el imap resource a imap_headerinfo. por otro lado cual es la razón de de no usar la iteración ?

[guardarmicorreo]

Iniciado por hhs

Si lo quieres dejar asi, necesitas pensar como pasar el imap resource a imap_headerinfo. por otro lado cual es la razón de de no usar la iteración ?

Pienso y no encuentro posibilidad alguna, según la doc de php al respecto de array_map los parámetros para el callback deben ser así. No veo ningún resquicio por donde poder meter el imap_stream.

El motivo es lograr un código fácil de leer, limpio, esto entiendo que implica evitar loops si tengo funciones nativas disponibles que en este caso pueden sustituir un foreach por ejemplo.

[Eleazan]

Nunca he usado imap, y puede que meta la pata pero...

Si $this->imap_stream no es un array, ¿pq usas array_map?

¿No sería más sencillo usar imap_headerinfo sin usar array map?
Vamos, digo yo... xD

Edit: Sorry, no habia visto la respuesta con array_fill... Aunq quizás te habria valido sólo con usar imap_headers ;)

[lolainas]

Creo que buscas esta función: function.imap-headers

con lo cual tu línea quedaría mas o menos así:

Código PHP:

class MailBox {
 
    private $stream;
 
    function __construct($box, $user, $password) {
        $this->stream = imap_open($box, $user, $password);
    }
 
    function getHeaders() {
        return imap_headers($this->stream);
    }
 
}
 

EDITO: Acabo de leer que no te sirve imap_headers, perdón!

Código PHP:

function getHeaders() {
    return array_map(function($msg) { return imap_header($this->stream, $msg); }, imap_search($this->stream, 'ALL'));
}
 

[guardarmicorreo]

Gracias Eleazan y lolainas!

Me reté a mi mismo buscar una manera de hacerlo en una sola línea y a ser posible sin loops o condicionales para tener un código limpio, de ahí mi empeño en utilizar llamadas de retorno y creí que con array_map podría hacerlo. El problema es que con array_map los dos argumentos deben ser arrays de la misma longitud para que en cada avance del puntero en cada array realice una operación con cada valor.

Una función nativa que solucione ese problema sería maravilloso :D

lolainas, leo tu edición y el código lo entiendo así:

Un array_map que tiene los siguientes parámetros:

1) una función anónima que tiene un parámetro y retorna imap_header que tiene el stream y $msg, lo que no entiendo es cómo paso $msg a la función anónima.

2) está claro, es un imap_search sin más.

[lolainas]

$msg es cada valor que devuelve imap_search, osea, un entero (el numero de mensaje)

[guardarmicorreo]

Vale ya lo he visto, estaba viendo la funcion anonima como parte del segundo argumento del array_map y no como la llamada de retorno de array_map, mi fault :D

Gracias a todos! ha sido magnífico

[guardarmicorreo]

Una pregunta crucial

Entre esta solución

Código PHP:

foreach(self::search("ALL") as $msg_number){$headers[]=\imap_headerinfo($this->imap_stream, $msg_number);}
return $headers;
 

Y esta otra

Código PHP:

return array_map(function($msg) { return imap_header($this->imap_stream, $msg); }, imap_search($this->imap_stream, 'ALL'));
 

¿Cual utilizaríais preferiblemente?

[Italico76]

Algo que he leido por ahi... las funciones array_ map() reduce() etc son propias de la programacion funcional y dan mucha potencia en el sentido de simplificar codigo pero tambien ofuscan bastante el codigo:

La solucion mas "limpia" seria el foreach()

[Eleazan]

Diria que array_map utiliza más memoria que foreach.

[hhs]

Iniciado por guardarmicorreo

El motivo es lograr un código fácil de leer, limpio, esto entiendo que implica evitar loops si tengo funciones nativas disponibles que en este caso pueden sustituir un foreach por ejemplo.

Que es mas legible ??

Código PHP:

foreach(self::search("ALL") as $msg_number){$headers[]=\imap_headerinfo($this->imap_stream, $msg_number);}
return $headers;
 
return array_map(function($msg) { return imap_header($this->imap_stream, $msg); }, imap_search($this->imap_stream, 'ALL'));
 

Código PHP:

foreach(self::search("ALL") as $msg_number)
{
    $headers[] = \imap_headerinfo($this->imap_stream, $msg_number);
}
return $headers;
 
return array_map(
    function($msg) {
        return imap_header($this->imap_stream, $msg);
    },
    imap_search($this->imap_stream, 'ALL')
);
 

El que tengas tu código en una linea no siempre significa que el código es limpio y legible. Solo es compacto.
De todas formas este tipo de cosas ya es preferencia de cada quien.

En cuanto a que solución usar pues te lo pongo de esta forma ¿Crees que se buena idea usar array_map solo para iterar ? en tu caso el foreach y el array_map estan haciendo lo mismo.

Para terminar si quieres escribir código mas legible hay dos principios que debes de tomar en cuenta:

• El código debe ser sencillo de entender
• El código debe de ser escrito para reducir el tiempo que le tomaria a una persona comprenderlo

Las ideas las tome prestadas del libro: The Art of Readable Code que seria buena idea te tomaras un tiempo para leer.
Tambien puedes leer la convención PSR-1 y PSR-2 las cuales aclaro, no estoy siguiendo en mis ejemplos.

[guardarmicorreo]

Muchas gracias hhs por tu nota final.

Aquí he aprendido varias cosas, es un placer! y disfruto con ello.

Saludos :D