Lier mon formulaire à ma bdd

Version imprimable

bonjour,

Je me suis lancée dans le php dpeuis peu..
J'ai créer un formulaire sur mon site. Les visiteurs peuvent y insérer un com.
J'airemaris que ces com soit stockés dans ma BDD mais cela ne fonctionne pas..
J'ai pourtant tout vérifié...
voici mo code :

Code:

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
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" 
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"> 
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" lang="fr"> 
<head> 
<title> envoyer</title> 
</head> 
<body> 
<?php 
require_once("config.php");
if ((isset($_POST["pseudo"]))&&(isset($_POST["mail"]))&&(isset($_POST["message"])))
{
 
mysql_connect(HOST,USER,PASSWD) or die ('Impossible de se connecter à la BDD'.mysql_error()); 
mysql_select_db(DB);
 
$pseudo=$_POST["pseudo"];
$mail=$_POST["mail"]; 
$message=$_POST["message"]; 
 
$pseudo=mysql_real_escape_string($pseudo);
$mail=mysql_real_escape_string($mail);
$message=mysql_real_escape_string($message);
 
$sql="SELECT * FROM message";
$Req="INSERT INTO mess VALUES('$pseudo','$mail','$message')"or die(mysql_error());
mysql_query($Req) or die('Erreur SQL !'.$sql.'<br>'.mysql_error());  
mysql_close();
}
 
?> 
</body>
 
</html>

Pensez-vous que j'ai fais une erreur quelquespart?? sachant qu'aucune erreur n'est retournée.

Merci bien.

31/12/2009, 09h48
Seb33300

Bonjour,

Citation:

Envoyé par senderia

J'airemaris que ces com soit stockés dans ma BDD mais cela ne fonctionne pas..

Une erreur s'affiche ? Qu'est ce qui se passe exactement ?
31/12/2009, 09h55
senderia

justement rien.
J'envoie mon formulaire et j'accède à ma page merci d'avoir laissé votre com (je m'ai pas mis ici dans mon code ) et puis qd je me rend sur phpmayadmin aucun enregistrement n'est présent..

alors voilà..ça fait 2 jours que je suis dessus et je sais plus quoi faire..

Juste un petit détail, même si ça ne règlera pas ton problème, au lieu de
Code:

1 2 3 4 5 6 7 $pseudo=$_POST["pseudo"]; $mail=$_POST["mail"]; $message=$_POST["message"]; $pseudo=mysql_real_escape_string($pseudo); $mail=mysql_real_escape_string($mail); $message=mysql_real_escape_string($message);
tu peux faire directement :
Code:

1 2 3 $pseudo=mysql_real_escape_string($_POST["pseudo"]); $mail=mysql_real_escape_string($_POST["mail"]); $message=mysql_real_escape_string($_POST["message"]);
Et sinon c'est normal que sur tes 2 requêtes, un coup ta table s'appelle message et l'autre coup elle s'appelle mess ?

Peux tu rajouter un echo devant cette ligne pour voir ce que ta variable contient ?
(Et le or die(mysql_error()) ne doit pas etre mis ici)

Code:

echo $Req="INSERT INTO mess VALUES('$pseudo','$mail','$message')";

alors j'ai fais les modif.. pas d'erreur.. rien ne 's'affiche.. avec echo c'est censé marquer quoi? dsl je débute..

voici le code corrigé :

Code:

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
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" 
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"> 
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" lang="fr"> 
<head> 
<title> envoyer</title> 
</head> 
<body> 
<?php 
require_once("config.php");
if ((isset($_POST["pseudo"]))&&(isset($_POST["mail"]))&&(isset($_POST["message"])))
{
 
mysql_connect(HOST,USER,PASSWORD) or die ('Impossible de se connecter à la BDD'.mysql_error()); 
mysql_select_db(DB);
 
$pseudo=mysql_real_escape_string($_POST["pseudo"]);
$mail=mysql_real_escape_string($_POST["mail"]);
$message=mysql_real_escape_string($_POST["message"]);
 
$sql="SELECT * FROM messages";
echo $Req="INSERT INTO messages VALUES('$pseudo','$mail','$message')";
mysql_query($Req);
mysql_close();
}
 
