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).

[Bovino]

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

C'est normal : tu as de gros problèmes avec les quotes...

Code :

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

$reponse = $bdd->query('SELECT * FROM OffresSql WHERE Ref='$Ref'');

Si une chaine commence par une simple quote, que va-t-il se passer selon toi à la simple quote suivante rencontrée dans le code ?

Code :

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

$sql = "SELECT * FROM OffresSql WHERE Ref=$MyRef'";

Pourquoi une simple quote qui se balade toute seule ?

[thais781]

La nouvelle syntaxe d'ABCIWEB marche impec, j'arrive maintenant à ajouter dans ma base sans soucis.

Il me reste par contre le soucis avec la partie Update.

Je n'arrive pas à faire en sorte que le test soit sur la colonne Ref et non sur l'Id.
Le résultat est que cela m'ajoute dans tous les cas un nouvel enregistrement même si la Ref est deja existante ... (pour info, ma ref à une forme D12345)

Une idée ?

Thais

[sabotage]

Comme te l'a dit Spartacusply, il faut un index UNIQUE sur ta colonne.

[thais781]

C'est le cas, ma colonne Ref contient la reference de l'enregistrement et un donc unique (enfin si l'on ajoute pas un enregistrement qui existe déja à la place de l'Updater mon pb actuel ....)

Le soucis est qu'avec le code d'Abciweb, il ne détecte pas que la ref est déjà présente dans la bdd donc il créé un nouvel enregistrement .
Je ne sais pas comment spécifié qu'il faut checker cette colonne 'Ref' ?

Thais

[ABCIWEB]

C'est le cas, ma colonne Ref contient la reference de l'enregistrement et un donc unique (enfin si l'on ajoute pas un enregistrement qui existe déja à la place de l'Updater

Il faut que "Ref" soit déclaré comme un index unique dans ta table. Est-ce le cas ?

Je ne pense pas car si c'était le cas et en supposant que la fonction ON DUPLICATE KEY UPDATE ne fonctionne pas sur ton système de base de donnée, au minimum tu recevrais un message d'erreur sql en essayant de rentrer une valeur de "Ref" déjà existante.