Problème de mise à jour d'une table

**SnickeursMan** · 16/01/2006, 18h27

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 :

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

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.

**XtofRoland** · 16/01/2006, 18h45

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.

**SnickeursMan** · 16/01/2006, 18h52

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.

**SnickeursMan** · 16/01/2006, 19h02

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 !

**titoumimi** · 17/01/2006, 11h33

Code :

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

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 :

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

echo '$hit'+1;

**Nesmontou** · 17/01/2006, 11h42

Salut, j'ai 2-3 remarques :

1° Tu as écrit

Code :

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

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 :

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

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 :

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

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 :

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

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 ;-)

**SnickeursMan** · 17/01/2006, 12h16

Merci beaucoup !

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

**titoumimi** · 17/01/2006, 12h18

donnes nous les résultat du

Code :

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

echo $strSQL;

et du

Code :

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

echo '$hit'+1;

, des fois que ...

**Nesmontou** · 17/01/2006, 12h20

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)

:

**SnickeursMan** · 17/01/2006, 12h25

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.

**SnickeursMan** · 17/01/2006, 12h26

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

**Nesmontou** · 17/01/2006, 12h28

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

:

Tu ne peux pas faire simplement

Code :

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

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

:

**SnickeursMan** · 17/01/2006, 12h29

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

**titoumimi** · 17/01/2006, 12h29

petite erreur de ma part :

Code :

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

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

**SnickeursMan** · 17/01/2006, 12h30

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 !