Script PHP - INSERT INTO

**Skunka** · 11/06/2018, 16h31

Bonjour,

je fais face à un problème assez particulier aujourd'hui, je m'explique: sur une page permettant la gestion des salariés de mon entreprise, un bouton "Nouveau" ouvre une modale contenant un formulaire à remplir. Quand tous les champs sont remplis, l'insertion réussie. Mais quand il manque un seul champ, l'insertion ne réussie pas et qu'importe le champ non remplis. Je remplis evidemment les champs obligatoires.

Voici la table:

Code MySQL :

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
 
CREATE TABLE IF NOT EXISTS salarie
 (
   numSalarie INTEGER NOT NULL AUTO_INCREMENT ,
   numNiveau INTEGER NOT NULL  ,
   numPoste INTEGER NOT NULL  ,
   nomSalarie CHAR(32) NOT NULL  ,
   prenomSalarie VARCHAR(128) NOT NULL  ,
   dateFIMO DATE NULL  ,
   dateFCO DATE NULL  ,
   dateValiditeCarteConducteur DATE NULL  ,
   dateValiditeQualificationConducteur DATE NULL  ,
   dateVisiteMedicale DATE NULL  ,
   AIPR VARCHAR(32) NULL  ,
   dateValiditeAIPR DATE NULL  
   , PRIMARY KEY (numSalarie) 
 )

Le formulaire :

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
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
 
<FORM method="POST" action="Scripts/salarie_create.php">
							Nom: <input type="text" name="nomSalarie"> <br>
							Prénom: <input type="te" name="prenomSalarie"> <br>
							* Format des dates: aaaa-mm-jj * <br>
							Prochaine visite médicale: <input type="text" name="dateVisiteMedicale"> <br>
 
							<!--Liste déroulante -->
							Niveau de droit:<select name="droit" style="width: 230px">
												<option value="0"> - - - - - - - - </option>
												<?php
												$req = "SELECT * FROM niveauSalarie ORDER BY numNiveau";
												try{
													$stmt = $pdo->prepare($req);
													$stmt->execute();
													while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
														echo "<option value='".$row["numNiveau"]."'>".$row["nomNiveau"]."</option>";
													}
												}
												catch(Exception $e){
													print "Erreur ! ".$e->getMessage(). "<br>";
												}
												?>
												</select> <br>
 
							<!-- Liste déroulante -->					
							Poste :	<select name="poste" style="width: 230px">
										<option value="0"> - - - - - - - - </option>
										<?php
										$req = "SELECT * FROM posteSalarie ORDER BY nomPoste";
										try{
											$stmt = $pdo->prepare($req);
											$stmt->execute();
											while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
												echo "<option value='".$row["numPoste"]."'>".$row["nomPoste"]."</option>";
											}
										}
										catch(Exception $e){
											print "Erreur !".$e->getMessage()."<br>";										}
										?>
									</select> <br> <br>
							< - - - Zone conducteur / chauffeur - - - > <br>
							Date précédente FCO: <input type="text" name="fimo"> <br>
							Date prochaine FCO: <input type="tex" name="fco"> <br>
							Date fin de validité de <br> la carte conducteur: <input type="text" name="carte"> <br>
							Date fin de validité de <br> la qualification conducteur: <input type="text" name="qualification"> <br>
							AIPR: 	<select name="aipr">
										<option value="0"> - - - - - - - - </option>
										<option value="Concepteur">Concepteur</option>
										<option value="Encadrant">Encadrant</option>
										<option value="Operateur">Opérateur</option>
									</select> <br>
							Date fin de validité AIPR: <input type="text" name="dateAIPR"> <br>
							<input type="submit" value="Confirmer">
						</FORM>

Le script PHP appelé:

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
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
 
