un REPLACE qui remplace pas

Version imprimable

Bonjour !

On vient de me refiler à résoudre un problème, mais ce n'est pas dans mes compétences, aussi je viens vers vous, les pros !

Le code suivant devrait pouvoir, paraît-il, remplacer un enregistrement. Or, il en rajoute un à chaque fois, sans supprimer ou modifier celui dont on récupére les variables !

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
$editFormAction = $_SERVER['PHP_SELF'];
if (isset($_SERVER['QUERY_STRING'])) {
  $editFormAction .= "?" . htmlentities($_SERVER['QUERY_STRING']);
}
 
if ((isset($_POST["MM_insert"])) && ($_POST["MM_insert"] == "form1")) {
  $insertSQL = sprintf("REPLACE INTO visiteurs (visiteur, passe) VALUES (%s, %s)",
                       GetSQLValueString($_POST['nom'], "text"),
                       GetSQLValueString($_POST['passe'], "text"));
 
  mysql_select_db($database_conncz, $conncz);
  $Result1 = mysql_query($insertSQL, $conncz) or die(mysql_error());
 
  $insertGoTo = "conf.php";
  if (isset($_SERVER['QUERY_STRING'])) {
    $insertGoTo .= (strpos($insertGoTo, '?')) ? "&" : "?";
    $insertGoTo .= $_SERVER['QUERY_STRING'];
  }
  header(sprintf("Location: %s", $insertGoTo));
}
 
$colname_Recordset1 = "-1";
if (isset($_POST['passe'])) {
  $colname_Recordset1 = $_POST['passe'];
}
mysql_select_db($database_conncz, $conncz);
$query_Recordset1 = sprintf("SELECT * FROM visiteurs WHERE passe = %s", GetSQLValueString($colname_Recordset1, "text"));
$Recordset1 = mysql_query($query_Recordset1, $conncz) or die(mysql_error());
$row_Recordset1 = mysql_fetch_assoc($Recordset1);
$totalRows_Recordset1 = mysql_num_rows($Recordset1);

Donc, apparemment, contrairement à ce qu'on nous avait affirmé, il ne suffit pas de remplacer INSERT par REPLACE.
Pourriez-vous nous décoincer svp :) ?
Merci !

07/07/2009, 03h40
jeca

Bonjour,

Extrait du manuel de référence mysql :

Citation:

REPLACE fonctionne exactement comme INSERT, sauf que si une vieille ligne dans la table à la même valeur qu'une nouvelle pour un index UNIQUE ou une PRIMARY KEY, la vielle ligne sera effacée avant que la nouvelle ne soit insérée
07/07/2009, 07h27
evevev

Ah... :cry: Bon... donc c'est pas possible alors...
Merci de m'avoir répondu ! :)
07/07/2009, 07h38
Antoun

Je n'ai pas cherché à lire tout ton fatras de code PHP (dont je rappelle qu'il est hors-charte sur ce forum consacré à MySQL). Mais est-ce que INSERT... ON DUPLICATE KEY UDATE... ne serait pas ce que tu cherches ? Sinon, merci de t'expliquer en français plutôt qu'en PHP ;)
07/07/2009, 14h09
evevev

Citation:

Envoyé par Antoun

merci de t'expliquer en français plutôt qu'en PHP ;)

... ça tombe bien, je parle mieux français que php !
Excuse pour l'erreur de forum, comme mon problème concernait la bdd, je n'ai pas vraiment été chercher plus loin... :oops:

En français donc : les visiteurs s'identifient avec nom et mot de passe. Je récupère ces paramètres et affiche sur la page suivante, dans un formulaire, leur petite annonce. Ils font la modification qu'ils veulent, et cliquent sur Envoyer.
Sauf que ça ne marche pas : le nouvel enregistrement qu'ils envoient ne remplace pas l'ancien (ce que je voudrais !), mais se rajoute dans la table.

Je suis allée voir ton lien, mais je suis pas beaucoup plus fortiche en anglais qu'en php... mais je vais essayer de piger !
07/07/2009, 15h18
Antoun

peux-tu lancer la requête suivante et nous donner le résultat ?

Code:

SHOW CREATE TABLE visiteurs

