Discussion :

[darkwhite]

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

Code :

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

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 :

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

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

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

[trotters213]

Evite également d'utiliser les short tags (<?), utilise la syntaxe complète <?php

[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

[trotters213]

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

[darkwhite]

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

[trotters213]

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.

[darkwhite]

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

Code :

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

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 :

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

Il y a eu 0 visites !

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

[darkwhite]

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

[trotters213]

Essaie ceci :

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

[darkwhite]

Avec ton code sa marche
Merci beaucoup de ton aide patiente.
Maintenant pourquoi le tiens marche et pas le mien ta compris ?

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

[darkwhite]

En tout cas merci beaucoup.