mysqli_fetch_array() expects parameter 1 to be mysqli_result

Bonjour,

j'ai le warning du titre avec ce code :

Code:

---

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 :

Citation:

---

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

---

Citation:

---

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 ?

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:

---

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

---

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

Code:

---

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

---

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 :

Citation:

---

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

---

Pour mémoire le code correspondant est :

Code:

---

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']; } ...

---

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

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

Code:

---

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

---

Code:

---

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 } ?>

---

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

Code:

---

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:

---

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;

---

...

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

Code:

---

`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:

---

$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" !

Citation:

---

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:

---

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

Citation:

---

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:

---

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

---