Discussion :

[zebulon94]

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 :

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

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 :

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

[sabotage]

D'ou vient $hits ?

[RideKick]

Salut ,
tu as essayé de suivre ce que te demande le message ?

[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.

[sabotage]

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 :

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

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']
}

[zebulon94]

ok juste petite question, la fonctions isset() permet de savoir si la variable est définie.
Dans notre exemple :

Code :

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

1
2
if(isset($_SESSION['hits'])){
}

Si l'utilisateur entreprend sa première connexion,

Code :

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

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

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

Merci de vos réponse
D.

[sabotage]

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

Code :

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

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

[zebulon94]

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 :

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
// 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;
}

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

[zebulon94]

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 ... Voici le code cela va expliquer mon raisonnement et tu me le confirmera .

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

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