?> 
<a href= "http://localhost/C/accueil.html"><img src="Image\BB.gif"></a> 
<br><br><br>
<center>
<font color="blue" size = 15> Je vous remercie pour le message!! </font>
</center>
 
</body>
 
 
</html>

31/12/2009, 11h01
Seb33300

C'est censé afficher la requête..

Mais si rien ne s'affiche, c'est que tu ne rentre pas dans la condition

Code:

if ((isset($_POST["pseudo"]))&&(isset($_POST["mail"]))&&(isset($_POST["message"])))

Peux tu montrer le code html du formulaire qui affiche cette page ?

Et sinon, est ce que le texte "Je vous remercie pour le message!!" s'affiche bien en bas de page ?

alors, le message Je vous remercie pour le message s'affiche sur une eutre page quand j'appuie sur "envoyer" (le bouton).

Voici mon formulaire :

Code:

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
                       "http://www.w3.org/TR/REC-html40/loose.dtd">
 
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
</head>
<body>
<h1 style="text-align : center;"> <img src="Image\R.gif"></h1>
<FORM id="livredor"  method="POST" action="envoyer.php">
<br/>
<br/>
<br/>
<table>
  <tr>
    <td>
      <strong>Votre Pseudo: </strong>
    </td>
    <td>
      <input name="pseudo" type="text" size="50"/>
    </td>
  </tr>
  <tr>
    <td>
      <strong>Votre mail: </strong>
    </td>
    <td>
      <input name="mail" type="text" size="50"/>
    </td>
  </tr>
  <tr>
    <td>
      <strong>Votre message : </strong>
    </td>
    <td>
      <textarea name="texte" cols="50" rows="20"/>
      </textarea>
    </td>
  </tr>
  <tr>
    <td>
      <input name="annuler" type="reset" value="Annuler"/>
    </td>
    <td>
      <input name="soumettre" type="submit" value="Envoyer"/>
    </td>
  </tr>
</table>
</form>
</body>
</html>

merci

31/12/2009, 11h18
Seb33300

Ton textarea s'appelle "texte" or tu essayes de récupérer "message" dans ton php.
Tu dois donc modifier le name de ton textarea en "message"
31/12/2009, 11h39
senderia

hello,
alors quand je met message la page où il y a écrit "je vous remercie;.." ne s'affiche plus.. ça me met " Internet Explorer ne peut pas afficher cette page Web " :pleure:

edit, qd je réactualise cette page j'accède à ma page je vous remercie...
comprends plus rien là
31/12/2009, 12h10
Seb33300

Surement un petit bug

Pense à cliquer sur résolu si ça l'est ;)
31/12/2009, 12h30
senderia

un bug? mais que faire alors? car rien ne s'affiche dans ma BDD..
pff je désespère là .. moi qui voulait créer moi même mon tout premier formulaire.. ben c'est raté lol

merci en tout cas pour ton aide.

Bonne fête.
bizz
31/12/2009, 12h34
Seb33300

je parlais d'un bug passagé de IE.

Mais la requête s'affiche maintenant ?
31/12/2009, 12h52
senderia

non, ça me met toujours "Internet Explorer ne peut pas afficher cette page Web " de plus rien ne se met dans la bdd.

j'ai essayé avec mozilla même pb.
31/12/2009, 14h05
Seb33300

Tu peux essayer de commenter mysql_connect et mysql_select_db ?
31/12/2009, 16h00
senderia

commenter?
te dire ce que fais ces deux lignes de commande?
31/12/2009, 22h44
Seb33300

Non, rajoute // devant les lignes
ça permet de ne pas les interpréter.
01/01/2010, 09h05
lapin7
bonjour et bonne année à tous.

moi je débarque ici pendant que tout le monde dort encore (il y en a même qui ronflent) et je lis un peu les questions/réponses, ça fait toujours du bien de s'instruire.

commenter une ligne en php c'est mettre // devant le code

Citation:

alors quand je met message la page où il y a écrit "je vous remercie;.." ne s'affiche plus.. ça me met " Internet Explorer ne peut pas afficher cette page Web "

la page appelée est envoyer.php, elle s'ouvre bien si tu l'appelles directement par l'url ?

