Redondance incrémentation dans Compteur

Version imprimable

Bonjour

J'ai mis en place un compteur dans un code PHP dont voici le principe:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
<?php
// ...
// Compteur de demandes
$fp = fopen("compteur.txt","r+"); // 1.On ouvre le fichier en lecture/écriture
$nbvisites = fgets($fp,11);       // 2.On récupère le nombre dans le fichier
$nbvisites++;                     // 3.On incrémente le nombre de visites(+1)
fseek($fp,0);                     // 4.On se place en début de fichier
fputs($fp,$nbvisites);            // 5.On écrit dans le fichier le nouveau nb
fclose($fp);                      // 6.On ferme le fichier
// Rediriger vers la page index
echo "<SCRIPT LANGUAGE='JavaScript'>document.location.href='index.html' </SCRIPT>";
?>

Le problème est que le compteur s'incrémente plusieurs fois au lieu de s'incrémenter qu'une seule fois lorsqu'on passe une fois dans le code.

Comment empêcher une redondance d'incrémentation ?

Merci

Cordialement

28/01/2015, 10h52
sabotage

Comment est appellé ce code ?
28/01/2015, 10h56
didpoy

re

Je l'appelle directement par un lien http:

exemple:http://....../compteur.php
28/01/2015, 13h35
Celira

Par "s'incrémente plusieurs fois", tu veux dire que tu ouvres la page une fois et ton compteur avance de 3 ou 4, ou tu ouvres plusieurs la page et à chaque fois ça incrémente de 1.
28/01/2015, 16h58
didpoy

re

Quand j'ouvre la page une fois le compteur s'incrémente de 2 ou 3...
29/01/2015, 09h58
Celira

Bizarre... J'ai essayé sans la redirection Javascript et en faisant à un echo($nbvisites); à la place, et ça marche parfaitement bien.

Est-ce que tu as aussi le problème sans la redirection ?

J'ai essayé en local et le problème ne semble pas se produire.

Mais cela se produit lorsqu'il est posé sur le serveur distant (Chez Free)

Voilà le codage un peu plus développé. Dans chaque cas il y un traitement différent qui correspond à mes besoins:

Code:

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
 
<?php
 
    // lecture de l'ip en cours
    $ip = isset($_SERVER['REMOTE_ADDR']) ? trim($_SERVER['REMOTE_ADDR']) : '';
    $ip1 = $ip;
    $ip = ip2long($ip);
 
        $filename = 'ips/'.$ip.'.txt';  // Nom du fichier avec IP
        if (file_exists($filename))  // Test si fichier existe
        {
            //Cas 1 - j'ai enlevé du code pour une meilleur compréhension
            // Dans chaque cas il y a un codage différent
            // Compteur de demandes
            $fp = fopen("compteur.txt","r+"); // 1.On ouvre le fichier en lecture/écriture
            $nbvisites = fgets($fp,11);       // 2.On récupère le nombre dans le fichier
            $nbvisites++;                     // 3.On incrémente le nombre de visites(+1)
            fseek($fp,0);                     // 4.On se place en début de fichier
            fputs($fp,$nbvisites);            // 5.On écrit dans le fichier le nouveau nb
            fclose($fp);                      // 6.On ferme le fichier
 
			// Rediriger vers la page index
			echo "<SCRIPT LANGUAGE='JavaScript'>document.location.href='index.html' </SCRIPT>";
 
        } else
                    // Cas 2  - j'ai enlevé du code pour une meilleur compréhension
                    // Compteur de demandes
                        $fp = fopen("compteur.txt","r+"); // 1.On ouvre le fichier en lecture/écriture
                        $nbvisites = fgets($fp,11);       // 2.On récupère le nombre dans le fichier
                        $nbvisites++;                     // 3.On incrémente le nombre de visites(+1)
                        fseek($fp,0);                     // 4.On se place en début de fichier
                        fputs($fp,$nbvisites);            // 5.On écrit dans le fichier le nouveau nb
                        fclose($fp);                      // 6.On ferme le fichier
 
			            // Rediriger vers la page index
			            echo "<SCRIPT LANGUAGE='JavaScript'>document.location.href='index.html' </SCRIPT>";
?>

Donc je ne vois pourquoi il y a redondance.

