AOP préparées pour l'INSERTION et SUR le DOUBLE de la CLÉ de mise à JOUR avec des espaces réservés nommés

J'aimerais commutateur AOP INSÉRER et mettre à JOUR des déclarations préparées à INSÉRER et SUR le DOUBLE de la CLÉ de mise à JOUR car je pense que ça va être beaucoup plus efficace que ce que je fais actuellement, mais je vais avoir de la difficulté à trouver la bonne syntaxe à utiliser avec des marqueurs nommés et bindParam.

J'ai trouvé plusieurs question similaire sur, mais je suis nouveau sur PDO et je ne pouvais pas réussi à adapter le code pour mes critères. C'est ce que j'ai essayé, mais ça ne fonctionne pas (il n'a pas d'insérer ou mettre à jour):

try { 
  $stmt = $conn->prepare('INSERT INTO customer_info (user_id, fname, lname) VALUES(:user_id, :fname, :lname)'          
 'ON DUPLICATE KEY UPDATE customer_info SET fname= :fname, 
                                            lname= :lname   
                                            WHERE user_id = :user_id'); 
  $stmt->bindParam(':user_id', $user_id);  
  $stmt->bindParam(':fname', $_POST['fname'], PDO::PARAM_STR);
  $stmt->bindParam(':lname', $_POST['lname'], PDO::PARAM_STR);      
  $stmt->execute();
}

C'est une version simplifiée de mon code (j'ai plusieurs requêtes, et chaque requête est compris entre 20 - 50 champs). Je suis actuellement à la mise à jour de la première et de vérifier si le nombre de lignes mises à jour est supérieure à 0 et si non, alors l'exécution de l'Insertion, et chacune de ces requêtes a son propre ensemble de bindParam consolidés.

Ne pas réutiliser le même espace réservé à plusieurs endroits dans la même requête. Est votre AOP jeu de connexion pour lancer des exceptions? Sauf si vous avez un besoin réel pour bindParam, bindValue ou d'un passage de paramètres par le biais de execute est un meilleur choix.
Quand vous dites que je ne devrais pas réutiliser le même espace réservé à de multiples endroits, voulez-vous dire que j'ai besoin d'avoir 2 jeux de bindParam déclarations? Je l'ai mis actuellement avec - catch(PDOException $e) {echo 'Erreur:' . $e->getMessage();} - et je ne reçois pas tous les messages d'erreur pour ce code.
Il suffit de le mettre try/catch autour de la requête n'est pas assez. Voir gestion des Erreurs et le configurer pour lancer des exceptions en cas d'erreur.
Pourriez-vous élaborer sur ce que tu veux dire? J'ai pensé à la façon dont je le fais dans la droite ligne avec ERRMODE_EXCEPTION dans le manuel, qui doit lancer une PDOException.
Votre exemple ne montre pas ERRMODE_EXCEPTION à définir. C'est pourquoi j'ai demandé si votre connexion est configuré pour le faire. Si vous ne l'avez défini, rien n'est plus nécessaire, les problèmes devraient lancer des exceptions.
Mon mauvais pour ne pas faire ça 😉 C'est me prendre un peu de temps pour obtenir une poignée sur PDO, et j'avais oublié que j'avais mis une partie de la gestion d'erreur dans le fichier de configuration.
J'ai pris vos suggestions et commuté à bindValue, mais j'aimerais clarifier ce que vous disiez à propos de la différence bindParam vs bindValue, et passage de paramètres par le biais de l'exécuter. (J'avais initialement pas été en mesure d'obtenir le code pour travailler avec bindValue j'ai donc été à l'aide de bindParam)
Voir cette question les différences entre les bindParam et bindValue. $stmt->execute(array(':fname' => $_POST['fname'])) peut également être utilisé pour transmettre des variables liées. À mon avis, execute est l'option la plus simple et devrait être préféré si vous en avez besoin les comportements spécifiques de bindParam ou bindValue. Votre kilométrage peut varier, bien sûr.
Merci pour ce lien - il si bien expliqué 🙂 j'étais sous l'impression que la liaison des valeurs ou des paramètres est plus sécurisé puis l'exécution d'un tableau. Suis-je trompé?

InformationsquelleAutor Chaya Cooper | 2012-12-30

17

