Discussion :

[MacSIM]

Bonjour,

voilà je débute en PHP/SQL et je voudrai un peu de votre aide pour débloquer mon problème

Explication de ce que je veux faire :

Une page avec des menus déroulants, et lorsque j'ai choisis les options de menus déroulants ça les ajoutes/modifies dans la base sql.

Donc, je suis pour l'instant lancé sur cette idée :

• Une première page nommée page1.php

Dans cette page, il y a un formualire avec 22 bloc de type 'text' avec leur valeur dejà défini (nombres de 1 à 22) + 22 menu déroulants qui trouvent leurs options (au nombre de 22 aussi) dans la base MySQL.

• Une deuxième page nommée script.php

Dans cette page, le script que je ne parvient pas à faire, enfin qui ne fonctionne pas

Ce que j'ai déjà fais :

• La page nommée page1.php

Les 22 blocs ainsi que les 22 menus déroulants qui s'alimentent dans la base SQL, c'est bon.
Un bouton Submit qui envoie les données sur la page script.php. L'envoie de données d'une page à l'autre fonctionne car j'ai testé sur un simple 'input type=text' et j'ai réussi à récupérer la donnée en passant par un echo.

Voici un bout de code :

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
 
<form action="script.php">
	<input type="text" value="1" maxlength="2" style="width: 30px" name="aa">
   <select name="p1">
<?php
$SQL = "SELECT * FROM grille_depart";
$res = mysql_query($SQL);
while($val=mysql_fetch_array($res)) {
   echo "<option>".$val["pilote"]."</option>\n";
}
?>
   </select>
 
[...]
 
<br>
	<input name="Boutton" type="submit" value="Envoyer">
 
</form>

• La page nommée script.php

Alors là j'ai pas fais grand chose car je test à chaque fois si ca fonctionne sur le 1er menu déroulant..mais voici le principal qui fonctionne pas

Code :

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

1
2
$query = "INSERT INTO grille_depart (place,pilote) VALUES ('23', "$_GET['p1']")";
$result = mysql_query($query, $link) or die($query . " - " . mysql_error());

Et là il me dit :

Parse error: parse error, unexpected T_VARIABLE in script.php on line 7

(la ligne INSERT INTO).



Voilà si quelqu'un peut m'aider parceque je trouve pas Merci à tous.

[SpiritOfDoc]

Bonsoir et bienvenu sur les forums developpez.net ^_^

Alors, il y a un problème dans cette ligne :

Code :

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

$query = "INSERT INTO grille_depart (place,pilote) VALUES ('23', "$_GET['p1']")";

Il faut faire ceci :

Code :

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

$query = 'INSERT INTO grille_depart (place,pilote) VALUES (\'23\', \''.$_GET['p1'].'\')';

Voilà, tu avais oublié de concaténé ta chaine. (j'en ai profité pour la passer en '' au lieu de "").

[MacSIM]

Bonsoir et merci pour la bienvenue,

merci beaucoup ca fonctionne

J'avais pensé à protéger avec les / mais j'avais fais que pour le GET, merci beaucoup maintenent je saurai

@++

[SpiritOfDoc]

Je suis pas sûr d'avoir compris ta réponse ^_^.

Mais de toute manière, il faut que tu fasses des vérifications sur le $_GET['truc'] avant de le rentrer dans ta Base de donnée. Ne jamais faire confiance à une donnée extérieur (ça passe par l'url, le visiteur peut la changer -_-).

Bonne soirée ^_^.

[MacSIM]

Je voulais dire que j'avais essayé ce que tu as donné en réponse, mais que j'avais pas mis les \ partout et j'avais laissé des "".

Enfin ca fonctionne et c'est bon, sinon pour l'url j'ai vu qu'on peut utiliser la méthode POST. Mais bon c'est un script d'administration du site donc qui a vocation à être visible que par moi (je mettrai un petit htaccess dans le répertoire).

Voilà et encore merci à toi, et à la communautée aussi en passant

[SpiritOfDoc]

Cela ne te coute rien de protéger tes entrées à l'aide de mysql_real_escape_string(); et au moins, c'est plus sécurisé ^_^. (Les pirates sont pas sensé aller là où ils vont xD)

[MacSIM]

Ok je vais essayé ca demain Si j'ai un soucis je reviens poser la question sur le forum