<?php 
include "../include/connexion.php";
//Si le nom du salarié est bien indiqué, ainsi que son prénom, son poste et son niveau de droit sur l'application
if (isset($_POST['nomSalarie']) && !empty($_POST['prenomSalarie']) && !empty($_POST['poste']) && !empty($_POST['droit'])) {
	$nom = $_POST['nomSalarie'];
	$prenom = $_POST['prenomSalarie'];
	$poste = $_POST['poste'];
	$niveau = $_POST['droit'];
	//On crée un nouveau salarié
	include "../Classe/salarie.php";
	$unSalarie = new Salarie ($nom, $prenom, $poste, $niveau);
 
	//On le met à jour en fonction des informations rentrées avant de l'enregistrer dans la BDD
	//SI *propriété* est mise, alors appelle de la fonction Setter avec param, SINON on vide l'attribut pour éviter que le nouveau salarié récupère la propriété d'un précédent salarié
	$vide = ""; //Variable pour vider
	if (isset($_POST['dateVisiteMedicale'])) {
		$unSalarie->setDateVisiteMedicale($_POST['dateVisiteMedicale']);
	} else {
		$unSalarie->setDateVisiteMedicale($vide);
	}
	if (isset($_POST['fimo'])) {
		$unSalarie->setDateFIMO($_POST['fimo']);
	} else {
		$unSalarie->setDateFIMO($vide);
	}
	if (isset($_POST['fco'])) {
		$unSalarie->setDateFCO($_POST['fco']);
	} else{
		$unSalarie->setDateFCO($vide);
	}
	if (isset($_POST['carte'])) {
		$unSalarie->setDateValiditeCarteConducteur($_POST['carte']);
	} else {
		$unSalarie->setDateValiditeCarteConducteur($vide);
	}
	if (isset($_POST['qualification'])) {
		$unSalarie->setDateQualificationConducteur($_POST['qualification']);
	} else {
		$unSalarie->setDateQualificationConducteur($vide);
	}
	if (isset($_POST['aipr'])) {
		$unSalarie->setAIPR($_POST['aipr']);
	} else {
		$unSalarie->setAIPR($vide);
	}
	if (isset($_POST['dateAIPR'])) {
		$unSalarie->setDateValiditeAIPR($_POST['dateAIPR']);
	} else {
		$unSalarie->setDateValiditeAIPR($vide);
	}
 
	//Enregistrement dans la BDD
	$unSalarie->create();
 
	//retour à la page salarie
	header("Location: ../page_salarie.php");
} 
?>

ainsi que ma fonction create():

Code PHP :

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

1
2
3
4
5
6
7
8
9
10
11
 
	public function create(){
		include "../include/connexion.php";
		try{
			$req = "INSERT INTO salarie VALUES (NULL,'".$this->_niveauSalarie."','".$this->_posteSalarie."','".$this->_nomSalarie."','".$this->_prenomSalarie."','".$this->_dateFIMO."','".$this->_dateFCO."','".$this->_dateValiditeCarteConducteur."','".$this->_dateValiditeQualificationConducteur."','".$this->_dateVisiteMedicale."','".$this->_AIPR."','".$this->_dateValiditeAIPR."')";
			$pdo->exec($req);
		}
		catch(Exception $e){
			print "Erreur ! ".$e->getMessage(). "<br/>";
		}
	}

Je n'arrive pas à définir ce qui cause l'empêchement de l'insertion dans ma bdd puisque celle ci fonctionne lorsque tous les champs sont remplis et non quand il en manque des non obligatoires...

Merci d'avance pour toutes aides apportées !
Cordialement,
Skunka.

**bonjourajax** · 11/06/2018, 16h49

Bonjour

Une cause du probleme

Code :

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

1
2
 
$req = "INSERT INTO salarie (colonnes) VALUES (NULL,'".$this->_niveauSalarie."','".$this->_posteSalarie."','".$this->_nomSalarie."','".$this->_prenomSalarie."','".$this->_dateFIMO."','".$this->_dateFCO."','".$this->_dateValiditeCarteConducteur."','".$this->_dateValiditeQualificationConducteur."','".$this->_dateVisiteMedicale."','".$this->_AIPR."','".$this->_dateValiditeAIPR."')";

mettre les colonnes désignées apres salarie

Il y a un message d'erreur?

**Celira** · 11/06/2018, 17h09

Déjà, utilise plutôt une requête préparée, vu que tu insères des données extérieures.

Quand tu dis "il manque un champ", je suppose que ça veut dire "quand un champ n'est pas rempli dans le formulaire". Le problème se pose avec n'importe quel champ ? Tu as un message d'erreur à nous montrer ?

Invité · 11/06/2018, 17h18

Bonjour,

1- que donne : echo $req; ?

2- Pourquoi toute la gestion des $unSalarie->set...... n'est pas directement dans la fonction create (en passant le $_POST en paramètre) ?
Je ne maitrise pas la POO, mais ça me semble étrange...

Invité · 12/06/2018, 10h29

Bonjour,

1-

....J'ai renseigné les colonnes à remplir avec une syntaxe normalement correcte:...

Non.

Il faut imperativement une requête préparée

2- "NULL" / 'NULL' est une chaîne de caractères. Ce n'est pas NULL.
Remets $vide = "";.

3- optimisation
Remplace tous les

Code :

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

1
2
3
4
5
	if (isset($_POST['dateVisiteMedicale'])) {
		$unSalarie->setDateVisiteMedicale($_POST['dateVisiteMedicale']);
	} else {
		$unSalarie->setDateVisiteMedicale($vide);
	}

Par des

Code :

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

1
2
	$dateVisiteMedicale = ( isset($_POST['dateVisiteMedicale']) ) ? $_POST['dateVisiteMedicale'] : $vide; // (condition ternaire)
	$unSalarie->setDateVisiteMedicale( $dateVisiteMedicale );

