insert si vide et update sinon

**laurentSc** · 11/12/2013, 17h05

Bonjour,

je voudrais mettre à jour une bdd de membres : si le membre est nouveau, faire un insert et sinon un update, donc je commence par compter le nombre de membres portant le nom et le prénom donnés. Mais mon code doit être faux car il devrait renvoyer 0 alors qu'il renvoit 4 (le nombre de champs de la table). Il est où mon problème ?

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
	$connexion=connect();
	$sql="SELECT * from `patient` WHERE (nom=:nom AND prenom=:prenom)";
	$prep = $connexion->prepare($sql);
	//Associer des valeurs aux place holders
	$prep->bindValue(':nom', $_POST['nom'], PDO::PARAM_STR);
	$prep->bindValue(':prenom', $_POST['prenom'], PDO::PARAM_STR);	
	$prep->execute();
	$colcount = $prep->columnCount();
	echo "colcount=".$colcount."<br/>";
	if ($colcount==0)	
			$sql="INSERT INTO `patient` (nom,prenom,date_naissance) VALUES (:nom,:prenom,:champ_date)";
	else
			$sql="UPDATE `patient` SET nom=:nom,prenom=:prenom,date_naissance=:champ_date";
 
	$prep = $connexion->prepare($sql);
 
	//Associer des valeurs aux place holders
	$prep->bindValue(':nom', $_POST['nom'], PDO::PARAM_STR);
	$prep->bindValue(':prenom', $_POST['prenom'], PDO::PARAM_STR);
	$prep->bindValue(':champ_date', setFrenchDate($_POST['date_naissance']), PDO::PARAM_STR);
 
	//Compiler et exécuter la requête
	$prep->execute();

**ABCIWEB** · 11/12/2013, 17h13

Avec mysql on peut faire une unique requête avec ON DUPLICATE KEY UPDATE.

EDIT et pour l'erreur, mysql te donne le résultat que tu lui demande (columnCount retourne le nombre de colonnes sélectionnées).

**ABCIWEB** · 11/12/2013, 17h25

A part cela je vois une erreur de conception dans ton code car deux patients peuvent avoir un même prénom et un même nom. Il faut utiliser un identifiant unique pour les distinguer (ex : numéro de secu).

**laurentSc** · 11/12/2013, 17h44

OK pour l'erreur de conception (je m'en occuperai plus tard), mais pour l'instant j'essaye d'utiliser ON DUPLICATE KEY UPDATE. Il est pas d'accord avec mon code :

ERREUR PDO dans C:\wamp\www\petit_site\action_patient.php L.28 : SQLSTATE[HY093]: Invalid parameter number

Où me suis-je tromper ?

Code php :

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

$sql="INSERT INTO `patient` (nom,prenom,date_naissance) VALUES (:nom,:prenom,:champ_date) ON DUPLICATE KEY UPDATE nom=:nom,prenom=:prenom,date_naissance=:champ_date";

**Bovino** · 11/12/2013, 17h59

je m'en occuperai plus tard

Ah... ben alors bon courage pour déterminer, entre nom, prenom et date de naissance lequel sera la clé unique !

**sabotage** · 11/12/2013, 18h06

Tu ne peux pas avoir deux paramètres avec le même nom

Code sql :

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

INSERT INTO `patient` (nom,prenom,date_naissance) VALUES (:nom,:prenom,:date) ON DUPLICATE KEY UPDATE nom=:nom_update, prenom=:prenom_update, date_naissance=:date_update

**ABCIWEB** · 11/12/2013, 18h11

Sur le principe

Code :

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

1
2
3
4
 "...ON DUPLICATE KEY UPDATE 
			nom = VALUES(nom)
			,prenom = VALUES(prenom)
			,date_naissance = VALUES(date_naissance)";

Mais pour que ON DUPLICATE KEY UPDATE fonctionne correctement il faudrait définir dans ta table une clé unique éventuellement constituée par la combinaison de plusieurs colonnes.

**Bovino** · 11/12/2013, 18h50

À vrai dire, j'ai surtout du mal à comprendre pourquoi tu veux faire un UPDATE...

