Discussion :

[laurentSc]

Bonsoir,

j'ai implémenté un compteur de visites quotidiens qui compte mais pas à chaque fois ; je ne comprends pas pourquoi, donc j'espère une aide.

Code du compteur :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
$mois=date("m");
$jour=date("d");
$annee="20".date("y");
 
$datetoday="20".$annee.$mois.$jour;
 
$sql="select date from visites"; /* la date est dans MySQL pour remettre à 0 si elle a changé ou sinon incrémenter le compteur */
$result = mysql_query($sql)or die(mysql_error());
 
while($answer = mysql_fetch_array($result, MYSQL_ASSOC)) {
    $date = $answer['date'];
    $format="%d-%d-%d %d:%d:%d";
    $nb=sscanf($date,$format,$anneesql,$moissql,$joursql,$min,$sec,$hour)
    } 
// Je fais un while, mais il n'y a qu'une seule ligne
 
if (($annee != $anneesql)||($mois != $moissql)||($jour != $joursql))  {
 
    $sql="update visites set nb_connectes_jour=1"; /* on initialise à 1, car on est le 1er utilisateur */
    $result = mysql_query($sql)or die(mysql_error());
 
    $sql="update visites set date=".$datetoday;
    $result = mysql_query($sql)or die(mysql_error());
}  
 
else {	
    $sid = session_id();
    if (!isset($_SESSION['id'])) $_SESSION['id']=0;
    if ($_SESSION['id']!=$sid) { /* on n'incrémente que si la session a changé, pas si on a juste changé de page sans quitter le site */
        $_SESSION['id']=$sid; 
 
       $sql="UPDATE visites SET nb_connectes_jour = nb_connectes_jour +1;";
       $result = mysql_query($sql)or die(mysql_error());
     }
 
}

[Hoder]

Bonjour,

Je suis pas vraiment sûr mais, quand vous faites :

Code :

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

1
2
3
4
5
6
 
$mois=date("m");
$jour=date("d");
$annee="20".date("y");
 
$datetoday="20".$annee.$mois.$jour;

Vous avez quelque chose comme $datetoday = "20101205"
Or, le sscanf lit selon le format :

Code :

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

$format="%d-%d-%d %d:%d:%d";

Et attend donc une entrée comme : "10-12-05 min:sec:hour" (avec $min, $sec et $hour qui ne doivent pas exister).
Je n'ai jamais utilisé sscanf mais d'après ce que j'ai compris de la documentation, ça récupére les informations selon le format qu'on indique et ici en fait, le format de la valeur enregistrée dans la base SQL ne semble pas correspondre au format demandé par sscanf.

En espèrant peut-être vous avoir mis sur une piste, je vous souhaite une bonne journée.

[laurentSc]

Merci de la réponse, mais ce n'est pas ça, car si je passe à MySQL une donnée au format "20101205" (c'est le format datetime de SQL), il stocke une donnée au format "2010-12-05 00:00:00", d'où le $format (et d'ailleurs, si ce que je faisais là était faux, ça ne marcherait jamais, alors que là, en gros une fois sur deux (ou un peu plus), ça marche...)

[Herode]

Où est placé le session_start() ?

[Hoder]

En testant votre code, j'ai trouvé deux erreurs (mais je pense que l'une des deux est simplement une erreur de copier/coller)
La première étant qu'il manque un ';' à la fin de la dernière instruction de la boucle.
Et la seconde surtout :

Code :

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

1
2
3
4
 
$annee="20".date("y");
 
$datetoday="20".$annee.$mois.$jour;

Il y a en fait 2 fois l'ajout du "20" ce qui donne par exemple "2020101205". Le format n'étant alors plus reconnu par MySQL, la valeur n'est pas enregistrée. Mais alors comment ça se fait que ça marche de temps en temps, aucune idée
J'espère que cette fois-ci sera la bonne

[laurentSc]

Merci de la réponse, Hoder ; j'ai enlevé l'un des deux 20 ; je crois que ça améliore les choses (pas sûr), mais ce n'est pas encore du 100% et de plus, je ne comprends pas comment ça pouvait parfois marcher avec une telle erreur...

Où est placé le session_start() ?

Au tout début du fichier (d'ailleurs, s'il était mal placé, ça ne marcherait jamais)

[Hoder]

