Problème de mise à jour d'une table

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

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, 17h45

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, 17h52

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, 18h02

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, 10h33

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, 10h42

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, 11h16

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, 11h18

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, 11h20

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, 11h25

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, 11h26

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

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

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, 11h29

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

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

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, 11h30

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 !