4- Quant au message d'erreur, il indique la ligne 169.
Quel est le code correspondant (cette ligne 169 et précédentes) ?

**Skunka** · 11/06/2018, 17h23

Envoyé par bonjourajax

Bonjour

Une cause du probleme

Code :

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

1
2
 
$req = "INSERT INTO salarie (colonnes) VALUES (NULL,'".$this->_niveauSalarie."','".$this->_posteSalarie."','".$this->_nomSalarie."','".$this->_prenomSalarie."','".$this->_dateFIMO."','".$this->_dateFCO."','".$this->_dateValiditeCarteConducteur."','".$this->_dateValiditeQualificationConducteur."','".$this->_dateVisiteMedicale."','".$this->_AIPR."','".$this->_dateValiditeAIPR."')";

mettre les colonnes désignées apres salarie

Il y a un message d'erreur?

Il me semble inutile d'indiquer les colonnes puisque la requête les traite toutes dans l'ordre présent dans la BDD (c'est en tout cas ce que m'ont appris mes professeurs

)
C'est bien pour ça que je cherche de l'aide, je n'ai aucun message d'erreur qui s'affiche (il se peut que j'ai oublié de traiter cela dans le code...) et le script appelé me ramène bien sur ma page.
Tu as vu juste Celira, je parlais bien d'un champ du formulaire non remplis.

Si vous avez la solution me permettant d'afficher le message d'erreur, j'en serais ravis, cela m'aidera sûrement!
Sinon, toutes autres propositions sont la bienvenue!

Edit pour jreaux62 qui m'écris pendant que j'écrivais:
où place tu ton echo $req? Dans la fonction create() j'imagine ?
pour les "set" ton explication est légère mais je pense comprendre, j'essaye cela dès demain!

Cordialement,
Skunka.

PS: ma journée se termine dans 5 minutes, je suis de retour demain matin ! :p

**Celira** · 11/06/2018, 17h58

Il n'est pas nécessaire d'écrire les colonnes dans une requête INSERT si on les alimente toutes, mais il est utile d'écrire les colonnes dans une requête INSERT. C'est comme le SELECT * : il vaut toujours mieux utiliser des requêtes explicites.Dans le cas d'une requête INSERT, ça permet d'être sûr de l'ordre dans lequel elles sont alimentées sans avoir à se référer à la base toutes les 3 minutes, ça évite (entre autres) d'avoir à repasser sur le code si tu ajoutes une colonne facultative à ta table et surtout ça évite les mauvaises surprises du genre "Hop ! On réinitialise la base et oh ! dans le nouveau script les colonnes sont par ordre alphabétique... Oups..."

Si tu veux des messages d'erreur avec PDO, il faut penser à activer les exceptions au niveau de la connexion :

Code :

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

1
2
$pdo = new PDO('...');
$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

