faire correctement un update

Version imprimable

04/10/2017, 19h42
laurentSc

faire correctement un update
Bonjour,
je voudrais copier un enregistrement de ma table afin que l'utilisateur qui souhaite en créer un nouveau puisse commencer par en dupliquer un avant de le modifier (comme 95% des données seront déjà bonnes, il gagnera du temps).
Donc je fais la requête suivante :
Code:

1 2 $sql='INSERT INTO `machines` (name,localization,pays) SELECT name,localization,pays FROM `machines` where name="'.$machselcopy.'"'; $qid->exec();
Il y a bien plus que ces 3 colonnes mais je complèterai quand ça sera au point. Mais la difficulté, c'est que "name" est une clé unique : le dump de la table donne :

Citation:

ALTER TABLE `machines`
ADD PRIMARY KEY (`id`),
ADD UNIQUE KEY `name` (`name`);

de ce fait, j'ai le message d'erreur

Citation:

Integrity constraint violation: 1062 Duplicata du champ 'ega' pour la clef 'name''

comment éviter cela ?
04/10/2017, 20h02
sabotage

Met une autre valeur pour le "name" pardis.
04/10/2017, 22h04
laurentSc

En effet, c'est plutôt logique ! Mais du coup, j'ai un petit problème pour faire ça : j'envisage de faire ma requête insert avec toutes les clés sauf name et id (primary key) suivi de 2 update. Mais pour ces 2 update, j'ai besoin d'identifier la ligne avec une clause where. Or les 2 clés qui permettraient de l'identifier ne sont pas encore définies. Donc du coup, y aurait-il moyen de faire mon insert initial avec toutes les clés et dans ce cas, comment mettre une valeur différente pour name et id, vu qu'avec la requête actuelle, je les copie ?
04/10/2017, 22h54
sabotage

Une fois que tu as fais l'insert, les deux clés sont définies et donc tu peux faire l'update.
05/10/2017, 06h14
laurentSc

Ok mais pour faire un update il faut bien dire sur quelle ligne (donc avec une clause where) et comme ces 2 clés ne sont pas encore renseignées je ne vois pas...
05/10/2017, 09h11
sabotage

Tu vas devoir fournir un "name" pour faire ton insertion donc tu as bien la référence non ?
Mais la bonne méthode c'est d'utiliser lastInsertId pour avoir l'id généré par l'insert.
http://php.net/manual/fr/pdo.lastinsertid.php

Le nouvel enregistrement étant juste une copie d'un enregistrement existant, pas de référence. Sinon, j'ai regardé lastinsertid mais pas compris ce qu'il fallait passer en argument à la fonction (et pas d'exemple d'utilisation).
Par contre, j'ai cherché comment obtenir le dernier enregistrement et voici comment je fais :
Code:

1 2 3 4 5 6 $sql='SELECT id from `machines` ORDER BY id desc LIMIT 1'; $qid = $bdd->query($sql); $row=$qid->fetch(PDO::FETCH_OBJ); $id_copy=$row->id; $sql='update `machines` SET name="'.$name_copy.'" where id="'.$id_copy.'"'; $bdd->query($sql);
La colonne id est de type smallint. Je suis étonné car même si cette colonne n'est pas déclarée comme auto_increment, elle l'est. Normal ?

05/10/2017, 14h42
Celira

Essayons de faire les choses sous un autre angle. Tu dupliques ta ligne directement en base et ensuite tu la modifies. Quel genre de modification ? est-ce que par hasard il n'y aurait pas le nouveau name dans cette modification ?

Sinon, tu peux inventer un nouveau name de la forme "Copie de (ancien name)" ou quelque chose du genre. Bon évidement si tu copies 2 fois la même ligne, tu vas juste reporter le problème, donc il faut que soit un utilisateur, soit un programme mette à jour le champ name pour une vraie valeur.
05/10/2017, 14h47
sabotage

Citation:

j'ai cherché comment obtenir le dernier enregistrement et voici comment je fais

Ce n'est pas la bonne façon.
Il faut utiliser lastInsertId et il n'y a pas necessité de paramètre.
05/10/2017, 15h27
laurentSc

En effet $id_copy=$bdd->lastinsertid(); marche très bien.

