Discussion :

[Seta-san]

Bonsoir

Sur une page, j'affiche un petit compteur qui indique le nombre de fois ou celle-ci à été chargé. A chaque fois que quelqu'un charge la page, je récupère l'ancienne valeur dans ma bdd et j'y ajoute +1.

Mais voila, j'ai ce genre de résulat quand j'actualise : 3+1 = 5 ; 5+1 = 7; 7+1 = 9 ... Il me monte le résulatde 2 par 2.

Code :

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

1
2
3
4
5
6
// On récupere la valeur du compteur actuel et on lui ajoute 1
$compteur_actuel = $donnees['compteur'];
$compteur = $compteur_actuel + 1 ;
 
// On met à jour le compteur dans la base de données
mysql_query("UPDATE requetes SET  compteur ='".$compteur."' WHERE id='".$id."'") or die(mysql_error());

Le code est pourtant bon ^^

[DaRiaN]

Bonsoir,
pour incrémenter un champ MySQL, j'écris souvent une requête de ce style :

Code :

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

$req = 'UPDATE `requetes` SET `compteur` = `compteur`+1 WHERE `id` = '.$id;

En ce qui concerne une incrémentation PHP, le plus souvent je fais :

Code :

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

1
2
3
4
  $i = 1;
  $i++;
 
  echo $i; // retourne 2

Cordialement,
DaRiaN.

[riete]

Bien qu'il puisse être un peu simplifié à mon avis au niveau de l'écriture, ton code me semble juste.
Je simplifierai comme cela:
ton Code:

Code :

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

1
2
$compteur_actuel = $donnees['compteur'];
$compteur = $compteur_actuel + 1 ;

Celui que je te suggère:

Code :

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

$compteur = $donnees['compteur'] + 1 ;

Bon c'est de la petite économie.

Par contre pour ton pb, je regarderais si ta page n'est pas chargée 2 fois à un moment. Ce qui expliquerait que ton pb ne vient pas d'une erruer de calcul comme tu le pense, mais d'autre chose.
Cela m'est déjà arrivé
Fait un echo après l'envoye de ta requête.

[Seta-san]

Merci pour vos réponses. Il y a bien un echo après ma requête qui m'indique bien l'évolution 2 par 2.

C'est étrange oO

[riete]

Ecris ce code et on vera, quand même ya un méchant truc là

Code :

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

1
2
3
4
5
6
7
echo $donnees['compteur']."<br>";
$compteur_actuel = $donnees['compteur'];
$compteur = $compteur_actuel + 1 ;
 
$Query="UPDATE requetes SET  compteur ='".$compteur."' WHERE id='".$id."'";
echo $Query."<br>";
mysql_query($Query) or die(mysql_error());

[Seta-san]

J'ai éffacé toutes les données de la bdd, et maintenant ca fonctionne correctement. Etrange. Merci pour votre aide en tout cas