Discussion :

[mOscar007]

Bonjour à tous,

Je suis en train de réaliser un système d'envoi de newsletter pour une liste de distribution.
L'envoi marche bien, le seul souci est de mettre un arrêt de 10 seconde au bout de (100 emails envoyés par exemple) et ensuite reprendre l'envoi pour la suite.
C'est à dire que mon script ne rentre jamais dans cette condition :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
if ($i % $nbr == 0){
       sleep(10); 
}

Je vous mets ci-dessous mon code avec l'exemple.

Merci pour votre aide !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
 
$query = "select email from dbtable where sub_status = 1 LIMIT 1000";
$req = $db->prepare($query);
$req->execute() or die(print_r($req->errorInfo()));
$rows = $req->fetchAll();
 
$emails = array();
$nbr = 100;
$i = 0;
 
foreach ($rows as $row) {
    $emails[]=$row['email'];
}
 
$count =  file_get_contents("count.txt");
 
for($i=$count;$i<count($emails);$i++){
    $to  = $emails[$i];
    $subject = 'Titre';
 
        if ($i % $nbr == 0){
             sleep(10); 
        }        
 
        $message = file_get_contents("newsletter.html");
 
        $headers  = 'MIME-Version: 1.0' . "\r\n";
    $headers .= 'Content-type: text/html; charset=iso-8859-1' . "\r\n";
        $headers .= 'From: Name <info@domain.com>' . "\r\n";
 
        if(mail($to, $subject, $message, $headers)) {
            $file = fopen("sentlist.txt","a+"); 
            fwrite($file, $to.",\r\n");
            fclose($file);
     } else {
                $file = fopen("notsentlist.txt","a+"); 
            fwrite($file, $to.",\r\n");
            fclose($file);
         }
 
         if(($i-$count)>=1000) {   
            $filec = fopen("count.txt",'w');
            fwrite($filec, $i);
            fclose($filec);
            break;
      }   
 
         $++;
}
$filec = fopen("count.txt",'w');
fwrite($filec, $i);
fclose($filec);

[sabotage]

Qu'est ce qui te permets de dire que tu ne rentres pas dans la condition ?

Au passage, tu as un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$++

qui se ballade ligne 48.

Également, sur la lecture des données, pourquoi mets-tu le résultat de la requête dans un tableau, pour parcourir ce tableau et refaire un tableau pour ensuite parcourir le deuxième tableau pour faire le traitement ?

Traites les données simplement depuis la requête (et pas besoin de prepare() ici non plus)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
$query = "select email from dbtable where sub_status = 1 LIMIT 1000";
$req = $db->query($query);
while ($row = $req->fetch()) {
    $to  = $row['email'];
    $subject = 'Titre';

Une méthode plus simple et précise pour traiter les lots et d'avoir une colonne dans la base de données indiquant si la ligne a été traitée ou non.
Tu peux donc directement requêter 1000 lignes non traitées sans te soucier de gérer une position.

[-Rpass-]

Pourquoi ne pas faire une condition simple

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

(if $i==100)

au lieu d'un modulo ?

[mOscar007]

Re,

Merci @sabotage et @Rpass pour vos réponses.

Pour répondre à la ques de @sabotage :

Qu'est ce qui te permets de dire que tu ne rentres pas dans la condition ?

Quand je lance l'envoi de ma newsletter, j'envoie les 100 premiers mails et ensuite je n'ai rien d'autres. C'est ci qui m'amène à supposer que je rentre pas dans la condition mentionnée. Comment fas-tu avec cette partie du code ? :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
$count =  file_get_contents("count.txt");
for($i=$count;$i<count($emails);$i++)
{
}

Pour @Rpass,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$nbr = 100;

est la variable qui détermine le nombre de mails à envoyer par paquet (soit 100 sur un total de 1000). Et

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$i = 0;

est la variable qui s'incrémente au fur et à mesure de l'envoi. D’où à la fin j'ai mis : $i++;

Pour savoir si on a atteint les 100 mails envoyés par paquet, il faut que $i % $nbr soit égale à 0. D’où la condition :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5

if ($i % $nbr == 0){
   sleep(10); 
}

Cdlt,

[-Rpass-]

Je te conseille de faire un echo ou un die dans ta boucle pour vérifier qu'il passe dans ta condition .
S'il s'arrête au bout de 100 c'est que soit tu as 100 mails en BDD soit le sleep fait planter le script

[Celira]

Quand je lance l'envoi de ma newsletter, j'envoie les 100 premiers mails et ensuite je n'ai rien d'autres.

Question bête : combien de mails étais-tu censé envoyer ? Tu peux faire des essais en modifiant le nombre de mails par vague, pour voir si tu restes coincé à une seule vague, à 100 mails....

[mOscar007]

@Celira,

Le nombre total de mails à envoyer est 1000.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3

$query = "select email from dbtable where sub_status = 1 LIMIT 1000"; 
$req = $db->query($query);

Même en changeant le nombre (50 par exemple) ça n'envoie que 50.

Cdlt,

[-Rpass-]

C'est bien ton sleep qui fait planter donc .

T'as check les erreurs les logs php ?

[Celira]

A tout hasard: quelle est la valeur du max_execution_time ?

[mOscar007]

Re,

@Celira, la valeur du max_execution_time est de 120.
@Rpass, un check les erreurs les logs php m'affiche des suivantes :

- PHP Notice: Undefined index: in C:\wamp\www\news\envoick.php on line 18 correspondant à :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$to = $emails[$i];

- PHP Warning: A non-numeric value encountered in C:\wamp\www\news\envoick.php on line 41 correspondant à :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if (($j-$count)>=1000) {

Mais ces erreurs n'empêchent pas l'envoi des mails.

Cdlt,

[-Rpass-]

Je comprends pas trop ce que tu cherches à faire avec le fichier count.txt. Peux tu expliquer plus en détails
Pour le coup tu ne vas pas juste parcourir tous les mails et les envoyer avec une intervalle de 10sec tous les 100 mails ?

- PHP Notice: Undefined index: in C:\wamp\www\news\envoick.php on line 18 correspondant à :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$to = $emails[$i];

Mais ces erreurs n'empêchent pas l'envoi des mails.

Dans ce cas ça pose problème car sans l'email de destination l'email ne partira pas.

[mOscar007]

count.txt est utilisé pour stocker le nombre d'e-mails courants envoyés.
Cela enverra 1000 e-mails de la base de données par exécution et sera
utilisé comme point de départ de la prochaine exécution.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11

if(($i-$count)>=1000) {   
    $filec = fopen("count.txt",'w');
    fwrite($filec, $i);
    fclose($filec);
    break;
}   

$filec = fopen("count.txt",'w');
fwrite($filec, $i);
fclose($filec);

[-Rpass-]

Tu as les erreurs php à chaque exécution ? Corrige les erreurs pour voir si tu as toujours le problème