Compteur de hits et de visites uniques

**Olivier Regnier** · 25/01/2008, 16h21

Bonjour

Je viens de terminer deux scripts.

Le contenu de la page shtml:

Code :

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

1
2
3
4
5
6
7
8
<html>
<head>
</head>
<body>
<!--#include virtual="/cgi-bin/hits.cgi"-->
<!--#include virtual="/cgi-bin/visites.cgi"-->
</body>
</html>

Bien entendu, il faut que le serveur d'hébergement accepte les SSI.
Si vous avez des avis ou bien des commentaires ou encore des solutions pour optimiser au maximum le code, je suis preneur.

Une petite correction s'impose, merci d'avance aux experts

**Woufeil** · 26/01/2008, 14h27

Salut Olivier,

Je précise que pour l'instant je n'ai vu que le premier fichier et que les scripts CGI ne sont pas franchement ma tasse de thé

Je n'aime pas franchement ta façon de gérer les fichiers... Tu utilises des handles que tu ne fermes jamais, et en plus ces handles ne sont pas en manjuscuke comme le veut la tradition.

Je te conseille d'utiliser des handles lexicaux, comme expliqué ici.

Sinon, toujours à propos des fichiers, je ne comprend pas bien ceci :

Code :

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

1
2
open fichier,'>',$hits;
open fichier,'>>',$hits;

Tu ouvres deux fois le même fichier dans le même handle ? Il y a une ligne de trop je pense

A part ça, tu utilises des variables globales, mais c'est ton choix... Personnellement je ne suis pas fan, mais on ne peut pas te le reprocher.

Voilà pour le premier fichier. Quand j'aurais un peut de temps, je lirais le second

**Olivier Regnier** · 27/01/2008, 08h08

Salut Woufeil

Envoyé par Woufeil

Je n'aime pas franchement ta façon de gérer les fichiers... Tu utilises des handles que tu ne fermes jamais, et en plus ces handles ne sont pas en manjuscuke comme le veut la tradition.

Je te conseille d'utiliser des handles lexicaux, comme expliqué ici.

J'en prends bonne note. D'ailleurs, les deux scripts sont corrigés

Envoyé par Woufeil

Sinon, toujours à propos des fichiers, je ne comprend pas bien ceci :
open fichier,'>',$hits;
open fichier,'>>',$hits;

au départ, j'utilisais la première ligne pour créer le fichier et la deuxième pour ensuite écrire dedans

la première ligne a été supprimé.

Envoyé par Woufeil

Voilà pour le premier fichier. Quand j'aurais un peut de temps, je lirais le second

C'est gentil, mais j'ai pris les devants