En testant avec ou sans session_regenerate_id(true), j'obtiens de bons résultats, donc je ne vois pas trop ce que je pourrai vous conseiller d'autre.
Cependant, vous devriez placer ces quelques lignes :

Code :

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

1
2
3
    $sid = session_id();
    if (!isset($_SESSION['id'])) $_SESSION['id'] = 0;
         $_SESSION['id'] = $sid;

Dans ce contexte :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
if (($annee != $anneesql)||($mois != $moissql)||($jour != $joursql))  {
 
    $sql="update visites set nb_connectes_jour=1"; /* on initialise à 1, car on est le 1er utilisateur */
    $result = mysql_query($sql)or die(mysql_error());
 
    $sql="update visites set date=".$datetoday;
    $result = mysql_query($sql)or die(mysql_error());
    $sid = session_id();
    if (!isset($_SESSION['id'])) $_SESSION['id'] = 0;
         $_SESSION['id'] = $sid;
}

Ainsi, vous évitez de recompter le premier utilisateur une seconde fois.
En effet, après l'enregistrement dans la base de donnée MySQL, ce premier utilisateur, lorsqu'il retombera sur ce script, ira dans la seconde partie de la condition 'else' et son $_SESSION['id'] n'ayant pas été enregistré, il sera de nouveau comptabilisé.
Ou après réflexion, vous pouvez aussi juste mettre :

Code :

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

$sql="update visites set nb_connectes_jour=0";

Au lieu de :

Code :

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

$sql="update visites set nb_connectes_jour=1";

Désolé de ne pouvoir vous apporter plus d'aide.
Bonne journée.

[gene69]

Code :

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

$annee="20".date("y");

tssss

Code :

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

$annee=date("Y");

Code :

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

$sql="UPDATE visites SET nb_connectes_jour = nb_connectes_jour +1;";

une table pour une valeur...tsssss si au moins c'était une table pour toutes les options...

après, le reste je suppose que les réponses proposées sont suffisantes.

[gene69]

en relisant toute les réponses, ya une solution qui me semble évidente pour faire un compteur de visiteurs connectés: puisqu'on utilises les sessions php, il suffit de compter le nombre fichier de session dont la date de modification est pas trop vieille.

pis si on a un peu d'astuce on peut même ouvrir les fichiers de sessions pour savoir qui y est, mais faut être astucieux.

[laurentSc]

Code :

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

$annee="20".date("y");

tssss

Code :

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

$annee=date("Y");

Non, ce n'est pas si nul, vu que la fonction data("y") retourne les 2 derniers chiffres de l'année, alors que MySQL en attend 4...

Code :

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

$sql="UPDATE visites SET nb_connectes_jour = nb_connectes_jour +1;";

une table pour une valeur...tsssss si au moins c'était une table pour toutes les options...

La table compte en fait 3 champs (compteur, nb_connectes_jour et date)

il suffit de compter le nombre fichier de session dont la date de modification est pas trop vieille.

Ca je veux bien, sauf que je ne sais pas où sont les fichiers de session, et si je savais, je ne saurais pas lire leur date de modif.

[Hoder]

Bonjour,

Non, ce n'est pas si nul, vu que la fonction data("y") retourne les 2 derniers chiffres de l'année, alors que MySQL en attend 4...

Ce qu'il/elle voulait dire, c'est que date("Y") est plus simple car cette fonction retourne les 4 chiffres directement ('y' != 'Y').

Et pour les fichiers de session, je ne sais pas non plus où ils sont

[laurentSc]

Ce qu'il/elle voulait dire, c'est que date("Y") est plus simple car cette fonctionne retourne les 4 chiffres directement ('y' != 'Y').

En effet, j'ai vérifié.
Et j'ai une autre idée pour faire un compteur de visites quotidien ; j'essaye puis informe ici.

[laurentSc]

Moi y en a pas comprendre, mais ça marche, donc ça me suffit. En fait, ce qui marchait mal, c'est le test :

Code :

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

1
2
3
$sid = session_id();
 
if ($_SESSION['id']!=$sid) {...

Mais comme je le fais déjà précédemment (pour le compteur totalisateur), je mets là à jour un flag ($sid_change) que je teste plus loin (pour le compteur quotidien), et comme ça, ça roule (va comprendre Charles, pourquoi, si au lieu de lever le flag, je refais le test, ça ne marche que aléatoirement...)