COMPTEUR DE VISITES

cs_metis15 Messages postés 311 Date d'inscription lundi 19 mai 2003 Statut Membre Dernière intervention 26 juin 2015 - 24 mai 2005 à 16:55
coucou747 Messages postés 12303 Date d'inscription mardi 10 février 2004 Statut Membre Dernière intervention 30 juillet 2012 - 31 mai 2005 à 18:32

Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/31560-compteur-de-visites

cs_metis15 Messages postés 311 Date d'inscription lundi 19 mai 2003 Statut Membre Dernière intervention 26 juin 2015
24 mai 2005 à 16:55

Tel quel, c'est inutilisable. J'y connais pas grand chose en codage, mais <?php manque, bon, c'est pas tout.
Où tu mets les 0.gif, 1.gif, etc...
Le script, tu le mets où ?
Bon, pour être didactique, il faut savoir qu'on s'adresse à des gens qui viennent ici pour trouver avec un max de détails.
Je pense que tu peux améliorer la présentation !
Merci pour tous les non codeurs (;o))) !

massacr Messages postés 233 Date d'inscription vendredi 2 juillet 2004 Statut Membre Dernière intervention 4 janvier 2007
24 mai 2005 à 17:58

Bah, Codes Sources, ce n'est pas un tutoriel, c'est un endroit pour développeurs. Donc, je pense qu'un développeur normalement constitué peut se rendre compte qu'il faut rajouter les balises <?php et ?>. Surtout qu'il a mis son code en "initié". On voit très bien qu'il faut mettre les images dans le dossier Images/, et on met le script sur la page où on veut le compteur !!!

Bon, c'est un bon code, simple. Il est vrai pas très recherché, mais si il marche... 6/20

Sadnotrue Messages postés 42 Date d'inscription jeudi 4 juillet 2002 Statut Membre Dernière intervention 31 mai 2005
24 mai 2005 à 18:45

Effectivement j'ai oublié de préciser ces points importants :
Ce code est plus optimisé pour les sites ayant une page de présentation.
Il manque comme la précisé Metis15 les balises <? ?> au début et à la fin du code.
Pour les images, il faut juste les placer dans un dossier Images à la racine du site.
Merci à Massacr qui a bien redéfini tout ça ;)
Pour le code, j'ai essayé de le faire le plus simple possible.
Désolé pour cette présentation très médiocre ;p Je code pas mal en php et c'est le premier code que je met en ligne. J'ai pas encore l'habitude de jouer les profs. Désolé encore, je m'appliquerais la prochaine fois :)

malalam Messages postés 10839 Date d'inscription lundi 24 février 2003 Statut Membre Dernière intervention 2 mars 2010 25
24 mai 2005 à 21:25

Hello,

je ne vais pas noter, mais je suis en grande partie d'accord avec Metis15. Il faut s'appliquer autant que faire se peut ici...justement parce que codes sources est quand meme tres frequentes par des debutants cherchant a s'ameliorer. Alors autant faire du mieux possible quand on poste une source. Parce qu'elle va peut-etre etre un exemple pour quelqu'un...

=> sadnotrue : puisque tu programmes en php depuis un bail, je trouve qu'il manque pas mal de choses dans ton code :
- tu devrais t'appliquer aussi pour le html, et eviter les balises depricees. Et utiliser un peu plus de css, meme si ok, c'est tout petit comme code..mais bon.
- tu ouvres un hypothetique fichier avec r+...quid si le fichier n'existe pas ? r+ ne le cree pas...il serait bon de tester son existence avant.
- fgets ($fp, 11) ? Tu es optimiste, lol, mais pourquoi pas.
- un while, suivi d'un for, et enfin un if la-dedans, tout ca pour afficher au final la bonne image, ca fait quand meme beaucoup! Tu devrais reflechir a optimiser tout ca un peu. Ce n'est pas tres complique d'ailleurs. Hint : tu recuperes une chaine de caractere, dans ton fichier...on accede assez facilement a chaque caractere d'une chaine...et chaque caractere de ta chaine correspond a un gif...tu es certain d'avoir besoin d'une boucle imbriquee pour parvenir a tes fins ?
- pour les gif, meme histoire que pour le fichier du compteur : un petit test d'existence serait pas mal.
- enfin, une fonction serait nettement plus adaptee, ne serait ce que parce que ca te permettrait de l'agrementer un peu de quelques possibilites supplementaires : nom du fichier compteur par exemple, si on en veut un par page... Puis c'est plus simple a appeler de toutes manieres.
- d'aucun raleront parce que tu ne prends pas en compte les IP, mais ca, perso, je m'en fous, c'est un choix ;-)

cs_metis15 Messages postés 311 Date d'inscription lundi 19 mai 2003 Statut Membre Dernière intervention 26 juin 2015
24 mai 2005 à 22:01

Hé oui, ce site n'est pas fréquenté que par des codeurs qui veulent rester entre eux. Je ne pense pas que ce soit l'idée directrice voulue par les créateurs.

Alors je fais ce qui semble logique, je mets le code entre les bonne balise php, des images 0.gif, 1.gif, etc. dans un dossier à la racine et ça dit...

Warning: fopen(): Unable to access compteur.dat in /var/www/free.fr/d/5/mapie15/__test-compteur.php on line 12

Warning: fopen(compteur.dat): failed to open stream: No such file or directory in /var/www/free.fr/d/5/mapie15/__test-compteur.php on line 12

Warning: fgets(): supplied argument is not a valid stream resource in /var/www/free.fr/d/5/mapie15/__test-compteur.php on line 13

Warning: fseek(): supplied argument is not a valid stream resource in /var/www/free.fr/d/5/mapie15/__test-compteur.php on line 15

Warning: fputs(): supplied argument is not a valid stream resource in /var/www/free.fr/d/5/mapie15/__test-compteur.php on line 16

Warning: fclose(): supplied argument is not a valid stream resource in /var/www/free.fr/d/5/mapie15/__test-compteur.php on line 17

Bon... En gros, ça marche pas chez moi ! :)