Redondance incrémentation dans Compteur

**didpoy** · 28/01/2015, 10h06

Bonjour

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

Code :

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

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

**sabotage** · 28/01/2015, 10h52

Comment est appellé ce code ?

**didpoy** · 28/01/2015, 10h56

Je l'appelle directement par un lien http:

exemple:http://....../compteur.php

**Celira** · 28/01/2015, 13h35

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.

**didpoy** · 28/01/2015, 16h58

Quand j'ouvre la page une fois le compteur s'incrémente de 2 ou 3...

**Celira** · 29/01/2015, 09h58

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 ?

**didpoy** · 30/01/2015, 20h07

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 :

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
 
<?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

**sabotage** · 30/01/2015, 20h43

Pourquoi le code du compteur est repeté dans le if et dans le else ?

**didpoy** · 30/01/2015, 21h02

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 !!!

**sabotage** · 30/01/2015, 21h23

Dans tous les cas tu fais l'incrementation et la redirection donc ces deux elements pourraient être mis une seule fois apres le test.

**didpoy** · 30/01/2015, 21h29

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

**sabotage** · 30/01/2015, 21h42

Est-ce que tu as essayé sans redirection comme te l'a indiqué Celira ?

**didpoy** · 30/01/2015, 21h46

oui j'ai essayé et j'ai eu le problème

**sabotage** · 30/01/2015, 21h57

Bon quand même c'est plus propre de faire une redirection HTTP :

Code :

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

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

**didpoy** · 30/01/2015, 22h08

ok merci, je vais tenter...

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

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
 
<?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

**Spartacusply** · 30/01/2015, 22h48

Si ce test là :

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

ne passe pas, il se passe quoi ?

**didpoy** · 31/01/2015, 10h07

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

**sabotage** · 31/01/2015, 10h47

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

Redondance incrémentation dans Compteur [PHP 5.0]

Langage PHP

Discussions similaires

Partager

Partager