Votre ON DUPLICATE KEY syntaxe n'est pas correcte.
```
$stmt = $conn->prepare('INSERT INTO customer_info (user_id, fname, lname) VALUES(:user_id, :fname, :lname)
    ON DUPLICATE KEY UPDATE fname= :fname2, lname= :lname2');

$stmt->bindParam(':user_id', $user_id);  
$stmt->bindParam(':fname', $_POST['fname'], PDO::PARAM_STR);
$stmt->bindParam(':lname', $_POST['lname'], PDO::PARAM_STR);      
$stmt->bindParam(':fname2', $_POST['fname'], PDO::PARAM_STR);
$stmt->bindParam(':lname2', $_POST['lname'], PDO::PARAM_STR);      
```
Vous n'avez pas besoin de mettre le nom de la table ou SET dans le ON DUPLICATE KEY clause, et vous n'avez pas besoin d'un WHERE de l'alinéa (il a toujours les mises à jour de l'enregistrement avec le double de la clé).

Voir http://dev.mysql.com/doc/refman/5.5/en/insert-on-duplicate.html

De vous a une erreur de syntaxe PHP: vous divisez la requête en deux chaînes.

Mise à JOUR:

Pour lier plusieurs paramètres:
```
function bindMultiple($stmt, $params, &$variable, $type) {
  foreach ($params as $param) {
    $stmt->bindParam($param, $variable, $type);
  }
}
```
Ensuite l'appeler:
```
bindMultiple($stmt, array(':fname', ':fname2'), $_POST['fname'], PDO::PARAM_STR);
```
- Merci pour l'expliquer clairement 🙂 Y a t'il un moyen d'avoir à créer deux bindParam énoncés pour chaque paramètre?
- Non, PDO exige que chaque espace réservé à l'être unique. Vous pouvez écrire une fonction qui prend un tableau d'espaces réservés et une variable, et les appels bindParam() dans une boucle pour lier le tout.
- Voir mise à jour en réponse
- Vous êtes génial 😀 je vais essayer la première chose le matin 🙂
- La première partie du code a fonctionné comme un charme, mais j'ai du mal à la faire fonctionner avec la liaison de plusieurs paramètres. Juste pour clarifier, dois-je créer des bindMultiple énoncés pour chaque élément? Aussi, je ne savais pas si j'avais raison de penser que je devrais laisser l'original bindParam déclaration pour user_id?
- Vous avez besoin d'utiliser une bindMultiple pour chaque variable que vous avez besoin de répéter dans le SQL. Vous avez besoin de répéter $_POST['fname'] et $_POST['lname'], mais il n'y a qu'un $user_id.
- Je viens de remarquer un problème. bindParam exige que la variable est une référence. Je viens de changer bindMultiple pour refléter cela.
- Heureux que vous avez pris que l' 😀 Il fonctionne parfaitement maintenant 😀 Est-il quelque chose que je dois faire différemment pour bindValue? Aussi, je suis en supposant que je peux ajouter des espaces réservés pour les autres requêtes (c'est à dire chercher), mais je me demandais si il y a des problèmes de performances, etc. que je doit garder à l'esprit?
- Pour bindValue() il n'est pas judicieux de se référer à la même colonne ou d'une variable à plusieurs reprises, donc ce n'est pas un problème.
- Je sais que c'est probablement une question très simple, mais pouvez-vous expliquer pourquoi il ne serait pas logique de le faire avec bindValue? Je ne suis pas tout à fait une poignée sur tous pas encore. @DCoder commenté que bindValue serait un meilleur choix puis bindValue (et j'étais heureux que votre impressionnant script a travaillé avec les deux ;-)) j'ai plusieurs requêtes pour chacun des champs (insérer & mise à jour des cours, chercher, etc.) et puisqu'il y a quelques centaines de champs, j'ai été en espérant pour simplifier la définition et la liaison des variables
- Désolé, mon erreur, je pensais bindResults. La seule différence avec bindValue est qu'il n'a pas besoin d'utiliser une référence, mais je pense que cela nécessite encore de l'unique espaces réservés.
- Merci pour préciser que 😀 Donc, juste pour vérifier, puis-je utiliser bindMultiple comme pour bindValue? Aussi, je suis en supposant que je peux ajouter des espaces réservés pour les autres requêtes (c'est à dire chercher), mais je me demandais si il y a des problèmes de performances, etc. que je doit garder à l'esprit?
- Oui, il suffit de le modifier pour l'utilisation normale de paramètre à la place de référence. Je ne pense pas qu'il y a des problèmes de performances, il n'y a rien de spécial à propos de le faire de cette façon.
- Pour le faire j'ai juste enlever le & à partir de "&$variable", à droite?
- Oui, bien sûr.
InformationsquelleAutor Barmar
16

À mon humble avis ci-dessous est la réponse appropriée pour toute personne venant à travers ce nouveau.

Remarque: cette déclaration suppose user_id est une CLÉ dans la table.

La DÉCLARATION, en effet, se tromper, mais l'on a accepté la réponse n'était pas tout à fait correct.

Si vous êtes l'insertion et la mise à jour en utilisant les mêmes valeurs (et pas de mise à jour avec des valeurs différentes), c'est la requête pseudo-code corrigé:
```
try { 
    //optional if your DB driver supports transactions
    $conn->beginTransaction();

    $stmt = $conn->prepare('INSERT INTO customer_info (user_id, fname, lname) ' . 
                'VALUES(:user_id, :fname, :lname)' .
                'ON DUPLICATE KEY UPDATE fname=VALUES(fname), lname=VALUES(lname)');
    $stmt->bindParam(':user_id', $user_id);  
    $stmt->bindParam(':fname', $_POST['fname'], PDO::PARAM_STR);
    $stmt->bindParam(':lname', $_POST['lname'], PDO::PARAM_STR);      
    $stmt->execute();

    //again optional if on MyIASM or DB that doesn't support transactions
    $conn->commit();
} catch (PDOException $e) {
    //optional as above:
    $conn->rollback();

    //handle your exception here $e->getMessage() or something
}
```
- +1 je suis d'accord que l'utilisation de VALUES() est plus facile quand vous avez besoin d'utiliser des paramètres. Mais comme une question de côté, fwiw vous n'avez pas besoin de casser votre chaîne de caractères en PHP comme vous le faites en Java. Vous pouvez mettre un multi-ligne de chaîne de caractères en un seul ensemble de citations.
- Est correct, mais nous nous plaisons à envelopper nos lignes à 80 ou 100 caractères dans notre société 🙂
InformationsquelleAutor Ligemer

Vous devez vous connecter pour publier un commentaire.