Merci

30/01/2015, 20h43
sabotage

Pourquoi le code du compteur est repeté dans le if et dans le else ?
30/01/2015, 21h02
didpoy

Re

C'est mon code qui nécessite cela.

Les 2 cas sont différents mais nécessite l'incrémentation du compteur tout de même.

Je ne peux pas le coder qu'une seule fois puisque je dois sortir vers ma page index selon le cas 1 o u 2 !!!
30/01/2015, 21h23
sabotage

Dans tous les cas tu fais l'incrementation et la redirection donc ces deux elements pourraient être mis une seule fois apres le test.
30/01/2015, 21h29
didpoy

re

Oui en effet, cela est envisageable et serait mieux...

Je vais revoir mon code pour arrivé à cette fin et voir si cela améliore l'incrémentation

je reviendrai ... :)

Merci
30/01/2015, 21h42
sabotage

Est-ce que tu as essayé sans redirection comme te l'a indiqué Celira ?
30/01/2015, 21h46
didpoy

oui j'ai essayé et j'ai eu le problème
30/01/2015, 21h57
sabotage
Bon quand même c'est plus propre de faire une redirection HTTP :
Code:

1 2 header('Location: index.html'); exit();

ok merci, je vais tenter...

Surtout voir demain comment n'avoir l'incrémentation et redirection qu'une seule fois dans mon code

Code:

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
 
<?php
 
    // lecture de l'ip en cours
    $ip = isset($_SERVER['REMOTE_ADDR']) ? trim($_SERVER['REMOTE_ADDR']) : '';
    $ip1 = $ip;
    $ip = ip2long($ip);
 
        $filename = 'ips/'.$ip.'.txt';  // Nom du fichier avec IP
        if (file_exists($filename))  // Test si fichier existe
        {
                $ip_blacklist = file('ips/'.$ip.'.txt');
 
                // test si l'ip existe dans le fichier
                if ( $ip == $ip_blacklist[0] )
                {
                        //Code 1
                        //.................
                        //.................
                                        // Compteur de demandes
                                        $fp = fopen("compteur.txt","r+"); // 1.On ouvre le fichier en lecture/écriture
                                        $nbvisites = fgets($fp,11);       // 2.On récupère le nombre dans le fichier
                                        $nbvisites++;                     // 3.On incrémente le nombre de visites(+1)
                                        fseek($fp,0);                     // 4.On se place en début de fichier
                                        fputs($fp,$nbvisites);            // 5.On écrit dans le fichier le nouveau nb
                                        fclose($fp);                      // 6.On ferme le fichier
 
			                            // Rediriger vers la page index
			                            header('Location: index.html');
                                        exit;
                 }
        } else
                    // Code 2
                    //.....................
                    //....................
                        // Compteur de demandes
                        $fp = fopen("compteur.txt","r+"); // 1.On ouvre le fichier en lecture/écriture
                        $nbvisites = fgets($fp,11);       // 2.On récupère le nombre dans le fichier
                        $nbvisites++;                     // 3.On incrémente le nombre de visites(+1)
                        fseek($fp,0);                     // 4.On se place en début de fichier
                        fputs($fp,$nbvisites);            // 5.On écrit dans le fichier le nouveau nb
                        fclose($fp);                      // 6.On ferme le fichier
 
			            // Rediriger vers la page index
			            header('Location: index.html');
                        exit;
?>

Faut que je trouve l'astuce

Merci

30/01/2015, 22h48
Spartacusply

Si ce test là :

Citation:

if ($ip == $ip_blacklist[0]) {

ne passe pas, il se passe quoi ?
31/01/2015, 10h07
didpoy

Si l'ip connectée est balcklistée, alors:
Je regarde si le blocage supérieur à 1 heure
si oui je débloque l'ip et je ré-autorise l'ip et incrémente mon compteur et je lance la suite d'un code d'envoi mail avec des données demandées
si non, je bloque la suite de la demande et je retourne à l'index

Si l'ip connectée n'est pas bloquée
J'incrémente le compteur et je lance la suite d'un code d'envoi mail avec des données demandées
31/01/2015, 10h47
sabotage

Encore une fois, ne répète pas du code, fais une fonction pour ton compteur par exemple.