Et sinon Celira, je duplique bien la ligne à part 2 colonnes (name=unique _key et id=primary_key) et pour ces 2 colonnes, pour name, je fais un update où je modfie "ancien_nom" par "ancien_nom_copy" (donc à charge de l'utilisateur de mettre le vrai nom) et id est auto-incrémentée (alors que cette clé n'est pas déclarée en auto_increment ; donc comment ça se fait ?)

clé primaire dupliquée

Bonsoir,

mon souci est que j'ai un message d'erreur comme quoi la clé primaire est dupliquée

Citation:

Integrity constraint violation: 1062 Duplicata du champ '' pour la clef 'PRIMARY'' in C:\wamp\www\eis2\eisnew\page\copyserver.php on line 70

alors qu'elle ne l'est pas :

Code:

'INSERT INTO `INFOAPPLI` (`autre`,`cadence`,`coda`,`ftpsccm`,`genesis`,`licenseserver`,`loadpoint`,`mentor`,`moldflow`,`pcnetlink`,`pec`,`prointralink`,`promecanica`,`samba`,`symphony`,`teamforge`,`em-plant`,`mdm`) SELECT `autre`,`cadence`,`coda`,`ftpsccm`,`genesis`,`licenseserver`,`loadpoint`,`mentor`,`moldflow`,`pcnetlink`,`pec`,`prointralink`,`promecanica`,`samba`,`symphony`,`teamforge`,`em-plant`,`mdm` FROM `INFOAPPLI` where name="DBMAP"

et
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 CREATE TABLE IF NOT EXISTS `infoappli` ( `name` varchar(20) COLLATE latin1_bin NOT NULL DEFAULT '', `pec` tinyint(4) DEFAULT NULL, `autre` tinyint(4) DEFAULT NULL, `cadence` tinyint(4) DEFAULT NULL, `genesis` tinyint(4) DEFAULT NULL, `licenseserver` tinyint(4) DEFAULT NULL, `teamforge` tinyint(4) DEFAULT NULL, `mentor` tinyint(4) DEFAULT NULL, `em-plant` tinyint(4) DEFAULT NULL, `moldflow` tinyint(4) DEFAULT NULL, `pcnetlink` tinyint(4) DEFAULT NULL, `loadpoint` tinyint(4) DEFAULT NULL, `prointralink` tinyint(4) DEFAULT NULL, `promecanica` tinyint(4) DEFAULT NULL, `mdm` tinyint(4) DEFAULT NULL, `samba` tinyint(4) DEFAULT NULL, `ftpsccm` tinyint(4) DEFAULT NULL, `symphony` tinyint(4) DEFAULT NULL, `coda` tinyint(4) DEFAULT NULL, PRIMARY KEY (`name`), UNIQUE KEY `name` (`name`) ) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_bin;
donc quel est le bug ?

06/10/2017, 23h28
sabotage

"duplicata" c'est un doublon.
Etant donné que tu n'as pas donné la valeur de "name" ça ne serait pas surprenant.
07/10/2017, 14h49
laurentSc
Je continue cette discussion même si le titre n'est plus bon...

OK, je commence par définir le nouveau nom :

Code:

$query='INSERT INTO `infoappli` (name) VALUES("'.$name_copy.'")';

Par contre ma méthode pour ensuite déclarer les autres colonnes ne convient pas.
Code:

1 2 $query='update `infoappli` SET pec=(SELECT pec from `infoappli` where name="'.$machselcopy.'") where name="'.$name_copy.'"'; $qid = $bdd->query($query);//essai avec une seule colonne
provoque le message d'erreur

Citation:

You can't specify target table 'infoappli' for update in FROM clause' in C:\wamp\www\eis2\eisnew\page\copyserver.php on line 76

Quelle est la bonne méthode ?

Maintenant, je tente de faire l'update en 2 requêtes :

Code:

1
2
3
4
5
6
7
8
9
10
11
	$query='INSERT INTO `infoappli` (name) VALUES("'.$name_copy.'")';
	$bdd->query($query); // d'abord, on crée l'enregistrement avec le name
 
	$sql='SELECT pec from `infoappli` where name="'.$machselcopy.'"';
	$qid=$bdd->query($query);
	$row=$qid->fetch(PDO::FETCH_OBJ);
	$pec=$row->pec; // on va chercher la valeur à copier dans infoappli
 
	$query='update `infoappli` SET pec='.$pec.' where name="'.$name_copy.'"';
 
	$qid = $bdd->query($query);

et ça se passe bien :D