Discussion :

[laurentSc]

Bonjour,

j'ai le warning du titre avec ce code :

Code php :

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

1
2
3
4
5
6
7
8
9
$timestamp=time();
$ip=$_SERVER['REMOTE_ADDR'];
$query ="select * from visites where ((".$timestamp." - timestamp <= 3600) and ip = ".$ip.")";
$result=mysqli_query($db,$query);
 
$timestampmax=0;
while ($ligne = mysqli_fetch_array($result)) {
	if ($ligne['timestamp']>$timestampmax) $timestampmax=$ligne['timestamp'];
}

et si j'active l'affichage des erreurs mysqli avec mysqli_report(MYSQLI_REPORT_ALL);le warning est remplacé par ces 2 messages :

Fatal error: Uncaught exception 'mysqli_sql_exception' with message 'You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '.0.1)' at line 1' in C:\wamp\www\test\compteur.php on line 27

mysqli_sql_exception: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '.0.1)' at line 1 in C:\wamp\www\test\compteur.php on line 27

qu'est-ce qu'il faut changer ?

[armel18]

bonjour,

dans la requête faudrait entouré la variable $ip par des quotes car cette dernière est une chaîne de caractères.

Code php :

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

1
2
 
       $query ="select * from visites where ( '".$timestamp."' - timestamp <= 3600) and ip = '".$ip."'";

[laurentSc]

Merci beaucoup pour l'explication mais il ne faut pas entourer $timestamp pour que ça marche :

Code :

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

$query ="select * from visites where ((".$timestamp." - timestamp <= 3600) and ip = '".$ip."')";

[laurentSc]

J'ai décoché "résolu" car comme ça marchait sous Wamp, je l'ai mis en ligne et j'ai de nouveau le warning :

mysqli_fetch_array() expects parameter 1 to be mysqli_result, boolean given in

Pour mémoire le code correspondant est :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
//maj table "visites"
$timestamp=time();
$ip=$_SERVER['REMOTE_ADDR'];
$query ="select * from visites where ((".$timestamp." - timestamp <= 3600) and ip = '".$ip."')";
$result=mysqli_query($db,$query);
 
// déterminer le timestamp le plus récent pour cet IP
$timestampmax=0;
while ($ligne = mysqli_fetch_array($result)) {
	if ($ligne['timestamp']>$timestampmax) $timestampmax=$ligne['timestamp'];
}
...

[Invité]

1/ Il n'est pas du tout judicieux d'appeler un champs "timestamp", car c'est un "mot réservé" SQL.

2/ ce n'est pas la bonne methode.
=> mysqli_num_rows()
si 0 : Ok on ajoute
sinon on update

et :
select id ... (pas *)

[armel18]

comme vient de le signifier jreaux62, timestamp est un mot réservé et si tu veux l'utiliser il faut le protégé :

Code php :

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

1
2
 
$query ="select * from visites where ((".$timestamp." - `timestamp` <= 3600) and ip = '".$ip."')";