Discussion :

[laurentSc]

Bonsoir,

je code un compteur de visites qui doit notamment afficher le nombre de visites quotidiennes et le nombre de visites annuelles. Pour ce faire, j'ai une table "visites" avec les champs suivants :

Code :

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

1
2
3
4
 
ip 	        varchar(16) 	 	 
timestamp 	int(10)  	 	 
counter_year 	int(10)

donc à chaque visite d'un nouvel IP ou d'un IP déjà présent mais dont le timestamp est supérieur d'au moins 2 minutes du précédent, nouvelle visite. A chaque nouvelle visite, j'incrémente le champ counter_year et un des enregistrements est 0,0,counter_year. A chaque minuit, j'efface tous les enregistrements sauf celui-là. Donc le nombre de visites quotidiennes est le nombre d'enregistrements -1 et le nombre de visites annuelles est indiqué dans chaque enregistrement.
Ce qui coince, c'est la requête pour effacer tous les enregistrements sauf celui dont l'IP est nul (afin de garder counter_year). Voici le code :

Code php :

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

1
2
3
$req='DELETE FROM visites WHERE ((timestamp < ' . mktime(0,0,0).') AND (ip IS NOT NULL))';
echo "req=".$req."<br/>";
mysqli_query($db, $req;);

L'echo n'est affiché que si je commente le mysqli_query donc l'exécution de cette requête pose problème et pourtant si je la teste avec phpMyAdmin, elle fonctionne, donc je ne comprends pas ; quelqu'un comprend ?

[Spartacusply]

Ben si pour moi elle devrait marcher, après le mieux c'est de faire un vrai débuggage (d'autant que tu utilises mysqli ce qui facilite grandement la tache) :

Code :

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

1
2
3
if (!$mysqli_query($db, $req;)) {
    die('Erreur : ' . mysqli_connect_error());
}

Une remarque toutefois, le test du timestamp est inutile, et au pire, la fonction mktime peut être remplacé par la fonction time().

Une deuxième remarque toutefois, ta table me paraît mal construite, pourquoi faire un champ "counnter_year" alors que tu peut tout à fait retrouver cette information en faisant un SUM() sur l'IP ?

[laurentSc]

Merci pour ta réponse à laquelle je réponds point par point :

1- si je mets tes lignes pour le débogage (avec PDO, j'aurais pas pu ?), non seulement, ça n'affiche pas l'erreur, mais page blanche...

2- inutile le test de timestamp Ca me sert à faire un compteur quotidien...

3- le SUM() sur l'IP me donne le nombre de visites quotidiennes mais pas le nombre de visites annuelles vu la RAZ à minuit...Pour compter le nombre de visites quotidiennes, je fais :

Code php :

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

1
2
3
$res = mysqli_query($db, 'SELECT COUNT(*) AS compteur FROM visites');
$data = mysqli_fetch_array($res);
$compteur_j = $data['compteur']-1;

[Bovino]

Code :

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

mysqli_query($db, $req;);

Qu'est-ce qu'il vient faire là le ; dans les paramètres ? Et si ça ne t'affiche rien, c'est qu'il est grand temps pour toi d'activer l'affichage des erreurs !

[Spartacusply]

Si tu aurais pu débugguer encore plus facilement avec PDO même, mais pas avec "mysql", que l'on voit encore bien trop souvent malgré son obsolescence.

Sinon au temps pour moi, je ne t'ai pas conseillé la bonne fonction ^^

Code :

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

1
2
3
4
if (!mysqli_query($db, $req)) {
    printf("Message d'erreur : %s\n", mysqli_error($db));
     exit;
}

Ta requête, sauf grossière erreur de ma part (ce qui est possible également), supprime toutes les lignes avec un timestamp inférieure à... l'heure actuelle. Or, il me semble qu'il ne peut y avoir de timestamp supérieure à l'heure actuelle. Donc ton delete vire tout. D'où le test sur le timestamp est inutile.

Pour la construction de table, je ne comprends pas pourquoi tu la vide tous les jours ? Pourquoi ne pas garder juste l'ip et le timestamp et faire la somme pour avoir le total annuel ? Question de performance ? Si tel est le cas, sache que normalement ce n'est pas un problème, avec des petits ajustement, ça devrait passer sans problème.
Pour une solution intermédiaire, ce qui me paraît envisageable c'est de faire un truc du genre, ip | day (champ DATE) | number. Comme cela tu n'auras au maximum qu'une ligne par jour et par visiteur.

Dans tous les cas, si tu souhaites garder ce compteur annuel, il me paraît nécessaire de le mettre ailleurs (dans une autre table par exemple, même s'il n'y a qu'une seule information !), parce que là elle se mélange avec les autres alors que ce n'est par le même type d'information.

[laurentSc]

Le ; dans la parenthèse, c'est visiblement une faute d'inattention de Spartacusply et j'avais fait un copier-coller sans la remarquer...L'affichage des erreurs, il est déjà là :

Code php :

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

1
2
3
<?php
error_reporting(E_ALL);
ini_set('display_errors','1');

mysql, ça fait des années que je l'ai abandonné...

J'ai changé l'affichage des erreurs avec mysqli mais aucune différence :

Notice: Undefined variable: mysqli_query in /var/www/vhosts/viradeduvercors.org/httpdocs/html5/compteur/visite.php on line 13 Fatal error: Function name must be a string in /var/www/vhosts/viradeduvercors.org/httpdocs/html5/compteur/visite.php on line 13

sauf que ce message m'a fait comprendre l'erreur : il y avait un $ en trop, et de plus, le point-virgule, il venait de mon code (je viens seulement de le voir (fatigué hier soir))

Je viens d'imaginer une solution avec une seule table : au lieu de faire une RAZ tous les jours, je ne vais la faire qu'une fois par an, et pour le compteur quotidien, je vais compter les enregistrements dont le timestamp date du jour en cours...

[Spartacusply]

Je viens d'imaginer une solution avec une seule table : au lieu de faire une RAZ tous les jours, je ne vais la faire qu'une fois par an, et pour le compteur quotidien, je vais compter les enregistrements dont le timestamp date du jour en cours...

Tout à fait, c'est la meilleure solution à envisager. Par contre, prévois plutôt une année glissante (ce qui signifie : du jour d'aujourd'hui au même jour de l'année précédente) plutôt qu'une année calendaire, ça paraît plus cohérent. Ce qui signifie que tu as juste à supprimer une fois par jour les enregistrements qui sont plus vieux d'une année.

[laurentSc]

Tout-à-fait, je vais faire ça...