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."')";

[Invité]

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
<?php
$query ="SELECT id FROM visites WHERE ((".$timestamp." - `timestamp`) <= 3600 AND ip = '".$ip."')";
$result = mysqli_query($db,$query);
$nbre_result = mysqli_num_rows($result);
// déterminer le timestamp le plus récent pour cet IP
if ($nbre_result>0) {
	echo 'visites RECENTES !'; // => UPDATE
} else {
	echo 'visites datant de plus de 3600 secondes !'; // => INSERT
}
?>

[laurentSc]

Avant de lire ces derniers posts, j'entreprenais de changer le nom de la colonne, mais vos réponses me vont très bien (sauf que ce n'est pas id mais ip )

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
//maj table "visites"
$timestamp=time();
$ip=$_SERVER['REMOTE_ADDR'];
$query ="select ip 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
$nbre_result = mysqli_num_rows($result);
 
if ($nbre_result==0) 
	$query = "INSERT INTO visites (ip,timestamp) VALUES('".$_SERVER['REMOTE_ADDR']."','".time()."')";
else
		$query="update visites set timestamp = ".$timestamp." where ip = ".$ip;
mysqli_query($db,$query);

et l'explication du warning, ce n'était pas une erreur de ma requête SQL, mais qu'il y avait déjà une table "visites" et

Code sql :

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

1
2
3
4
CREATE TABLE IF NOT EXISTS `visites` (
  `ip` varchar(16) COLLATE utf8_unicode_ci NOT NULL,
  `timestamp` int(10) NOT NULL
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;

...

[Invité]

J'ai mis volontairement "id", car dans ta table, il manque :

Code sql :

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

`id` int(10) NOT NULL AUTO_INCREMENT,

"id" étant UNIQUE, il identifie à coup sûr la ligne.

Sans ça, il faut que ta requête UPDATE soit :

Code :

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

$query="UPDATE visites SET `timestamp` = '".$timestamp."' WHERE ((".$timestamp." - `timestamp`) <= 3600 AND ip = '".$ip."');

Car, si j'ai bien compris, plusieurs lignes peuvent avoir le même "ip" !

[laurentSc]

Car, si j'ai bien compris, plusieurs lignes peuvent avoir le même "ip" !

Tu as bien compris car si le même visiteur revient au bout d'un certain temps, je souhaite le compter comme une visite supplémentaire...
Sinon, j'ai retenu ta 2e solution car si j'essaye la première, à

Code sql :

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

1
2
3
4
5
CREATE TABLE IF NOT EXISTS `visites` (
`id` int(10) NOT NULL AUTO_INCREMENT,
  `ip` varchar(16) COLLATE utf8_unicode_ci NOT NULL,
  `timestamp` int(10) NOT NULL
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;

il me répond

Incorrect table definition; there can be only one auto column and it must be defined as a key

et je compte bien conserver le champ "ip".
Au fait, il manquait une cote :

Code php :

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

$query="UPDATE visites SET `timestamp` = '".$timestamp."' WHERE ((".$timestamp." - `timestamp`) <= 3600 AND ip = '".$ip."')";