COMPTEUR DE VISITE - FICHIER TXT - CODE MINIMAL

Commentaire de LeFauve42 le 25/07/2011 13:32:26

Oui, c'est minimaliste, mais je pense que tu as des problemes de re-entrence lorsque plusieurs visiteurs sont sur ta page en meme temps (c'est a dire a quelques ms pres).
Il faudrait utiliser flock() sur ton fichier (et complexifier un peu puisque du coup tu perds file_put_contents()).

Commentaire de LeFauve42 le 26/07/2011 13:14:54

Bien vu !
Je ne connaissais pas cette nouveaute.

Auquel cas, ca reste tres simple.

Par contre, tu devrais donner quelques details sur la mise en oeuvre (comme "penser a changer les droits du fichier .txt pour que le process du serveur HTTP puisse ecrire dedans) et mettre a jour le source avec le LOCK_EX.

Eric

Commentaire de SuperChouquette le 26/07/2011 15:25:57

ce n'est pas tiré du sdz, montre moi la page ou tu as vu ça.. et on t'as dit la solution il suffit de mettre le locker si tu as un site extrêmement fréquenté..
Après ce n'est pas parce que c'est simple que c'est sans intérêt.. Il y'a pleins de codes de compteurs ultra compliqués sur ce site qui ont nettement moins d'intérêts à mes yeux.

Commentaire de LeFauve42 le 26/07/2011 15:52:47

C'est effectivement un bon exemple, et tres interessant (tout le monde n'a pas envie d'un framework de 2000 lignes de PHP pour faire des petits trucs simples).
"La perfection n'est pas atteinte quand on ne peut plus rien ajouter, mais quand on ne peut plus rien enlever" :o)

J'aurai mis la note max si tu avais mis ca dans une fonction et donne quelques explications sur le deploiement (comme choisir un repertoire dans lequel le serveur peut ecrire (et pas juste donner les droits sur le fichier comme je disais plus haut, car comme le fichier va etre cree par le premier appel du script, les droits seront forcement bons pour celui-ci)) mais FWIW je ne met pas souvent d'aussi bonnes notes ;o)

Commentaire de Romano58 le 26/07/2011 16:47:58

J'en arrive à lire tellement de connerie sur ce site que je viens d'avoir une idée de génie pour un script de folie (et qui prend pas beaucoup de ligne).
Puisqu'on ce fiche de savoir si on en est à centaines de visites en plus ou en moins, vous devriez développer une fonction qui renvoie un nombre aléatoire, dans le genre "Aujourd'hui, vous êtes le 1 000 968 em visiteur sur le site, et XXXXX personnes sont mortes dans le monde depuis hier".

C'est vrai, après tout, puisque on ce fiche de la pertinence des informations à afficher dans la page.

Ou alors proposer un VRAI compteur, c'est pas beaucoup plus compliquer de rajouter les ip qui viennent sur la page dans un fichiers texte, et d'ajouter + 1 au compteur si l'ip n'est pas présente. Au moins, ça changera de l'exemple basique que tu viens mettre ici (et qu'on trouve partout sur le net soit dit en passant), et ça méritera une vrai note.

Commentaire de LeFauve42 le 27/07/2011 12:59:07

> Sinon, créer un cookie sur le Navigateur et n'incrémenter que si le cookie n'est pas présent.

C'est ce que je fais : Ca permet d'avoir un compteur genre "Cette page a ete vue xxx fois par yyy personnes differentes"...

Commentaire de cod57 le 28/07/2011 16:34:53

bonjour
d'accord avec toi phpAnonyme un simple javascript de g..... anali..
en fait tu peux gagner une ligne SuperChouquette


<?php
! file_exists('compteur.txt') ? fopen('compteur.txt','w+') : file_put_contents('compteur.txt', $compteur=file_get_contents('compteur.txt')+1, LOCK_EX);
if(isset($compteur)){echo $compteur;}
?>

Commentaire de LeFauve42 le 29/07/2011 13:24:02

phpAnonyme: Rien n'est jamais TROP minimaliste : Si tu peux obtenir le meme resultat de 2 manieres, la plus courte est toujours la meilleure (par meme resultat, j'entend "qui fonctionne dans tous les cas").

De plus, il y a beaucoup d'exemples de 2 ou 3 lignes qui sont des horreurs (offrant au choix des trous de securite ou une utilisation de ressources sans communes mesures avec la fonction remplie) et ce n'est pas evident pour un debuttant de faire le tri.

En mettant une bonne note a ces 3 lignes-ci plutot qu'a d'autres, ca permet de aussi d'indiquer qu'il s'agit d'un bout de code qui fait correctement ce qu'on attend de lui sans mauvaise surprise.

Alors d'accord ce n'est pas de la "rocket-science" mais il faut bien commencer un jour :o)

Commentaire de cod57 le 29/07/2011 14:11:18

bonjour
@LeFauve42++

Commentaire de Romano58 le 29/07/2011 14:49:55

Selon un admin, mes remarques ne sont pas constructives. J'attend que le webmaster veuille bien supprimer mon compte (pas trouver l'option adequate dans le menu..). Ce que je disais sur une autre source la semaine dernière, à force de sur-noter des codes "bidons", de "plussoyer" les erreurs de programmation et d'aimer les "gros code moche", on ce retrouve avec des gars qui détectent des pixels mort de façon logiciels..

Mais ce n'est pas très étonnant, étant donné le peux de personne ayant les pieds sur terre ici.

Commentaire de pierreSabatier le 11/09/2011 18:24:49

Bonjour tout le monde, au lieu de maugréer comme vous le faites, j'ai écrit un petit script qui, à la lecture de vos commentaires peu élogieux, vous ravira ou le cas échéant, relancera la discussion sur cette fonctionnalité.