problème lors de l'appel d'une fonction

Bonsoir à tous,

Je renconte un problème que je n'arrive pas à solutionner :

J'ai une fonction qui incrémente un compte que voici :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

//incremente le compteur de vue du message function ajoutCompteurVueMessage($iIdMessage){   $sSql = " UPDATE ma_table SET nbre_vue = nbre_vue + 1 WHERE id_message= $iIdMessage AND id_pere=0" ;   $bRequete = mysql_query($sSql) or die(mysql_error());   }

---

ensuite je l'appelle comme ceci

Code:

---

1 2 3 4 5 6 7 8 9

if(isset($_REQUEST['id_discussion']) && (int)$_REQUEST['id_discussion'] !='' ){   //j'ajoute le clic de la visite $sNomCookie = 'message' . $_REQUEST['id_discussion']; if (!isset ($_COOKIE[$sNomCookie])){ $oMaClasse->ajoutCompteurVueMessage($_REQUEST['id_discussion']); setcookie('message' . $_REQUEST['id_discussion'], $_REQUEST['id_discussion']); } }

---

Le problème c'est que ca me mets +2 ou +3 à chaque fois.

Je comprends pas y'a pas de boucles...
Faut-il que je fasse un SELECT compteur puis un UPDATE compteur +1 ?
Cette méthode à toujours marché jusqu'ici (je viens de passer sur Mysql5)

Merci à vous

Code:

---

1 2 3 4 5 6 7 8 9 10

//incremente le compteur de vue du message function ajoutCompteurVueMessage($iIdMessage){   // Attention aux guillemets, ca vient surement de la ... $sSql = "UPDATE `ma_table` SET `nbre_vue` = `nbre_vue` + 1 WHERE `id_message`= ".$iIdMessage." AND `id_pere`=0" ;   // pas de retour donc pas d'affectation de variable, ca sert a rien tu perds du temps mysql_query($sSql) or die(mysql_error());   }

---

Désolé max44410 pour cette réponse tardive, et merci de tes commentaires.

Mais pourquoi les guillements pourraient géner ma requete ?

Qu'y a t-il de génant à faire des fonctions qui ne retournent rien ? en quoi le délai d'execution est ralongé ?

Merci

max44410, alors la chapeau, je suis developpeur PHP depuis pas mal de temps, et tu m'as scié !
Bien que je n'ai l'explication, le fait de retourner une variable( booléenne) depuis ma fonction et de tester en retour qu'elle soit bien la, fait que mon probleme est reglé....

Si tu as l'expliation, je suis preneur, car la je ne vois pas.

Bon en tout cas, pour ceux qui passeraient et qui veulent la soluce à mon probleme :
j'ai ajouté ca à ma fonction

Code:

---

1 2 3 4 5 6

if($bRequete == TRUE){ $bRetour = TRUE ; }else{ $bRetour = FALSE ; } return $bRetour ;

---

et ensuite je teste le retour

Code:

---

1 2 3 4 5

if (!isset ($_COOKIE[$sNomCookie])){ $bRetour = $oMaClasse->ajoutCompteurVueMessage($_REQUEST['id_discussion']); if($bRetour == TRUE){ setcookie('message' . $_REQUEST['id_discussion'], $_REQUEST['id_discussion']); }}

---

Code:

---

1 2 3 4 5 6 7 8 9 10

function ajoutCompteurVueMessage($iIdMessage){   // Attention aux guillemets, ca vient surement de la ... $sSql = "UPDATE `ma_table` SET `nbre_vue` = `nbre_vue` + 1 WHERE `id_message`= ".$iIdMessage." AND `id_pere`=0" ;   // pas de retour donc pas d'affectation de variable, ca sert a rien tu perds du temps return mysql_query($sSql);   }

---

Plus rapide encore ...
Je ne pense pas que cela vienne du retour... je ne me souvient pas que l'on soit obliger de retourner une valeur dans une fonction (fonction sans retour = procedure). Mais bon si ca fonctionne ... c'est le principale.

le truc que je te disais c'était que ca ne servait a rien d'affecter :

Code:

---

$bRequete = mysql_query($sSql) or die(mysql_error());

---

apres tu en faisais quoi de $bRequete ?? rien ?? donc ca sert à rien d'utiliser une variable inutilement.

Code:

---

mysql_query($sSql) or die(mysql_error());

---

ceci suffisait.

Voila, content que ton problème soit réglé.

AHhhhhhhhhhhhh Ok, je comprends ce que tu voulais dire maintenant.
J'e pense avoir compris la cause de pb.

j'ai l'impression que si je ne teste pas le retour, la fonction peut etre lancée plusieurs fois en fonction de la vitesse de chargement de la page.
ca me parait un peu bizarre, mais c'est a priori la solution la plus plausible.