undifined variable sur compteur.

Version imprimable

Bonjour à tous,

Je me remet tout doucement au php et au dev web. J'ai voulu rajouter un compteur et je me retrouve avec une erreur que je n'arrive pas à corriger.

L'erreur est la suivante :

Code:

Unknown: Your script possibly relies on a session side-effect which existed until PHP 4.2.3. Please be advised that the session extension does not consider global variables as a source of data, unless register_globals is enabled. You can disable this functionality and this warning by setting session.bug_compat_42 or session.bug_compat_warn to off, respectively in Unknown on line 0

Voici le code du compteur :
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 session_start(); // DEMARRE LA SESSION // SAUVGARDE LA VARIABLE hits DANS LE FICHIER DE SESSION $_SESSION['hits'] = $hits; //le problème viendrait d'ici... // TRAITEMENT SUR LE FICHIER TEXTE if(empty($hits)){ $fp=fopen("compteur.txt","a+"); //OUVRE LE FICHIER compteur.txt $num=fgets($fp,4096); // RECUPERE LE CONTENUE DU COMPTEUR fclose($fp); // FERME LE FICHIER $hits=$num - -1; // TRAITEMENT $fp=fopen("compteur.txt","w"); // OUVRE DE NOUVEAU LE FICHIER fputs($fp,$hits); // MET LA NOUVELLE VALEUR fclose($fp); // FERME LE FICHIER } // AFICHAGE DU COMPTEUR echo "<TABLE align=center>"; echo "<TR>"; echo "<TD STYLE='border:1pt Solid navy;' >"; echo "<FONT FACE='Verdana, Arial, Helvetica, sans-serif' SIZE=1>"; echo "Visiteurs : $hits"; // VARIABLE DU COMPTEUR echo "</FONT>"; echo "</TD>"; echo "</TR>"; echo "</TABLE>"; ?>
Merci d'avance pour vote aide.

D.

17/08/2009, 16h41
sabotage

D'ou vient $hits ?
17/08/2009, 16h42
RideKick

Salut ,
tu as essayé de suivre ce que te demande le message ?
17/08/2009, 16h48
zebulon94

euh $hit .... bonne question, en fait je veux ouvrir une session pour que le compteur evite de s'incrémenter de 1 à chaque changement de page .

RideKick, oui j'ai essayé de suivre se que me dis le message. L'erreur est en réalité caché mais cela ne fonctionne toujours pas ...

D.

Si $_SESSION['hits'] est juste la pour marquer que tu as déjà fait l'incrémentation, c'est plutot quelque chose comme ca que tu veux :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
session_start(); 
 
if(isset($_SESSION['hits'])){ // si $_SESSION['hits'] est défini 
  $fp=fopen("compteur.txt","a+"); //OUVRE LE FICHIER compteur.txt
  $num=fgets($fp,4096); // RECUPERE LE CONTENUE DU COMPTEUR
  fclose($fp); // FERME LE FICHIER
  $hits=$num - -1;  // TRAITEMENT
  $fp=fopen("compteur.txt","w");  // OUVRE DE NOUVEAU LE FICHIER
  fputs($fp,$hits); // MET LA NOUVELLE VALEUR
  fclose($fp);  // FERME LE FICHIER
  $_SESSION['hits'] = TRUE; // on défini $_SESSION['hits']
}

18/08/2009, 09h33
zebulon94
ok juste petite question, la fonctions isset() permet de savoir si la variable est définie.
Dans notre exemple :
Code:

1 2 if(isset($_SESSION['hits'])){ }
Si l'utilisateur entreprend sa première connexion,

Code:

isset($SESSION['hits']) =>retourne FALSE

donc
le if() ne sera pas utlisé, et j'aurais une non ?

Merci de vos réponse ;)
D.
18/08/2009, 09h35
sabotage

Heu oui pardon je l'ai fais à l'envers :oops:

Code:

if(!isset($_SESSION['hits'])){ // si $_SESSION['hits'] n'est pas défini

Ya pa de mal, bon sa marche mais a chaque fois que la page est rafraichis sa me fait +1 en visite alors voila à quoi j'ai pensé, peux tu me dire se que tu en pense ...?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
// TRAITEMENT SUR LE FICHIER TEXTE
if(!isset($_SESSION['hits'])){ //si isset($_SESSION['hits']) n'est pas def alors 
  $fp=fopen("compteur.txt","a+"); //OUVRE LE FICHIER compteur.txt
  $num=fgets($fp,4096); // RECUPERE LE CONTENUE DU COMPTEUR
  fclose($fp); // FERME LE FICHIER
  $hits=$num - -1;  // TRAITEMENT
  $fp=fopen("compteur.txt","w");  // OUVRE DE NOUVEAU LE FICHIER
  fputs($fp,$hits); // MET LA NOUVELLE VALEUR
  fclose($fp);  // FERME LE FICHIER
  $_SESSION['hits'] = $hits;
}
else //Si elle est def alors on récupère seulement la valeur du fichier pour 
      //que se soit à jour 
{
  $fp=fopen("compteur.txt","a+"); //OUVRE LE FICHIER compteur.txt
  $num=fgets($fp,4096); // RECUPERE LE CONTENUE DU COMPTEUR
  $hits=$num;
}

18/08/2009, 09h55
sabotage

J'ai vérifié mais le code tel que je l'ai ecris (avec la correction) fonctionne : le rafraichissement de la page n'incremente pas la valeur dans compteur.txt

Au passage, pourquoi tu fais - - 1 au lieu de +1 ?

Oui, mais chez moi il me retourne une erreur lors de l'affichage de la variables $hits. c'est pour palier à cela que qu'il y a le else ... :oops: Voici le code cela va expliquer mon raisonnement et tu me le confirmera .
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 session_start(); // DEMARRE LA SESSION // TRAITEMENT SUR LE FICHIER TEXTE if(isset($_SESSION['hits']) == FALSE){ $fp=fopen("compteur.txt","a+"); //OUVRE LE FICHIER compteur.txt $num=fgets($fp,4096); // RECUPERE LE CONTENUE DU COMPTEUR fclose($fp); // FERME LE FICHIER $hits=$num + 1; // TRAITEMENT $fp=fopen("compteur.txt","w"); // OUVRE DE NOUVEAU LE FICHIER fputs($fp,$hits); // MET LA NOUVELLE VALEUR fclose($fp); // FERME LE FICHIER $_SESSION['hits'] = $hits; } else { $fp=fopen("compteur.txt","a+"); //OUVRE LE FICHIER compteur.txt $num=fgets($fp,4096); // RECUPERE LE CONTENUE DU COMPTEUR $hits=$num; } // AFICHAGE DU COMPTEUR echo "<TABLE>"; echo "<TR>"; echo "<TD STYLE='1px border:Solid navy;' >"; echo "<FONT FACE='Verdana, Arial, Helvetica, sans-serif' SIZE=1 color=#A0A0A0>"; echo "nombre de visites : $hits"; // VARIABLE DU COMPTEUR echo "</FONT>"; echo "</TD>"; echo "</TR>"; echo "</TABLE>"; // IMPORTANT // //ne pas oublier de créer le fichier compteur.txt ?>
Voila comme sa si sa intéresse quelqu'un .... tout y es .. bien sur après validation de sabotage.

Citation:

Au passage, pourquoi tu fais - - 1 au lieu de +1 ?

C'est une bonne question, j'avais récupéré se code pour ne pas perdre de temps car le site doit avoir une version béta le plus rapidement possible... Mais en fait sa ne ma pas arrangé, sa me servira de leçon et puis sa ma permis de me remettre dedans lol

Merci en tout cas