QUI VISITE QUOI?

Commentaire de kiwi1nut le 07/06/2006 21:28:39

Voila je suis ouvert a tous commentaires et à tous vos conseils et suggestions ;-)

Commentaire de massacr le 08/06/2006 11:59:07

Désolé, mais étant de mauvaise humeur aujourd'hui, je suis pret pour te critiquer. Je vais néanmoins essayer d'être constructif.

Déjà, beaucoup de codes existent pour faire la même chose, une petite recherche s'imposait avant la publication.
Le code est très sale. Tu mélange HTML et PHP, tu duplique tes variables, tu appelle tes variables de serveur directement, au lieu de passer pas $_SERVER, etc etc etc.
Dans ip_log et ip_ban, tu devrais utiliser tes fonctions pour ouvrir la connexion sql, récupérer les données du serveur, etc, car beaucoup de lignes sont dupliquées.
Bref, lis des tutos sur le php, apprends à créer des fonctions et reviens corriger ce code. Propre, il servira peut etre à quelqu'un.
A+

Commentaire de kiwi1nut le 08/06/2006 22:17:27

salut massacr, effectivement tu portes bien ton pseudo, lol!
bref avant de publier ma source crois mois j'avais cherché et n'avasi trouvé que des scripts qde compteurs de visites, le mien servira a gérer les ip et de bannir les boulets. Je suis entrain de bosser sur un tableau de stat en complément...
Pour mon code qui est sale et mélengé je ne demande pas mieux d'apprendre car cela fait trois mois que je code en php et je ne maîtrise que peu de choses.
Pour les fonctions je m'y met et je remettrais la modif.
A bientôt

Commentaire de kiwi1nut le 08/06/2006 23:49:43

Oui c'est exact mais j'ai fait ce script en rapport avec mon site car j'ai pas mal de visiteurs et quelques boulets...
Je voulais savoir quelle ip visitait quelles pages. C'est pourquoi le tableau affiche ton ip plusieurs fois. Je travaille sur le fait que le tableau affiche une ip une fois mais avec toutes les pages visitées.

Commentaire de malalam le 09/06/2006 00:07:38 administrateur CS

Hello,

honnêtement, j'avais pas maté le code (j'ai pas le temps de tous les mater à mon grand dam).
Là, j'ai jeté un oeil.
Ben...c'est vrai que c'est TRES mal codé. Quant à t'expliquer ce qui ne va pas, ça prendrait trop de temps.
Je te suggère de regarder quelques codes dont les commentaires sont bons, et de comparer ta façon de coder avec celle de leurs auteurs.
Si j'installe ton code sur mon serveur, c'est simple : il ne tournera même pas. J'aurai des dizaines de lignes d'erreurs.
Même pas d'ailleurs puisqu'il plantera d'entrée, au : <?
Bref...please, corrige au moins la qualité du code. Parce que des compteurs, on en a des tonnes sur phpcs. Après, on avisera.
Fais un tour sur php.net, tu verras comment ils codent, eux.
En vrac, tu utilises des superglobales $HTTP_USER_AGENT par exemple, alors qu'il daudrait utiliser $_SERVER['HTTP_USER_AGENT'].
Les echo ("..."); c'est un peu vieillot, on se croirait revenu au temps de PHP3...
Bref...corrige.

Commentaire de malalam le 09/06/2006 08:13:55 administrateur CS

Bah au contraire, on est sympas avec les débutants... ;-)
- pour le débutant qui crée un code, on le pousse à s'améliorer
- pour les débutants qui regardent un code, on leur montre ce qui n'est pas bon à prendre dans ce code, ce qu'il faut y apporter.

Nous sommes "beginners friendly" :-)

Commentaire de puppetmastr le 13/06/2006 15:54:57

ca a l'air pas mal mais je n'arrive pas a le faire fonctionner :/

il faut modifier le config.php

Pour ceux qui utilisent EasyPhp que faut-il mettre pour les 2 derniers parametres ?
$host= "localhost"; //Hote
$user= "root"; //Nom de l'utilisateur
$pass= ""; //Mot de passe  pour acceder a la base de donnees
$base= ""; //Nom de la base de donnees

merci