Compter le nombre de fois ou une page a été vue

[fabrice88]

Bonjour,
en parralele avec mon autre message sur ce forum je voudrait savoir comment faire pour compté le nombre de fois ou un sujet de mon forum aura été vue apr les internautes ?
J'imaginai faire comme ceci : dans ma table forum_sujet, je rajoute un nouveau champ que je nomme 'vues'.
ensuite lorsq'un internaute clique sur un sujet pour voir son detail je cré un compteur qui ajoute +1 a chaque ouverture du sujet.

Est ce que l'idée est bonne ou est ce qu'il faut procéder autrement ?

[M&B Multimédia]

Ton idée est bonne... personnellement je ne vois pas comment tu pourrais faire autrement.

[ZelkiN]

En effet tu n'as guerre le choix, et puis la plupart des compteurs sont fait ainsi, après tu peux gérer au niveau des ip pour ne pas compter plusieurs fois la meme page vu au meme utilisateur

[fabrice88]

ok merci bon alors voilà se que je fais :

Code: Tout sélectionner

   //création de la requête SQL: 
   $sqlvue = "UPDATE forum_sujets SET vue = $vue+1 WHERE id = ".$_GET['id_sujet_a_lire']; 
   $reqvue = mysql_query($sqlvue);


lors de la toute premiere ouverture de la page j'ai bien l'incrementation du + 1 qui se fait et j'ai donc mon champ vue qui prend la valeur de 1 au lieu de 0.
par contre apres sa ne fonctionne plus. si je retourne sur la meme page mon champ vue n'est plus incrémenté il reste à 1.

[spout]

ok merci bon alors voilà se que je fais :

Code: Tout sélectionner

   //création de la requête SQL: 
   $sqlvue = "UPDATE forum_sujets SET vue = $vue+1 WHERE id = ".$_GET['id_sujet_a_lire']; 
   $reqvue = mysql_query($sqlvue);


lors de la toute premiere ouverture de la page j'ai bien l'incrementation du + 1 qui se fait et j'ai donc mon champ vue qui prend la valeur de 1 au lieu de 0.
par contre apres sa ne fonctionne plus. si je retourne sur la meme page mon champ vue n'est plus incrémenté il reste à 1.

C'est le "$vue" qui semble bizarre, je sais pas ce qu'il contient, ça devrait plutôt être:

Code: Tout sélectionner

   //création de la requête SQL: 
   $sqlvue = "UPDATE forum_sujets SET vue = vue+1 WHERE id = ".$_GET['id_sujet_a_lire']; 
   $reqvue = mysql_query($sqlvue);


[fabrice88]

ah oui zut ! Merci.
Sa fonctionne.

[ZelkiN]

Euh Spout a raison d'une part, d'autre part attention a la sécurité o_O, ne met jamais un GET pas traité dans une requête SQL !

Si il s'agit d'un nombre, vérifie le avec un is_numeric, ou au moins met un addslashes($_GET['..']) sinon tu risques d'avoir des problèmes

Bon courage !

[spout]

Pour éviter les comptages multiples il y a un moyen simple (mais pas infaillible):
- Quand un comptage est fait, envoyer un cookie vers le navigateur du visiteur.
- Sauvegarder la dernière IP qui a incrémenté le compteur.
- Avant chaque incrémentation du compteur, vérifier si cookie et si pas la même IP.

Si un petit malin supprime les cookies, il est encore bloqué par la dernière IP.
La faille c'est si le visiteur change d'IP et supprime ses cookies, mais c'est quand même beaucoup mieux que sans vérification.

Edit: oui j'ai pas fait gaffe, il faut évidement bien vérifier le contenu du _GET avant de le transmettre à la requête.
Avec mysql_real_escape_string() http://php.net/manual/fr/function.mysql-real-escape-string.php au lieu de addslashes()

[fabrice88]

sinon tu risques d'avoir des problèmes

quel genre de probleme peut on avoir ?

[spout]

sinon tu risques d'avoir des problèmes

quel genre de probleme peut on avoir ?

http://fr.wikipedia.org/wiki/Injection_SQL

[fabrice88]

ok donc si je met en tout debut de ma page :

Code: Tout sélectionner

$id_sujet_a_lire = mysql_real_escape_string($_GET['id_sujet_a_lire']);


C'est ok ?

[spout]

oui

[fabrice88]

ok merci. alors du coup dès que j'intégre des données dans ma base sql je doit mettre mysql_real_escape_string().

Est ce que dans le cas de la requete ci dessous je doit la modifié :

Code: Tout sélectionner

$sql = 'INSERT INTO forum_reponses VALUES("", "'.$_POST['auteur'].'", "'.$_POST['message'].'", "'.$date.'", "'.$_GET['numero_du_sujet'].'")';

pour (mais sa va être long si y'a beaucoup de champs a remplir...):

Code: Tout sélectionner

$auteur = mysql_real_escape_string($_POST['auteur']);
$message= mysql_real_escape_string($_POST['message']);
$date = mysql_real_escape_string($_POST['date']);
$numero_du_sujet = mysql_real_escape_string($_GET['numero_du_sujet']);

$sql = 'INSERT INTO forum_reponses VALUES("", "'.$auteur.'", "'.$message.'", "'.$date.'", "'.$numero_du_sujet.'")';

[HawkEye]

NB: tu as pensé que chaque crawl de GoogleBot, Mediapartners, Slurp! et ses amis va faire une comptabilisation de plus ?

[hyadex]

Et si tu as de nombreuses pages vues par jours, ca va te faire un sacré nombre de requêtes SQL !