Comme ça ? :?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
Table  Create Table  
visiteurs CREATE TABLE `visiteurs` (
  `ID_visiteur` int(11) NOT NULL auto_increment,
  `visiteur` varchar(30) NOT NULL default '',
  `passe` varchar(30) NOT NULL default '',
  `titre` varchar(50) NOT NULL default '',
  `descriptif` text NOT NULL,
  `photoP` varchar(50) NOT NULL default '',
  `coord` varchar(200) NOT NULL default '',
  `tel` smallint(16) NOT NULL default '0',
  `date` datetime NOT NULL default '0000-00-00 00:00:00',
  `positiongeo` varchar(50) NOT NULL default '',
  `commune` varchar(20) NOT NULL default '',
  `categorie` varchar(30) NOT NULL default '',
  `prix` varchar(50) NOT NULL default '',
  `animaux` varchar(100) NOT NULL default '',
  PRIMARY KEY  (`ID_visiteur`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1

edit : depuis que j'ai posté mon message hier, j'ai rajouté des champs...

07/07/2009, 17h06
Antoun
C'est parfait. Pour que ton REPLACE marche, il faudrait qu'il y ait une clé unique sur la colonne visiteur :

Code:

ALTER TABLE visiteurs ADD UNIQUE KEY (visiteur) ;

Une fois que ceci est en place, tu as deux options :
- REPLACE : ça fonctionne comme un INSERT ; si le visiteur existe déjà, il est détruit et re-créé... ce qui veut dire aussi que toutes les infos qui ne sont pas ré-insérées sont perdues, et qu'une nouvelle ID est générée (sauf si tu la mets elle aussi dans le REPLACE)
- INSERT... ON DUPLICATE KEY UPDATE ... : en gros, tu fais un INSERT, qui sera remplacé par un UPDATE en cas de violation de clé unique
Exemple pour le second cas :
Code:

1 2 INSERT INTO visiteurs (visiteur, passe) VALUES (%s, %s) ON DUPLICATE KEY UPDATE passe = %s
EDIT : je précise qu'avec mon exemple, ton sprintf doit envoyer deux fois le mot de passe
07/07/2009, 17h25
evevev

Je tiens à te remercier tout de suite de ces explications très claires. Je m'occupe de ça demain, et repasserai ici pour te re-remercier ! :D

Pas pu attendre demain... trop d'impatience ! :)
Mais j'ai une erreur :

Code:

Erreur de syntaxe près de '; REPLACE INTO visiteurs (visiteur, passe) VALUES ('kk', '' à la ligne 1

J'ai peut-être fait une erreur dans la requête :
Code:

1 2 3 4 5 if ((isset($_POST["MM_insert"])) && ($_POST["MM_insert"] == "form1")) { $insertSQL = sprintf("ALTER TABLE visiteurs ADD UNIQUE KEY (visiteur) ; REPLACE INTO visiteurs (visiteur, passe) VALUES (%s, %s)", GetSQLValueString($_POST['nom'], "text"), GetSQLValueString($_POST['passe'], "text"));
J'ai déjà été confronté à ça, je crois que c'est DW qui rajoute des apostrophes, ou qui en retire, je ne sais plus... J'ai fait plusieurs tentatives d'ajouts et de retrait de ', mais toujours cette erreur de syntaxe...

07/07/2009, 21h41
Antoun

L'erreur de syntaxe, c'est que tu passes deux requêtes à la fois, ce que ne permet pas mysql_query.

Le ALTER TABLE, tu ne le fais qu'une fois, par exemple dans phpMyAdmin. Une fois que la clé unique est là, il n'y a pas besoin de la re-créer à chaque fois.

Sinon, je ne connais pas trop sprintf, et je ne suis pas trop sûr de ce que ça produit... si ça merde encore, fais un echo $insertSQL, qu'on voit le résultat du sprintf.
08/07/2009, 07h51
evevev

C'est tout bon ! Merci ! :D
08/07/2009, 11h55
Antoun

Tu as bien conscience qu'avec REPLACE, tu vas perdre le titre, le numéro de téléphone, l'adresse de la photo, etc., à chaque fois qu'un utilisateur va changer son mot de passe ?