À part les cas rares (et que tu ne sembles d'ailleurs pas chercher) d'une femme qui s'est mariée, je vois pas comment une personne peut changer de nom, de prénom ou de date de naissance... mais dans ce cas là, ce toujours un INSERT qu'il faut faire...

**laurentSc** · 11/12/2013, 18h52

Ca me rappelle quelque chose car une fois j'avais déjà utilisé cela, mais j'ai plus rien dessus ; tu peux préciser cette histoire de clé unique ?

Et sinon, merci Sabotage ; j'avais compris ta remarque sans lire ton code ...et :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
	$sql="INSERT INTO `patient` (nom,prenom,date_naissance) VALUES (:nom,:prenom,:champ_date) ON DUPLICATE KEY UPDATE nom=:nom2,prenom=:prenom2,date_naissance=:champ_date2;";
 
	$prep = $connexion->prepare($sql);
 
	//Associer des valeurs aux place holders
	$prep->bindValue(':nom', $_POST['nom'], PDO::PARAM_STR);
	$prep->bindValue(':prenom', $_POST['prenom'], PDO::PARAM_STR);
	$prep->bindValue(':champ_date', setFrenchDate($_POST['date_naissance']), PDO::PARAM_STR);
	$prep->bindValue(':nom2', $_POST['nom'], PDO::PARAM_STR);
	$prep->bindValue(':prenom2', $_POST['prenom'], PDO::PARAM_STR);
	$prep->bindValue(':champ_date2', setFrenchDate($_POST['date_naissance']), PDO::PARAM_STR);

**laurentSc** · 11/12/2013, 19h08

Envoyé par Bovino

À vrai dire, j'ai surtout du mal à comprendre pourquoi tu veux faire un UPDATE...

À part les cas rares (et que tu ne sembles d'ailleurs pas chercher) d'une femme qui s'est mariée, je vois pas comment une personne peut changer de nom, de prénom ou de date de naissance... mais dans ce cas là, ce toujours un INSERT qu'il faut faire...

Mais moi par exemple, j'ai changé 3 ou 4 fois de date de naissance...

.Effectivement le update, c'est idiot ; ce que je voulais éviter, c'est faire plusieurs fois un insert de la même personne (mais c'est vrai qu'il faut tenir compte du cas de 2 personnes homonymes...)

**Bovino** · 11/12/2013, 19h23

tu peux préciser cette histoire de clé unique ?

Ben pour que MySQL puisse déterminer qu'il y a violation de contrainte d'unicité, il faut bien se baser sur une clé unique (sinon la notion de DUPLICATE KEY perd une grande partie de son sens).

Or dans ton cas, il me semble qu'aucun champ ni aucune combinaison de champs ne permet de déterminer à coup sûr une unicité. Parce que rien n'interdit à deux homonymes d'être nés le même jour, soit tous les champs dont tu parles identiques.

**laurentSc** · 11/12/2013, 19h47

Supposons (c'est pas encore fait, mais ça semble logique) que dans la table `patient` je rajoute un champ num_secu est-ce que du coup cette requête irait ?

Code sql :

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

INSERT INTO `patient` (nom,prenom,date_naissance,num_secu) VALUES (:nom,:prenom,:champ_date,:num_secu) ON DUPLICATE KEY UPDATE nom=:nom2,prenom=:prenom2,date_naissance=:champ_date2,num_secu=:num_secu2;

**sabotage** · 11/12/2013, 19h57

Tu ne viens pas de dire qu'effectivement faire un UPDATE n'avait pas de sens ?

**laurentSc** · 11/12/2013, 20h22

Oui, c'est vrai et je me trouve un peu nul, mais cela dit, ce nouveau champ (num_secu) me semble nécessaire vu que je veux éviter plusieurs "insert" de la même personne et que comme le remarquait Bovino, il est possible que 2 personnes du même nom soient nées le même jour...Mais le update, il aurait pas son utilité si par exemple seulement quelques champs étaient saisis et un autre plus tard (quoique ça reste de l'insert (je réfléchis en écrivant)). Et je me dis que ce champ num_secu ne serait pas obligatoire, mais devrait être saisi si un homonyme apparaissait. Mais (je réfléchis encore), il devra être possible de sélectionner un patient déjà saisi par une liste déroulante, et comme je ne me vois pas y proposer le numéro de sécu, pour distinguer 2 homonymes, on pourrait rajouter dans ce cas un numéro au prénom (exemple : Jean1, Jean2...). Qu'en pensez-vous ?

**ABCIWEB** · 11/12/2013, 21h19

1 / Oui "num_secu" est indispensable et doit être la clé unique ou primaire. Ta priorité est d'intégrer ce champ dans tes tables avant d'aller plus loin.

2 / Pour trouver facilement un client dans ta table (pour consultation ou faire un update etc.) tu peux par exemple faire une requête ajax avec jquery et autocomplète sur le nom. Cela te proposera une liste déroulante en retour dans laquelle tu pourras faire un choix. Mais dans cette liste il faudra afficher aussi à côté du nom, le numéro sécu et éventuellement le prénom et la date de naissance pour pouvoir faire rapidement un choix.
Sinon je vois pas comment on pourrait choisir entre jean1, jean2, ça correspond à rien

**laurentSc** · 11/12/2013, 21h30

Oui, t'as raison, ça correspond à rien. mais je me dis que ce champ num_secu pourrait ne pas être obligatoire, sauf si un nom identique existe déjà et donc du coup, il ne serait que très rarement à saisir : est-ce correct ?

**ABCIWEB** · 11/12/2013, 21h45

Totalement incorrect

Ce doit être un champ obligatoire dans ta table. Si tu réfléchis deux secondes un champ unique ne peux pas être facultatif sinon il pourrait être vide et donc équivalent dans deux lignes distinctes pour lesquelles on ne l'a pas renseigné.
Une clé unique ou une clé primaire sert précisément à pouvoir faire une distinction non équivoque entre deux lignes. Elle doit donc être obligatoirement renseignée et évidemment non vide sinon mysql va te répondre

**laurentSc** · 11/12/2013, 22h02

Excuse-moi d'avoir du mal à me convaincre, mais ce qui peut rendre les lignes distinctes, c'est la combinaison des 3 champs (nom ET prenom ET num_secu) et si y a besoin d'un champ, on pourrait en rajouter un de type hidden et qui serait par exemple la concaténation des 3 premiers ; ce champ pourrait servir de clé primaire et ça simplifierait la plupart du temps la saisie : est-ce jouable ?

**ABCIWEB** · 11/12/2013, 23h30

Oui c'est possible... mais cela ne va pas te faciliter les choses. Dans ce cas tu peux créer dans ta table une clé primaire ou unique composée de ces trois colonnes.
Je vois pas trop l'histoire du champ de type hidden par contre.

**laurentSc** · 12/12/2013, 00h05

Envoyé par ABCIWEB

Oui c'est possible... mais cela ne va pas te faciliter les choses.

OK, mais le but de ce site, c'est justement de simplifier la vie des utilisateurs (pas la mienne). Aujourd'hui, ils le font avec un crayon et un papier et le but est de créer un truc où y aurait le moins de saisie possible (et rassure-toi, même si en phase de mise au point je mets ça sur internet, au final, ça doit tourner sur réseau local pour la confidentialité médicale).
Sinon (et comme j'y croyais, je viens de commencer à le coder), l'idée du champ hidden, c'est de rajouter un 6e champ à la table :

Code sql :

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

1
2
3
4
5
6
7
8
9
CREATE TABLE IF NOT EXISTS `patient` (
  `id_medecin` mediumint(8) unsigned NOT NULL AUTO_INCREMENT,
  `nom` varchar(20) DEFAULT NULL,
  `prenom` varchar(15) DEFAULT NULL,
  `date_naissance` varchar(30) DEFAULT NULL,
  `num_secu` int(15) DEFAULT NULL,
  `homonyme` varchar(50) DEFAULT NULL,
  PRIMARY KEY (`homonyme`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=5 ;

qui par l'action du formulaire serait créé comme étant la concaténation de nom.prenom.(char)num_secu (j'abrège la syntaxe pour me faire comprendre) et qui servirait de clé primaire au formulaire. Et l'intérêt de ce champ supplémentaire, c'est qu'il serait forcément unique puisque s'il y a répétition du nom et prénom, y aura forcément le numéro de sécu pour faire la différence.
Ca colle ?

Au fait, merci de m'aider.

insert si vide et update sinon [PDO]

PHP & Base de données

Discussions similaires

Partager

Partager