Discussion :

[jmarey]

Bonjour,
j'essais d'envoyer des mails par paquets et j'ai un soucis de redirection
Voila ce que je fais

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
 
        define('NBMAILS',50);  // nombre de mail par paquets 
        $nombreTour = 0;   
        if( isset($_GET['debut']) ) 
                    { 
                        $debut=$_GET['debut']; 
                    } else  
                    { 
                        $debut=0; 
                    } 
 
$Test='SELECT COUNT(*) FROM TClientBis WHERE News = 1 '; 
$Rest=mysql_query($Test,$db); 
$nombreEmail = mysql_numrows($Rest); 
 
$Requete='SELECT NoID,User,Nom,Prenom,Email,Pass,News,Activite FROM TClientBis WHERE News = 1 LIMIT '.$debut.','.NBMAILS.' ';
$Result=mysql_query($Requete,$db); 
$NbreClients = mysql_numrows($Result); 
for ($i=0;$i<$NbreClients;$i++) 
    {  
    // ici je taite les envois de mail  : mail($Email,$Subject,$Texte,$Headers); 
     $nombreTour++; // pour connaitre le nombre de mails envoyés j'ai initialisé à 1 avant la boucle et je l'incrémente 
        }   
     $Total = NBMAILS+$debut;  // j'ajoute  
 
if($nombreEmail == $nombreTour) header('Location: index.php?g_pg=9&NbreEmail='.$nombreEmail .'&NbreTour='.$nombreTour.' '); 
 
    else header('Location: '.$_SERVER['PHP_SELF'].'?debut='.$Total.'');

Ca fonctionne mais la redirection au final ne fonctionne pas : j'arrive à 500 et pas de redirection pour sortir de la page !
Est ce que quelqu'un voit ou je merdois
Merci de votre aide

[sabotage]

As-tu contrôlé tes variables ?
En regardant ton code est diagonale, je diras que tu ne tombes simplement pas pile sur 500 lignes dans ta base et donc que ta condition n'est jamais vérifiée.

[jmarey]

Comment verais tu la vérification ?

[sabotage]

avec un simple echo ou un var_dump

[jmarey]

merci,
mais un print ou echo, j'ai dépassé ce stade !

Si j'ai posté ce post c'est que j'ai fais le tour du problème sans trouver de solution.

A force de ligne de code on a + les idées bien claires.....d'où mon post

Merci quand même

[sabotage]

Un echo dit tout pourtant.

$nombreTour est le nombre de lignes traitées dans une page.
$nombreEmail est le nombre de lignes à traiter.

Ils ne peuvent être égaux que s'il n'y a qu'une seule fournée de mail.

[RunCodePhp]

Salut

Code :

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

1
2
3
$Test='SELECT COUNT(*) FROM TClientBis WHERE News = 1 '; 
$Rest=mysql_query($Test,$db); 
$nombreEmail = mysql_numrows($Rest);

La fonction mysql_num_rows() (qui d'ailleurs comporte une erreur de syntaxe car c'est mysql_num_rows et non mysql_numrows) renvoi le nombre de lignes que retourne la requête.
Or, la particularité de cette requête c'est que le COUNT(*) fera qu'il y aura uniquement 1 seule ligne de retournée, soit 0 soit 1 (jamais au-delà).
C'est la valeur du COUNT(*) qui contient le nombre de mail, c'est plutôt celle-ci qu'il faudrait exploiter, non ?

Comme ceci :

Code :

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

1
2
3
4
5
 
$Test='SELECT COUNT(*) AS total FROM TClientBis WHERE News = 1 '; 
$Rest=mysql_query($Test, $db);
$clients = mysql_fetch_array($Rest);
$nombreEmail = (int)$clients['total'];

$nombreTour est le nombre de lignes traitées dans une page.
$nombreEmail est le nombre de lignes à traiter.

Effectivement, ça devrait être plutôt $debut + $nombreTour au niveau de la condition pour sortir à la fin, non ?

D'ailleurs, j'aurais plus vu d'incrémenter $debut dans la boucle, ensuite, transmettre sa valeur pour le prochain paquet, tout simplement, non ?

Quelque chose dans ce genre là :

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
 
$debut = (isset($_GET['debut']) && (int)$_GET['debut'] > 0) ? (int)$_GET['debut'] : 0;
 
// Nombre total de mails à envoyer
$Test = 'SELECT COUNT(*) AS total FROM TClientBis WHERE News = 1 '; 
$Rest=mysql_query($Test, $db);
$clients = mysql_fetch_array($Rest);
$nombreEmail = (int)$clients['total'];
 
// Requête par paquet (50)
$sql = 'SELECT ... blablabla ... LIMIT '.(int)$debut.', 50';
... etc ...
while ($mails = mysql_fetch_array($rs)) {
    // Envoi des mail
 
    // Incrémentation
    $debut++;
}
 
// Redirections
if ($nombreEmail == $debut) {
    // Tous les mails envoyés : On sort définitivement
}
else {
    // Envoie du paquet suivant
}

Enfin, c'est une idée, par sût que ça soit correcte.