Discussion :

[Denis Placé]

Bonjour,

Je souhaite incrémenter la valeur présente dans le champ 'hits' de la table `HitsParItems` lorsque le champ id est égal à la variable de session $_SESSION['idItem'].
Mon code est le suivant:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
			$select = "SELECT * FROM `HitsParItems` WHERE id = '".$_SESSION['idItem']."'";
			$result =mysql_query($select,$link) or die ('Erreur :'.mysql_errno().mysql_error());
			$rows = mysql_num_rows($result);
 
			if ($rows > 0)
				{
					while ($rows=mysql_fetch_array($result))
						{
							$hits=$rows['hits'] +1;
							$requete = mysql_query("UPDATE `HitsParItems` SET hits='".$hits."' WHERE id = '".$_SESSION['idItem']."'");
						}
				}

Ceci étant, ce champ ne s'incrémente pas donc .... j'ai une erreur que je ne vois pas.
Pouvez-vous m'aider, svp ?

[Invité]

Bonjour,
tout ton code se résume à :

Code php :

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

	$requete = mysql_query("UPDATE `HitsParItems` SET hits=hits+1 WHERE id = '".intval($_SESSION['idItem'])."'", $link);

N.B. Tu es bien conscient que l'extension mysql_ est obsolète depuis plusieurs années ?

[Denis Placé]

Bonjour Jreaux62,

Merci de ton aide, contrairement à ce que je croyais dans un premier temps, cela ne fonctionne pas même si c'est beaucoup plus succinct.
Le nouveau script est donc le suivant :

Code :

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

1
2
3
4
5
6
 
			include "mysql_conf.inc.php"; // Inclusion du fichier de connexion à la base de données
			$link=mysqli_connect($host,$login,$password) or die ('Erreur :'.mysqli_errno().mysqli_error());	
			$base = mysqli_select_db($base,$link);
 
			$requete = mysqli_query("UPDATE `HitsParItems` SET hits=hits+1 WHERE id = '".intval($_SESSION['idItem'])."'", $link);

Quant au changement de l'extension, même si je programme de façon bénévole, il va effectivement falloir que je me mette à la page ...

[Denis Placé]

Bonjour,
Mon code devenu le suivant grâce au conseil de Jreaux62 n'incrémente toujours pas ma variable hits.
Pouvez-vous m'aider, svp ? Merci.

Code :

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

1
2
3
4
5
include "mysql_conf.inc.php"; // Inclusion du fichier de connexion à la base de données
			$link=mysqli_connect($host,$login,$password) or die ('Erreur :'.mysqli_errno().mysqli_error());	
			$base = mysqli_select_db($base,$link);
 
			$requete = mysqli_query("UPDATE `HitsParItems` SET hits=hits+1 WHERE id = '".intval($_SESSION['idItem'])."'", $link);

[Invité]

Bonjour,

TU UTILISES A L'ENVERS la fonction mysqli_query() !

=> mysqli_query ( mysqli $link , string $query [, int $resultmode = MYSQLI_STORE_RESULT ] )

mysql_query() et mysqli_query() n'ont pas la même syntaxe !
=> peux-tu nous montrer le contenu du fichier mysql_conf.inc.php ? ? (en masquant tes paramètres perso de connexion, bien entendu !)
Car il ne faut pas mélanger mysql_ et mysqli_ !

Question subsidiaire :
Pour quoi ne pas avoir simplement mis la ligne $link=mysqli_connect(...................); DANS le fichier mysql_conf.inc.php ?


En phase de test, tu peux afficher les erreurs :

Code php :

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

1
2
    $query = "UPDATE `HitsParItems` SET hits=hits+1 WHERE id = '".intval($_SESSION['idItem'])."'";
    $requete = mysqli_query( $link, $query ) or die ('Erreur :'.mysqli_errno().mysqli_error()); // or die...... à enlever en production

Cette écriture te permet aussi de vérifier la requête :

Code php :

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

echo $query;

UTILISATION DE REQUÊTES PRÉPARÉES :
Quand tu as des paramètres à transmettre via une requête SQL, il faut SYSTÉMATIQUEMENT utiliser une REQUÊTE PRÉPARÉE !
Voir :

• mysqli_prepare()
• mysqli_stmt_bind_param()

La requêtre ci-dessus devient alors :

Code php :

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

1
2
3
4
5
6
    $query = "UPDATE `HitsParItems` SET hits=hits+1 WHERE id = ? ";
    $requete = mysqli_prepare( $link, $query ) or die ('Erreur :'.mysqli_errno().mysqli_error()); // or die...... à enlever en production
    /* Lecture des marqueurs */
    mysqli_stmt_bind_param( $requete, 'i', $_SESSION['idItem'] ); // 'i' correspond à un entier. Donc plus besoin de intval()
    /* Exécution de la requête */
    mysqli_stmt_execute( $requete );

[Denis Placé]

Bonsoir à tous et plus particulièrement à Jreaux62 qui m'a encore aidé.
Le fichier mysql_conf.inc.php ne contient que mes paramètres de connexions :

Code :

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

1
2
3
4
5
6
7
8
 
<?php
// Paramètres de configuration de la base de données
$host = 'XXX'; // Nom du serveur MySQL
$base = 'XXX'; // Nom de la base de données
$login = 'XXX'; // Nom d'utilisateur d'accès à la base de données
$password = 'XXX'; // Mot de passe d'accès à la base de données
?>

Je vais regarder de plus près les requêtes préparées mais je ne comprends pas comment fonctionne le ? de la 1ère ligne. Aurais-tu la gentillesse de m'éclairer, stp ?

Code :

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

1
2
 
$query = "UPDATE `HitsParItems` SET hits=hits+1 WHERE id = ? ";

[Denis Placé]

Bonjour,

Après avoir regardé les liens envoyés par Jreaux62, je considère mon problème résolu.
Merci à tous (surtout à Jreaux62 )