C'est quelle erreur qui s'affiche ? détaille le message d'erreur stp, ou bien met une capture d'écran

du côté html, il y a d'innombrables erreurs de syntaxes

les \ devrait être des /

Code:

<img src="Image\R.gif"><img src="Image\BB.gif">

tu mets <FORM et tu fermes </form

en (x)Html toutes les balises sont écrites en minuscules

tu fermes bien tes balises input mais pas celle des images et des br

Code:

<img src="chemin/image.ext" /> <br />

tu mets bien un style à ton h1 mais ton élément font est mal renseigné
15 quoi ? paires de chaussettes ?

remplace :

Code:

<font color="blue" size = 15> Je vous remercie pour le message!! </font>

par :

Code:

<p style="font-size:15px; color: #0000ff;"> je vous remercie pour le message!!</p>

[EDIT] si les champs de la table "message" sont "pseudo, mail, mess" la requête s'écrit :
Code:

1 2 $Req="INSERT INTO message(pseudo, mail, mess) VALUES('$pseudo','$mail','$message')"or die(mysql_error()); mysql_query($Req) or die('Erreur SQL !'.$sql.'<br>'.mysql_error());
le select ici ne sert à rien

Je rajouterais une balise meta entre les balises "head" pour le retour automatique à la page de formulaire genre :

Code:

<meta http-equiv="refresh" content="1; url=formulaire.php">
01/01/2010, 18h43
senderia

bonjour,
dsl pour la réponse tardive mais je n'ai pas pu passer avant avec tous les invités à la maison.
Je repasse demain en début de soirée.
Je testerais ce qui a été mis.
Merci bien!!

Bonne année aussi!!!
01/01/2010, 19h19
senderia

me revoilà, je profite d'un petit moment pour m'eclipser vous mettre ce que je viens de découvrir!!
j'ai une erreur..enfin quelquechose se passe

Citation:

Notice: mysql_connect() [function.mysql-connect]: SQL safe mode in effect - ignoring host/user/password information in C:\wamp\www\C\Livredor\envoyer.php on line 13

si j'ai bien compris je n'arrive pas à me connecter à la bdd?

merci

bizzz
01/01/2010, 19h48
lapin7

clic gauche sur l'icône de WAMP > PHP > Configuration PHP > décocher "safe mode"
01/01/2010, 23h19
senderia

re,
alors c'est bien ce que j'avais compris.. j'ai vu ça sur le net.. merci lapin!!
mais pourquoi avec le safe mode cela ne fonctionne pas?? que signifie ce mode?

encore merci pour tout! :ccool::ccool:

bonne soirée.
Bisous
02/01/2010, 07h24
lapin7

une partie de la réponse :

http://php.developpez.com/faq/?page=ini#ini_safe-mode
02/01/2010, 16h29
senderia

Bonjour,

Pour dire que tout fonctionne à présent. Et j'en suis ravie :P

Par contre juste encore une dernière question. Quand je vais donc dans mon localhost pour afficher mes pages en html ou php sous IE j'ai tout qui s'affiche (images, arrière plan) mais qd j'essaye sous mozilla je n'ai pas les imaes ou mes arrières plan..
est-ce parce que je b'ai pas fait de feuille de style?? (css)

Merci bcp :)
Bizzz
02/01/2010, 18h58
lapin7

le mieux est de faire une feuille de style à part mais on peut très bien l'incorporer dans la page ou bien directement dans la balise concernée

si ton image de fond ne s'affiche pas dans un navigateur c'est que la balise est mal renseignée

exemple :

Code:

<div style="background:#f8f8f8;">bla bla bla</div>

ou avec une image qui se trouve ici dans le dossier images)

Code:

<div style="background: url(images/image_de_fond.jpg);">bla bla bla</div>
03/01/2010, 10h30
senderia

je pense que je vais faire une feuille de style car avec ces balises, je n'ai pas réussi à mettre mon arrière plan..
peut-être que je les utilise mal..
bref.
Je vais tester ça aujourd'hui.

Merci encore.

Salut,

2-3 petits commentaires à la volée :

