mail() dentro de for, envía mas de 60 veces al usuario

[Sensorium]

Hola estimados colegas, estoy teniendo un problema que me tiene un poco preocupado, el hecho es que tengo un Newsletter que envía promociones a todos los usuarios de mi base de datos una vez al día.

Este Newsletter verifica por usuario, las preferencias de envío y compone el email de acuerdo a lo seleccionado por el User.

Lo hemos probado con 4 o 5 emails sin problema pero cuando los hacemos con toda la base de datos(4000), me envía los emails duplicados en cuestión de 1 por segundo a la misma persona, pude llegar a constatar que recibí 120 emails a la vez.

Intentamos en vez de enviar el email, generamos un archivo guardando un log como si fuese enviado y este guarda todos los usuarios sin duplicados, por eso descartamos el script.


El codigo es algo similar a esto:

Código PHP:

  foreach( $users as $u ): 
            $html_ofertas = array(); 
            # 
            if( !empty($u->email)): 
                $userCats =  explode(",", $u->preferer_cat_ids); 
             
                foreach(  $this->currentDeals as $cupon ): 
                     
                    if( in_array( $cupon->catg_id, $userCats ) ): 
                        $html_ofertas[] = $this->htmlOferta($cupon); 
                    endif; 
                     
                endforeach; 
                 
                if( count( $html_ofertas ) ): 
                    $message = $this->htmlHeader(  ) . implode('',$html_ofertas) . $this->htmlFotter($u); 
                    $this->sendEmailToUser($u, $message); 
                endif; 
                 
            endif; 
             
        endforeach; 
 
 
   private function sendEmailToUser( $user, $message ){ 
        $from            = "Mi Empresa <[email protected]>"; 
        $headers         = "From: $from\r\n"; 
        $headers        .= "Content-type: text/html\r\n"; 
        $email_subject   = "Tus ofertas para hoy"; 
        mail($user->email, $email_subject, $message, $headers);  
    }  
   

Se agradece cualquier ayuda!

[ocp001a]

Quizá se estén ejecutando varias instancias de tu script al mismo tiempo. De no ser el caso, verificaría la lista de emails. A simple vista no noto nada raro.

[ocp001a]

Lo olvidaba. Te sugiero generar un numero al azar (random) y crear un archivo con el nombre del numero aleatorio, de esa forma podrías verificar si se ejecutó en múltiples instancias.

[Sensorium]

ocp001a, Muchas gracias por contestar tan rápidamente, esta comunidad es grandiosa, siempre lo fue.

En cuanto a tus sugerencias, probamos las instancias de la class, como tú, también pensé en ese problema, pero la clase se instancia solo una vez, también probé generado un archivo que guardara el email del usuario cuando intentaba enviarlo y el resultado fue el esperado, es decir, sin emails repetidos.

Nuevamente, gracias.

[ocp001a]

Supongo el problema tampoco es el que yo sospechaba, sin embargo no me refería a la instancia de la clase, si no del script. A mi me sucedió en algunas ocasiones, Suponiendo que tu script se llame enviaemails.php, a veces había varios enviaemails.php corriendo al mismo tiempo, cada uno instanciando una vez su clase.

[Rodrhigo]

yo creo que en tu consulta SQL estas trayendo email duplicados
prueba agregar un DISTINCT despues del select...
ejemplo:
(select distinct email from correos where email is not null and email != "")

y esta linea podrias preguntarla en tu consulta.
if( !empty($u->email)):


y te pego una recomendacion que hacen en la pagina de php al querer enviar un email

Es importante indicar que la función mail() no es conveniente para grandes volúmenes de correo en bucle. Esta función abre y cierra un socket SMTP para cada correo, algo que no es muy eficiente.

Para enviar una gran cantidad de correo, ver los paquetes » PEAR::Mail y » PEAR::Mail_Queue.