Discussion :

[thais781]

Bonjour,

Je cherche à faire un truc simple.
Si la référence existe, on fait un UPDATE
Si la référence n'existe pas un INSERT TO

Mon pb est au premier if, mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<?php
$MyRef="1234";
try{
    $bdd = new PDO('mysql:host=localhost;dbname=MyNameBd', 'root', 'root');
}
catch(Exception $e){
    die('Erreur : '.$e->getMessage());
}
$reponse = $bdd->query('SELECT * FROM OffresSql WHERE Ref='$Ref''); 
$row = count($reponse->fetchAll());
echo ($row);
if($row!=0){  // l'url existe déjà, on affiche un message d'erreur
    echo '<font color="red">Désolé, mais cette URL existe déjà dans notre base.</font>';
}
?>

Sachant que ma base a bien un champ "Ref" .....

Et la ... page blanche ....ca plante avec le WHERE

Une idée ?

Thais

[ABCIWEB]

La bonne syntaxe ici, début de l'exemple n° 2

Sinon il existe aussi la syntaxe INSERT ... ON DUPLICATE KEY UPDATE qui permet de ne faire qu'une requête pour les deux cas de figure.

[thais781]

Merci pour ton aide, je vais regarder de près le ON DUPLICATE KEY ...

Mais, mon soucis est plus l'utilisation d'une variable PHP dans la Query.
En gros,

Tranformer :

Code :

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

1
2
 
$sql = "SELECT * FROM OffresSql WHERE Ref='D1234'";

En :

Code :

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

1
2
3
 
$MyRef="D1234";
$sql = "SELECT * FROM OffresSql WHERE Ref=$MyRef'";

Avec la bonne syntaxe pour passer cela en paramètre ....

Thais

[ABCIWEB]

Il vaut mieux faire une requête préparée quand il y a une variable

Code :

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

1
2
3
$sql = "SELECT * FROM OffresSql WHERE Ref = ?";
$reponse = $bdd->prepare($sql);
$reponse->execute(array($MyRef));

[thais781]

Perfect ...

J'ai regardé de plus pres le DUPLICATE ....
Petites questions,
1. le ON DUPLICATE KEY UPDATE se fais sur l'ID ou peut on lui spécifier un champ différent (dans mon cas, ce serai "Ref")
2. Est-ce que ma syntaxe est la bonne en y ajoutant la partie execute(array .... ?


mon code ci-dessous

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
    // On prepare la requete
    $req = $bdd->prepare('INSERT INTO OffresSQL(Ref, Status, Titre, Region, Domaine, Img, Texte2, Texte3,MyDate) VALUES (:Ref, :Status, :Titre, :Region, :Domaine, :Img, :Texte2, :Texte3, :MyDate) ON DUPLICATE KEY UPDATE (:Ref, :Status, :Titre, :Region, :Domaine, :Img, :Texte2, :Texte3, :MyDate)');
    $req->execute(array(
        'Ref' => $MyRef,
        'Status' => $Status,
        'Titre' => $Titre,
        'Region' => $Region,
        'Domaine' => $Domaine,
        'Img' => $Img,
        'Texte2' => $Texte2,
        'Texte3' => $Texte3,
        'MyDate' => $MyDate
    ));

Thais

[Spartacusply]

Oui ça marche aussi sur une colonne unique qui n'est pas la clef primaire http://dev.mysql.com/doc/refman/5.0/...duplicate.html

[ABCIWEB]

Si Ref est une colonne unique dans la bdd et qu'il n'y a pas d'autres colonne unique (en plus de l'id) ça devrait fonctionner correctement. En fait cela fonctionne pour toute colonne unique (quelle qu'elle soit) donc si il y en y plusieurs il ne faut pas qu'elles prêtent à confusion (attention les clés composées de plusieurs colonnes sont également prises en considération). Si l'id est incrémenté automatiquement tu ne le renseigne pas dans les données à insérer donc cela ne pose pas de problème.

Faudrait revoir la syntaxe, normalement ce serait plutôt :

Code :

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
$sql = "INSERT INTO OffresSQL
(Ref, Status, Titre, Region, Domaine, Img, Texte2, Texte3, MyDate) 
VALUES 
( ?, ?, ?, ?, ?, ?, ?, ?, ?) 
ON DUPLICATE KEY UPDATE 
   Status = VALUES(Status),  
   Titre = VALUES(Titre),
   Region = VALUES(Region),
   Domaine = VALUES(Domaine),
   Img = VALUES(Img),
   Texte2 = VALUES(Texte2),
   Texte3 = VALUES(Texte3),
   MyDate= VALUES(MyDate)
";
 
$req = $bdd->prepare($sql);
$req->execute(array($MyRef, $Status, $Titre, $Region, $Domaine, $Img, $Texte2, $Texte3, $MyDate));

Faut pas mettre la colonne "Ref" dans le update puisque précisément si la valeur de Ref est déjà existante tu ne l'update pas mais tu update uniquement les autres champs (ou certains autres champs).

Quand on met Status = VALUES(Status) cela veut dire que la valeur de "Status" sera la même pour l'update qu'elle serait pour la requête d'insertion (la valeur de $Status = $_POST['status'] la plupart du temps).
Si besoin on pourrait indiquer autre chose en cas d'update par exemple Status = VALUES(2) ou encore Status = VALUES(Status + 3) etc. ou encore on pourrait ne pas faire d'update sur certains champs (suffit de ne pas les indiquer dans ce cas).