Discussion :

[Delba146]

Bonjour,
Je debute en php et j'essaie depuis plusieurs jours d'ajouter des données dans une base ce qui fonctionne bien par contre j'ai un message d'erreur :

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
25
26
27
28
29
30
31
32
33
34
35
36
37
<?php 
 
$host="sql.free.fr";
$base="frevale";
$passe="motdepasse";
 
  //on se connecte à MySQL
 
mysql_connect($host,$base,$passe);  
 
 // on seléctionne la base 
 
mysql_selectdb("Test");
 
if(isset($_GET['Nom'])) $Nom=$_GET['Nom']; else $Nom="Vide";
if(isset($_GET['Prenom'])) $Prenom=$_GET['Prenom']; else $Prenom="Vide";
 
 
// insertion
 
mysql_query ("INSERT INTO Test VALUES  (Now(),'$Nom','$Prenom')");
 
 
// relecture derniere entrée
 
$reponse = mysql_query("INSERT INTO Test VALUES (Null,Now(),'$Nom','$Prenom')") or die("'Erreur SQL !'");
 
//$reponse = mysql_query ("SELECT * FROM Test ORDER BY id DESC LIMIT 1"); // Requête SQL  
 
 while ($donnees = mysql_fetch_array($reponse)) // On boucle pour afficher toutes les données
 {
 echo "**|".$donnees['Nom']."|".$donnees['Prenom']."|**";
 }
 
// on ferme la connexion à mysql
mysql_close();
 ?>

En retour j'ai ce message :
Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /mnt/116/sdb/9/1/frevale/toto2.php on line 30



Par contre dans la bd je vois bien l'enregistrement se faire.

Merci pour votre aide et indulgence.

[Obsidian]

Bonjour,

À la ligne 26, ta requête de relecture est un INSERT INTO (probablement copié-collé depuis la précédente) et pas un SELECT FROM comme elle le devrait. Tu as probablement décommenté la mauvaise ligne puisque je vois que tu l'as écrite dans celle qui suit.

Par ailleurs, l'API « mysql_ » est obsolète. Il existe bien « mysqli_ » pour la remplacer mais si tu débutes en PHP et en base de données, je te suggère de partir directement sur PDO.

J'ajoute également qu'écrire un script qui ne fait pas d'authentification utilisateur préalable et qui insère directement des valeurs tirées de $_GET[] (donc de l'extérieur) dans la requête SQL est une grave faille de sécurité : c'est de cette façon que l'on introduit une « SQL injection ». Il faut au minimum « sanitizer » le contenu de tes variables mais si tu le peux, essaie de prendre directement l'habitude de construire des requêtes paramétrées (et typées).

Bon courage.

[laurentSc]

Bonjour (et bienvenu dans DVP),

Obsidian a raison de te signaler une erreur sur ta requête SQL ligne 26 ainsi que de te conseiller de privilégier PDO. Y a même encore mieux (je trouve) : https://github.com/rawsrc/PDOPlusPlus ou en français (ce n'est pas à jour (2020) mais le principe n'a pas changé) : https://www.developpez.net/forums/bl...dutiliser-pdo/. Et la classe utilise PDO (comme son nom l'indique).
Cette classe est simple à prendre en main et privilégie la sécurité (échappement des données dans un insert, ce que tu ne fais pas, et que PDO permet aussi (requêtes préparées))

Autres remarques sur tes requêtes SQL :
1- aux lignes 21 et 26, tu gères la même table et cependant, à la ligne 21 y a 3 colonnes et 4 à la ligne 26...
2- dans la ligne 28 (qui est commentée), tu fais un SELECT *. A ne pas faire. Il vaut mieux préciser le nom des colonnes (du genre SELECT (date, nom, prenom) from Test ) et pareil pour le INSERT ligne 21 (INSERT INTO Test (date, nom, prenom) VALUES (Now(),'$Nom','$Prenom') ) sauf que le nom et le prénom, il faudrait les échapper.