Problème de mise à jour d'une table

Bonsoir, j'ai un gros problème et ça fait depuis le week end que je m'arrache les cheveux dessus.

Voilà je voudrais faire un compteur de page (combien de fois a été vu un article). Alors j'ai rajouté champ "hit" dans ma table puis quand l'utilisateur clique sur l'article la table fait appelle à un fichier avec la méthode include qui se charge de se connecter à la base de donné et de mettre à jour le champ en ajoutant +1.

Mais ca ne marche pas pour tout les articles. Et je ne comprend vraiment pas pourquoi !

Voici le code :

Code:

---

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

<? require("admin/config.inc");       $db = mysql_connect("$dbhost", "$dblogi", "$dbpass") or Die("Base Down !");   mysql_select_db("$dbbase",$db) or Die("Base Down !");       $req = MYSQL_QUERY("SELECT * FROM $TBL_RIDDIMS");     $hit = addslashes(stripslashes(trim($hit)));   $rqt = Mysql_Query("UPDATE $TBL_RIDDIMS SET genre='$genre', titre='$titre', year='$year', auteur='$auteur', tracklist='$tracklist', date='$date', extrait='$extrait', url='$url', hit='$hit'+1 WHERE id='$id'"); ?>

---

Quelqu'un peu m'aider SVP ? Je l'ai fais pour une autre table mais bizarrement rien ne marche !!! Le code est le même sauf les champs de la table.

un conseil,
ton sql, met le dans une variable.
et fait un echo de cette variable (juste le temps de comprendre pourquoi ca plante.)

un truc me semble bizare, hit='$hit'+1
c'est possible que ce soit correcte (php etant tres faiblement typé)
mais faire une addition sur un string j'evite. (mais c'est bien possible que ca ne soit pas ton erreur.)

aussi pk un $ devant ta table ? $TBL_RIDDIMS ... juste une remarque, selon ton code ca peut etre tt a fait correcte.

En réalité $TBL_RIDDIMS est une variable dans le fichier config.inc qui désigne la table dans la base de données.

Je vais faire ce que tu m'as dis.

Tu m'as demandé de mettre mon SQL dans une variable mais c'est à dire ?

Tu veux parler de la requête $rqt pour mettre à jour le champ ?

si oui rien ne s'affiche !

Code:

---

1 2 3

$strSQL = "SELECT * FROM '.$TBL_RIDDIMS; echo $strSQL; mysql_query($strSQL);

---

ca va te permettre de voir la requête qui sera envoyée à ton mysql_query

essaie aussi ca, pour vérifier ta valeur (voir, essayer un is_int() pour être sur du type) :

Code:

---

echo '$hit'+1;

---

Salut, j'ai 2-3 remarques :

1° Tu as écrit

Code:

---

1 2 3

$db = mysql_connect("$dbhost", "$dblogi", "$dbpass") or Die("Base Down !");   mysql_select_db("$dbbase",$db) or Die("Base Down !");

---

Tu utilises des variables, donc c'est inutiles de les mettre entre "

Tu peux faire

Code:

---

1 2 3

$db = mysql_connect($dbhost, $dblogi, $dbpass) or Die("Base Down !");   mysql_select_db($dbbase,$db) or Die("Base Down !");

---

Ca ne va pas résoudre ton problème, mais autant faire simple ;-)

2° Comme te l'a dit XtofRoland, autant stocker te requête dans une variable

Code:

---

1 2 3

$sql = 'SELECT id FROM ma_table'; $query = mysql_query($requete); ...

---

3° Dans une requête SQL, si tu utilises des champs de type numérique, il ne faut pas mettre de ', sinon, tu en mets

Ton script devient

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24

<? require("admin/config.inc");   $db = mysql_connect($dbhost, $dblogi, $dbpass) or Die("Base Down !"); mysql_select_db($dbbase, $db) or Die("Base Down !");   $sql = 'SELECT * FROM '.$TBL_RIDDIMS; $req = MYSQL_QUERY($sql);   $hit = addslashes(stripslashes(trim($hit)));   $sql = 'UPDATE '.$TBL_RIDDIMS; $sql .= ' SET genre=\''.$genre.'\','; $sql .= ' titre=\''.$titre.'\','; $sql .= ' year='.$year.','; $sql .= ' auteur=\''.$auteur.'\','; $sql .= ' tracklist=\''.$tracklist.'\','; $sql .= ' date=\''.$date.'\','; $sql .= ' extrait=\''.$extrait.'\','; $sql .= ' url=\''.$url.'\','; $sql .= ' hit='.($hit + 1); $sql .= ' WHERE id='.$id; $rqt = mysql_query($sql); ?>

---

J'ai supposé que tes champs year et id étaient de type numérique

Personnellement, je préfère les ' aux " pour les chaînes de caractères, mais tu peux les remettre si tu préfères

Bon développement ;-)

Merci beaucoup !

J'ai modifier le code comme tu m'as indiquer mais je rencontre toujours le même problème :(

donnes nous les résultat du

Code:

---

echo $strSQL;

---

et du

Code:

---

echo '$hit'+1;

---

, des fois que ...

Et tu as remarqué un truc commun sur les articles qui ne sont pas mis à jour :?: Tu as testé la requête dans PHPMyAdmin (par exemple) :?:

Sa marche très bien quand je passe par l'administration de mon site quand je rentre un chiffre dans un champ texte. Mais quand j'utilise fais la requete +1 sa foire.

Lol le résultat qui s'affiche avec la requete en écho que tu m'as demandé sé uniquement : 1

Au fait, tu es obligé de mettre à jour tous les champs :?:

Tu ne peux pas faire simplement

Code:

---

1 2 3

$sql = 'UPDATE '.$TBL_RIDDIMS; $sql .= ' SET hit='.($hit + 1); $sql .= ' WHERE id='.$id;

---

:?:

Non je suis pas obligé, je veux juste mettre à jour le champ HIT

petite erreur de ma part :

Code:

---

1 2 3 4

$strSQL = "SELECT * FROM ".$TBL_RIDDIMS; echo $strSQL; mysql_query($strSQL);

---

(une double quote au lieu d'une simple lors de la concaténation)

toutes mes excuses :oops:

Merci apparement sa marche le code que tu viens de me donner ! Je te remerci énormément ! Maintenant je vais tester sur les autres tables.

merci encore beaucoup beaucoup ! :D

Citation:

---

Envoyé par titoumimi

une double quote au lieu d'une simple lors de la concaténation

---

Euh, c'est pas la même chose :?:

bah j'avait commencé ma chaine par une double, mais fini par une simple... forcément, il a pas apprécié la plaisanterie :D

Ah, ok, j'avais pas vu, je croyais que tu disais que pour concaténer il fallait des doubles quotes obligatoirement

Mille excuses :oops:

Sa marche impec, mais quel etait alors le problème detout cela ?