1)- tu devrais virer ton $sql="SELECT * FROM message";
sinon tu vas afficher tous les messages de ta dB en cas d'erreur, je ne suis pas certain que ça soit le but,

2)- tu devrais ajouter au "my_real_string_escape()" évoqué précédemment un beau htmlspecialchars pour tes variables POST, ça t'évitera les risques d'injection SQL.

Citation:

$message = mysql_real_escape_string(htmlspecialchars($_POST['message']));

3)- Vu le formulaire diabolique, tu peux tout balancer dans une même page, ça t'évitera les redirections et autre refresh en ts genres.

4)- le "safe_mode", il y en a un pour PHP mais aussi un autre global pour SQL. Tous deux sont à paramétrer dans ton fichier de configuration "php.ini". Si tu es sur WAMP, attention, je crois de mémoire qu'il y a plusieurs fichiers php.ini. (apache chroot jail), donc choisis le bon. Etant donnée l'erreur, tu as probablement une ligne du type :

Citation:

[SQL]
sql.safe_mode = On

qu'il te faut alors changer en

Citation:

[SQL]
sql.safe_mode = Off

5)- Quand ça fonctionnera, tu pourras aussi penser à ajouter un contrôle des formats que ton visiteur rentre avec des RegEx. C'est l'utilisation des fonctions preg_match() et preg_match_all(). A savoir pour plus tard.

6)- Le style, ça se fait une fois que tout fonctionne. Et oui, c'est mieux avec des fichiers de css séparés que tu lies dans les déclaration de ton header.

7)- Bon courage,

8)- Allez, un petit exemple sans les regex, juste pour illustrer mes commentaires :
Code:

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 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"> <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" lang="fr"> <head> <title>mon formulaire de la mort qui tue</title> <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" /> </head>  <style type="text/css"> form { text-align:left; } </style> <body> <?php // Traitement des variables $_POST sur la même page (la fameuse "tu_pointes_vers_ta_page_unique_qui_fait_tout.php") // Quand le visiteur aura cliqué sur "envoyer", la page se relancera sur elle-même, les variables $_POST seront alors définies, // et cette partie de code s'executera, donc ta dB sera renseignée if (isset($_POST['pseudo']) AND isset($_POST['message'])) { // Si il y a quelque chose à enregistrer if ($_POST['pseudo'] != NULL AND $_POST['message'] != NULL) { // Connexion à la dB mysql_connect("localhost", "ton_id", "ton_pwd"); $my_connexion= mysql_select_db("ta_base"); // Si ta connexion est bien établie, alors on y va if ($my_connexion) { // Sécurisation des variables reçues avant de les balancer dans la dB $message = mysql_real_escape_string(htmlspecialchars($_POST['message'])); $pseudo = mysql_real_escape_string(htmlspecialchars($_POST['pseudo'])); // etc... ajouter tes autres variables que tu veux récupérer de ton formulaire avec $_POST['valeur_de_name_du_formulaire'] // Là quand tu seras grand, tu pourras ajouter un contrôle plus fin des entrées avec des regex // ... // La requete d'enregistrement dans la dB, // attention, il faut bien mettre les champs dans le même ordre que celui de ta table dans la dB // Dans cet exemple, le premier champ est volontairement vide, car je suppose que la table a un premier champ "autoincremental" mysql_query("INSERT INTO ta_table VALUES('', '$pseudo', '$message')"); // Une fois débuggué, si tu veux éviter les "warnings" potentiels, tu peux ajouter un "@" devant la commande comme ceci // @mysql_query("INSERT INTO ta_table VALUES('', '$pseudo', '$message')"); // Déconnexion de la dB mysql_close(); // Bah voila, le tour est joué :-) } // Sinon, tu peux balancer un message d'erreur qui ne doit te servir qu'au debugage et en rien ne constituer une info pour tes visiteurs hackers else { echo "<br/>désolé, pas de connexion à la dB"; } } } ?>  <form action="tu_pointes_vers_ta_page_unique_qui_fait_tout.php" method="post"> <p> Pseudo : <input type="text" name="pseudo" /><br /> Message : <input type="text" name="message" /><br />  <input type="submit" value="Envoyer" /> </p> </form> </body> </html>