Probleme pour un compteur

Version imprimable

salut à tous,
débutant en php, je voulais juste faire un compteur (simple) qui comptabilise les visites. J'ai ce code :
Code:

1 2 3 4 5 6 7 8 9 <? $monfichier = fopen('compteur.txt', 'r+'); $pages_vues = fgets($monfichier); // On lit la première ligne (nombre de pages vues) $pages_vues++; // On augmente de 1 ce nombre de pages vues fseek($monfichier, 0); // On remet le curseur au début du fichier fputs($monfichier, $pages_vues); // On écrit le nouveau nombre de pages vues fclose($monfichier); echo '<p>Il y a eu ' . $pages_vues . ' visites !</p>'; ?>
Et dans mon fichier index j'ai :

Code:

<? include("compteur.php");

Or le nombre reste toujours a 0 et ne bouge pas. Ou est donc l'erreur dans ce code hyper simple pour vous ?
Merci d'avance

23/08/2007, 14h45
Raideman

Je sais pas, la syntaxe a l'air correcte.
Vérifie que tu as bien un 0 dans ton fichier compteur (si ton fichier n'a pas de contenu la premiere fois que tu lances le compteur, ca marchera pas).
Vérifie que tu as bien les droits sur le fichier compteur.txt sur ton serveur pour lire et écrire dedans.

AUssi, essaie de mettre plus de controle dans ton code (genre sur le fopen par exemple). ca permet d'avoir un retour d'erreur.

Ceci dit, a part ca, je pense que le code que tu soumets devrait fonctionner.
23/08/2007, 14h49
trotters213

:salut:
Evite également d'utiliser les short tags (<?), utilise la syntaxe complète <?php
23/08/2007, 14h52
darkwhite

Moi aussi sa m'etonne car j'ai fait :

un chmod -R 777 /var/www (donc la je devrais etre tranquille non ?)
j'ai mis la valeur 0 dans le fichier texte.

A part ça sa devrait marcher non ... et bien non
23/08/2007, 15h00
trotters213

:salut:
Chez moi ça fonctionne très bien.

Tu mets bien un 0 dans ton fichier compteur.txt ?
Tes 2 fichiers index.php et compteur.php sont bien dans le même répertoire ?
Tu affiches toutes les erreurs : error_reporting(E_ALL|E_STRICT); ?
23/08/2007, 16h11
darkwhite

:yaisse2: Salut
-J'ai bien mis 0 dans le fichier compteur.txt (si je suis pas complètement taré zéro c'est ça 0 ).
-Mes deux fichiers sont bien au même endroit
Je comprend pas la dernière phrase. C'est quelque chose que je devrais faire ou que j'ai fait ? enfin bref sa sert a quoi cette fonction et ou la mais on en place
23/08/2007, 16h36
trotters213

Citation:

Envoyé par darkwhite

Je comprend pas la dernière phrase. C'est quelque chose que je devrais faire ou que j'ai fait ? enfin bref sa sert a quoi cette fonction et ou la mais on en place

C'est une fonction que tu dois mettre au tout début de ton script afin d'être sur que tu affiches toutes les erreurs (bien évidemment le jour où tu mettras ton site en ligne, tu la supprimes).
Elle te permet d'afficher les erreurs, les warnings et les notice sans avoir à modifier ton fichier php.ini.

Rajoutes là en haut de ton script, réexécute ton script de comptage et dis nous si quelquechose apparait à l'écran.

Donc si j'ai bien compris :
Maintenant mon compteur php est :
Code:

1 2 3 4 5 6 7 8 9 10 <? error_reporting(E_ALL|E_STRICT); $monfichier = fopen('compteur.txt', 'r+'); $pages_vues = fgets($monfichier); // On lit la première ligne (nombre de pages vues) $pages_vues++; // On augmente de 1 ce nombre de pages vues fseek($monfichier, 0); // On remet le curseur au début du fichier fputs($monfichier, $pages_vues); // On écrit le nouveau nombre de pages vues fclose($monfichier); echo '<p>Il y a eu ' . $pages_vues . ' visites !</p>'; ?>
Et il apparait quelque chose a l'écran, oui, mais je doute que sa soit sur le script lui même :

Citation:

Notice: Use of undefined constant E_STRICT - assumed 'E_STRICT' in /var/www/compteur.php on line 2

Il y a eu 0 visites !

23/08/2007, 16h54
trotters213

OK, dans ce cas supprime le E_STRICT et laisse simplement error_reporting(E_ALL); (tu utilises une version antérieure à PHP 5 je suppose).
23/08/2007, 17h36
darkwhite

Quand j'enlève E_STRICT, on revient au début :
le nombre n'évolue pas quand on actualise la page.

Essaie ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
<?php
if (!file_exists('compteur.txt')) 
{
  $file=fopen('compteur.txt', 'w+');
  fwrite($file, '1');
}
else 
{
  $file=fopen('compteur.txt', 'r+');
  $cpt = trim(fgets($file));
  rewind($file);
  fwrite($file, $cpt+1);
}
fclose($file);
echo 'Il y a eu '.($cpt+1).' visites !';
?>

Si le fichier n'est pas créé, il le crée et passe le compteur à 1 sinon il le lit et l'incrémente.

23/08/2007, 21h50
darkwhite

Avec ton code sa marche
Merci beaucoup de ton aide patiente.
Maintenant pourquoi le tiens marche et pas le mien ta compris ? :lol:
23/08/2007, 21h56
trotters213

Bè je sais pas vraiment si c'est le $pages_vues++ ou le fputs qui foirait.
C'est vrai que c'est bizarre car chez moi les 2 versions marche.
Je pencherais plutôt pour le fputs qui est un alias de fwrite donc il vaut mieux utiliser fwrite.
23/08/2007, 22h00
darkwhite

En tout cas merci beaucoup.