Conditions pour un compteur

Version imprimable

bonjour,
je récupère l'adresse ip, le timestamp et l'addresse url que j'enregistre dans une table pour pouvoir faire un compteur du nombre de visite de chaque fiche article, voici le code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
$retour = mysql_query('SELECT COUNT(_ip) AS nbre_entrees, _ip, _url FROM tmp_connectes WHERE _ip=\'' . $_SERVER['REMOTE_ADDR'] . '\' GROUP BY _ip') or die (mysql_error());
$donnees = mysql_fetch_array($retour);
 
if ($donnees['nbre_entrees'] == 0) // L'ip ne se trouve pas dans la table, on va l'ajouter
{
    mysql_query('INSERT INTO tmp_connectes VALUES(\'' . $_SERVER['REMOTE_ADDR'] . '\', ' . time() . ', \''.$_SERVER['REQUEST_URI'].'\')')or die(mysql_error());
 
		$i=$nb_visite+1;
		$sql_maj = "UPDATE tb_shirt SET nbvisite_shirt=".$i." WHERE id_shirt=$ids";
		mysql_query($sql_maj) or die(mysql_error());
 
}
elseif($donnees['_ip']==$_SERVER['REMOTE_ADDR'] && $donnees['_url']==$_SERVER['REQUEST_URI']) // L'ip se trouve déjà dans la table, on met juste à jour le timestamp
{	
 
    mysql_query('UPDATE connectes SET _timestamp=' . time() . ', _url=\''.$_SERVER['REQUEST_URI'].'\' WHERE ip=\'' . $_SERVER['REMOTE_ADDR'] . '\'');
}elseif($donnees['_ip']==$_SERVER['REMOTE_ADDR'] && $donnees['_url']!=$_SERVER['REQUEST_URI']){
 
		$i=$nb_visite+1;
		$sql_maj = "UPDATE tb_shirt SET nbvisite_shirt=".$i." WHERE id_shirt=$ids";
		mysql_query($sql_maj) or die(mysql_error());
}

rééditer :
le problème c'est qu' avec cette condition,si l'utilisateur revient sur la même page,le compteur sera incrémenté quand même, ... quelle doit être la bonne condition ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
if ($donnees['nbre_entrees'] == 0) // L'ip ne se trouve pas dans la table, on va l'ajouter
{
    //on ajoute l'ip dans la table, le timestamp  et l'url de la page
    // puis on incrémente le compteur (nombre de visite sur la page de l'article)
 	
}
elseif($donnees['_ip']==$_SERVER['REMOTE_ADDR'] && $donnees['_url']==$_SERVER['REQUEST_URI']) // L'ip se trouve déjà dans la table & l'url aussi
{	
	//je mets à jour seulement le timestamp
        // dans mon cas, sa ne sert à rien mais bon ...
}elseif($donnees['_ip']==$_SERVER['REMOTE_ADDR'] && $donnees['_url']!=$_SERVER['REQUEST_URI'])//l'ip se trouve déjà  dans la table mais l'url est différent
{
    //on ajoute l'ip dans la table, le timestamp  et l'url de la page
    // puis on incrémente le compteur (nombre de visite sur la page de l'article)
 
}

quoi qu'il en soit il incrémente toujours ... (il mais qui est 'il' ?lol php ? la condition ?)
enfin bref ne devenons pas schizophrène -_-'
quel doit être exactement la condition ?

05/09/2007, 11h24
akara

à force d'enregistrer les adresses ip , l'url et le timestamp c'est pas génant ??
mais bon d'un coté je m'enfou car c'est pour la soutenance que je passe le 20 septembre :cry:, mais pour info perso , j'aimerai savoir quel serai le meilleur moyen....
,

Citation:

_ip _timestamp _url

127.0.0.1 1188813156 /akstore/fiche_shirt.php?id=0000001
127.0.0.1 1188821356 /akstore/fiche_shirt.php?id=0000001
127.0.0.1 1188821383 /akstore/fiche_shirt.php?id=0000001
127.0.0.1 1188983533 /akstore/fiche_shirt.php?id=0000004
127.0.0.1 1188983536 /akstore/fiche_shirt.php?id=0000004
127.0.0.1 1188983561 /akstore/fiche_shirt.php?id=0000004
127.0.0.1 1188983564 /akstore/fiche_shirt.php?id=0000004

alors que j'ai fait la condition que si _ip & _url existe pas d'incrémentation & pas ajout dans la table mais il le fait quand même :aie:
05/09/2007, 11h27
koopajah

Affiches la requete que tu fais pour savoir si l'adresse est deja dans la base. Ensuite teste la dans phpmyadmin, il y a de fortes chances pour qu'il y ait un espace en trop par exemple ou une betiste de ce genre
05/09/2007, 11h43
akara

Merci,
je ne pensais pas que c'était un problème de requête !
je me suis pris la tête avec le count(*) et le group by

alors que cette simple reqête aurait suffit largement...

'SELECT _ip, _url FROM tmp_connectes WHERE _ip=\'' . $_SERVER['REMOTE_ADDR'] . '\'';

merci

//éditer, petite question :
comment font en général les webmaster pour le compteur ?
il faut effacer ce qu'il y a dans la tabe ? mais quand exactement

essaie d'utiliser un seul elseif... parce que tu te répètes ... dedans

essaie ceci voir après le premier if:

Code:

1
2
3
4
5
6
7
if($donnees['_ip']==$_SERVER['REMOTE_ADDR'] && $donnees['_url']!=$_SERVER['REQUEST_URI']){
 
		$i=$nb_visite+1;
		$sql_maj = "UPDATE tb_shirt SET nbvisite_shirt=".$i." WHERE id_shirt=$ids";
		mysql_query($sql_maj) or die(mysql_error());
}else{
mysql_query('UPDATE connectes SET _timestamp=' . time() . ', _url=\''.$_SERVER['REQUEST_URI'].'\' WHERE ip=\'' . $_SERVER['REMOTE_ADDR'] . '\'');