@jreaux : Concernant la POO, le fait qu'il crée à partir de l'objet me parait valable : On construit un Salarie (comme on construit un tableau de données) et ensuite on le sauvegarde en base.
En revanche, ce qui m'embête, c'est que la méthode create est apparemment dans la classe Salarie, alors que Salarie est visiblement un objet métier (la représentation d'un salarié). Pour moi, il faut 2 classes : Salarie qui représente le salarié, avec nom, prénom, etc...) et SalarieDao (ou SalarieManager ou quelque chose qui genre) qui s'occupe de la communication avec la base.

**Skunka** · 12/06/2018, 09h36

J'ai renseigné les colonnes à remplir avec une syntaxe normalement correcte:

Code PHP :

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

1
2
 
$req = "INSERT INTO salarie (numSalarie, numNiveau, numPoste, nomSalarie, prenomSalarie, dateFIMO, dateFCO, dateValiditeCarteConducteur, dateValiditeQualificationConducteur, dateVisiteMedicale, AIPR, dateValiditeAIPR) VALUES (NULL,'".$this->_niveauSalarie."','".$this->_posteSalarie."','".$this->_nomSalarie."','".$this->_prenomSalarie."','".$this->_dateFIMO."','".$this->_dateFCO."','".$this->_dateValiditeCarteConducteur."','".$this->_dateValiditeQualificationConducteur."','".$this->_dateVisiteMedicale."','".$this->_AIPR."','".$this->_dateValiditeAIPR."')";

Egalement modifié un peu le code au niveau de la connexion pour activer les exceptions:

Code PHP :

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

1
2
3
4
5
6
7
8
9
10
 
<?php
try{
	$pdo = new PDO('mysql:host=localhost;dbname=ttpm;charset=utf8', 'root', '');
	$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
} 
catch(Exception $e){
	die('Erreur :'.$e->getMessage());
}
?>

J'avais déjà un catch(Exception), j'ai ensuite ré effectué plusieurs essaies d'insertion et le résultat reste le même: l'insertion fonctionne quand tous les champs du formulaire sont renseignés mais ne fonctionne pas quand au moins un champ du formulaire n'est pas renseigné. J'ai même essayé de laisser un champ obligatoire en blanc mais je ne récupère toujours pas de message d'erreur ...
Dans mon script salarie_create.php , j'en suis venu à modifier le contenu de ma variable $vide par $vide = "NULL" pour que, lors de l'insertion, mon SGBD reçoivent une valeur NULL plutôt que vide mais cela n'a pas aidé :/

Ensuite j'ai entrepris cette modification du script salarie_create.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
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
 
<?php 
include "../include/connexion.php";
//Si le nom du salarié est bien indiqué, ainsi que son prénom, son poste et son niveau de droit sur l'application
if (isset($_POST['nomSalarie']) && !empty($_POST['prenomSalarie']) && !empty($_POST['poste']) && !empty($_POST['droit'])) {
	$nom = $_POST['nomSalarie'];
	$prenom = $_POST['prenomSalarie'];
	$poste = $_POST['poste'];
	$niveau = $_POST['droit'];
	//On crée un nouveau salarié
	include "../Classe/salarie.php";
	$unSalarie = new Salarie ($nom, $prenom, $poste, $niveau);
	//Enregistrement dans la BDD
	$unSalarie->create();
 
	//On récupère le numéro du salarié créer pour le mettre à jour
	$req = "SELECT numSalarie FROM salarie WHERE nomSalarie = ".$nom." AND prenomSalarie = ".$prenom;
	try{
		$stmt = $pdo->prepare($req);
		$stmt->execute();
	}
	catch(Exception $e){
		print "Erreur ! ".$e->getMessage(). "<br/>";
	}	
 
	//On le met à jour en fonction des informations rentrées avant de l'enregistrer dans la BDD
	//SI *propriété* est mise, alors appelle de la fonction Setter avec param, SINON on vide l'attribut pour éviter que le nouveau salarié récupère la propriété d'un précédent salarié
	$vide = ""; //Variable pour indiquer un champ non renseigné
	if (isset($_POST['dateVisiteMedicale'])) {
		$unSalarie->setDateVisiteMedicale($_POST['dateVisiteMedicale']);
	} else {
		$unSalarie->setDateVisiteMedicale($vide);
	}
 
	if (isset($_POST['fimo'])) {
		$unSalarie->setDateFIMO($_POST['fimo']);
	} else {
		$unSalarie->setDateFIMO($vide);
	}
 
	if (isset($_POST['fco'])) {
		$unSalarie->setDateFCO($_POST['fco']);
	} else{
		$unSalarie->setDateFCO($vide);
	}
 
	if (isset($_POST['carte'])) {
		$unSalarie->setDateValiditeCarteConducteur($_POST['carte']);
	} else {
		$unSalarie->setDateValiditeCarteConducteur($vide);
	}
 
	if (isset($_POST['qualification'])) {
		$unSalarie->setDateQualificationConducteur($_POST['qualification']);
	} else {
		$unSalarie->setDateQualificationConducteur($vide);
	}
 
	if (isset($_POST['aipr'])) {
		$unSalarie->setAIPR($_POST['aipr']);
	} else {
		$unSalarie->setAIPR($vide);
	}
 
	if (isset($_POST['dateAIPR'])) {
		$unSalarie->setDateValiditeAIPR($_POST['dateAIPR']);
	} else {
		$unSalarie->setDateValiditeAIPR($vide);
	}
 
	$unSalarie->update($stmt);
}
 
//retour à la page salarie
header("Location: ../page_salarie.php");
 
?>

Je créer plus tôt le salarié puis je fait appelle à sa méthode Update pour mettre à jour le salarié fraîchement crée pour obtenir ce message d'erreur:

Nom : Capture.JPG
Affichages : 2333
Taille : 102,8 Ko

Nom : Capture.JPG
Affichages : 2333
Taille : 102,8 Ko

Je constate que le problème initial est donc la syntaxe de la requête INSERT. Ferais-je mieux de partir sur deux méthodes "create()"? Car, dans mon contexte, il existe deux types de salariés: les chauffeurs qui possède les propriétés de date et les autres. Avoir un createChauffeur() qui prend en compte une syntaxe approprié pour les dates et un create() qui renseigne uniquement les propriétés non daté, valables pour tous les salariés ?

Pour moi, il faut 2 classes : Salarie qui représente le salarié, avec nom, prénom, etc...) et SalarieDao (ou SalarieManager ou quelque chose qui genre) qui s'occupe de la communication avec la base.

Je ne vois pas très bien l'utilité de cette seconde classe ? SalarieDao serait la classe qui permet l’interaction avec la BDD de la classe salarie ?