non-récupération des données d'un formulaire

**laurentSc** · 06/12/2013, 00h49

Bonsoir,

je fais un formulaire qui me semble très simple et pourtant pas moyen d'en récupérer les données :

Code html :

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

1
2
3
4
5
6
7
<form action="action_patient.php" type="post">
<table>
<tr><td>Nom : </td><td><input type="text" name="nom" size="20"></td></tr>
<tr><td>Prénom : </td><td><input type="text" name="prenom" size="20"></td></tr>
<tr><td></td><td><input type="submit" value="envoyer"></td></tr>
</table>
</form>

action_patient.php

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
    <?php
 
$PARAM_hote='----'; // le chemin vers le serveur
$PARAM_nom_bd='----'; // le nom de votre base de données
$PARAM_utilisateur='----'; // nom d'utilisateur pour se connecter
$PARAM_mot_passe='----'; // mot de passe de l'utilisateur pour se connecter
$connexion = new PDO('mysql:host='.$PARAM_hote.';dbname='.$PARAM_nom_bd, $PARAM_utilisateur, $PARAM_mot_passe);	
$sql="INSERT INTO `patient` (nom,prenom) VALUES (".$_POST['nom'].",".$_POST['prenom'].")";
echo "sql=".$sql."<br/>";
$connexion->exec($sql);
	?>

Dans le sql, dans VALUES, y a rien...
c'est quoi le bug gros comme une maison ?

**armel18** · 06/12/2013, 01h09

salut,
ton formulaire ne transmet pas les données en POST car à la place de l'attribut method du form tu as mis type

Code html :

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

1
2
3
4
5
6
7
<form action="action_patient.php" method="post">
<table>
<tr><td>Nom : </td><td><input type="text" name="nom" size="20"></td></tr>
<tr><td>Prénom : </td><td><input type="text" name="prenom" size="20"></td></tr>
<tr><td></td><td><input type="submit" name="envoyer" value="envoyer"></td></tr>
</table>
</form>

ce qu'il faut toujours faire:
-effectuer une vérification sur les données reçues
-activer la gestion d'erreurs pdo
-Échapper les données reçues

**laurentSc** · 06/12/2013, 01h52

J'avais raison de parler de gros bug ; en fait, il y en avait 2 ("type" au lieu de "method" et données non échappées) mais il doit y en avoir un 3e car l'action, erreur 500... (pas trouvée)
(et dis-moi si la gestion des erreurs de pdo est correcte) :

Code html :

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
38
39
40
41
42
43
44
45
46
47
48
49
<?php
header('Content-type: text/html; charset=UTF-8');
session_start();
?>
<!DOCTYPE html>
<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="content-type">
    <title></title>
	    <meta content="text/html; charset=utf-8" http-equiv="content-type">
    <title></title>
 <link rel="stylesheet" href="http://code.jquery.com/ui/1.10.3/themes/smoothness/jquery-ui.css">
 
  <script src="http://code.jquery.com/jquery-1.9.1.js"></script>
 
  <script src="http://code.jquery.com/ui/1.10.3/jquery-ui.js"></script>
 
  <link rel="stylesheet" href="/resources/demos/style.css">
 
  <script>
 
  $(function() {
 
    $( "#datepicker" ).datepicker({
 
      changeMonth: true,
 
      changeYear: true
 
    });
 
  });
 
  </script>	
  </head>
  <body>
 
<?php include("menu.html"); ?>
<form action="action_patient.php" method="post">
<table>
<tr><td>Nom : </td><td><input type="text" name="nom" size="20"></td></tr>
<tr><td>Prénom : </td><td><input type="text" name="prenom" size="20"></td></tr>
<tr><td>Date naissance : </td><td><input type="text" name="date_naissance" id="datepicker"></td></tr>
<tr><td></td><td><input type="submit" value="envoyer"></td></tr>
</table>
</form>
 
  </body>
</html>

action_patient.php :

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
15
16
17
18
19
20
21
22
23
24
25
26
27
28
    <?php
 function setFrenchDate($date) {
		$dateintermediaire=date_parse_from_format("d/m/Y",$date); //strptime marche pas
		$day=$dateintermediaire['day'];
		$month=$dateintermediaire['month'];
		$year=$dateintermediaire['year'];
		return($year."-".$month."-".$day);//conversion fr->ISO (format bdd)
 
	} 	
 
try {
$PARAM_hote='---'; // le chemin vers le serveur
$PARAM_nom_bd='---'; // le nom de votre base de données
$PARAM_utilisateur='---'; // nom d'utilisateur pour se connecter
$PARAM_mot_passe='---'; // mot de passe de l'utilisateur pour se connecter
$connexion = new PDO('mysql:host='.$PARAM_hote.';dbname='.$PARAM_nom_bd, $PARAM_utilisateur, $PARAM_mot_passe);	
$sql="INSERT INTO `patient` (nom,prenom,date_naissance) VALUES ("\"".$_POST['nom']."\",\"".$_POST['prenom']."\",\"".setFrenchDate($_POST['date_naissance'])."\")";
//$sql="INSERT INTO `patient` (nom,prenom) VALUES (".$_POST['nom'].",".$_POST['prenom'].")";
echo "sql=".$sql."<br/>";
$connexion->exec($sql);
}
 
catch(PDOException $e) {
    $msg = 'ERREUR PDO dans ' . $e->getFile() . ' L.' . $e->getLine() . ' : ' . $e->getMessage();
    die($msg);
}
 
	?>

(je n'ai pas resimplifié le code)

**Celira** · 06/12/2013, 10h13

Question bête : les deux fichiers sont bien dans le même dossier ?

Au passage, il faudrait échapper les données dans ta requête en utilisant PDO::quote (ou en passant par une requête préparée)

Invité · 06/12/2013, 10h39

Bonjour Laurent,

là tu fais vraiment peur !

mauvaise utilisation de <table>,
pas de vérification des données transmises ni de gestion d'erreur,
requête non-sécurisée,
...

C'est une vraie PASSOIRE que tu nous proposes là !

A LIRE ABSOLUMENT :

**Bovino** · 06/12/2013, 10h44

Code :

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

"INSERT INTO `patient` (nom,prenom,date_naissance) VALUES ("\""

non-récupération des données d'un formulaire

Langage PHP

Vue hybride

Discussions similaires

Partager

Partager