MySQL contrainte unique

Bonjour à tous,

Code:

---

1 2 3 4 5

CREATE TABLE table ( adresse_email varchar(128) NOT NULL, UNIQUE KEY adresse_email (adresse_email) ) ENGINE=InnoDB DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ;

---

Comment gérer en PHP l'erreur retournée par la contrainte unique sur le champ adresse_email lorsqu'un utilisateur essaie d'entrer une email qui existe déjà dans la table table.

Merci d'avance...

J'ai essayé ceci pour gérer correctement l'erreur

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

define('DB_USER', 'root'); define('DB_PASSWORD', ''); define('DB_CONNECTION_STRING', 'mysql:host=localhost;dbname=test');   function insertdb($email) { global $db; $sql = 'INSERT INTO duplicate (email) VALUES (:email)'; $statement = $db->prepare($sql); $statement->bindValue(':email', $email); if ( $statement->execute() ) return $db->lastInsertId(); else return $statement->errorInfo(); }   $db = new PDO(DB_CONNECTION_STRING, DB_USER, DB_PASSWORD); $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); $db->query('SET NAMES utf8');   $email = 'bmw@yahoo.fr'; $user = insertdb($email); $erreur =& $user;

---

Message d'erreur affiché par le navigateur pour doublon rencontré

Citation:

---

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'bmw@yahoo.fr' for key 'email'' in C:\wamp\www\test\duplicate.php on line 11

PDOException: SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'bmw@yahoo.fr' for key 'email' in C:\wamp\www\test\duplicate.php on line 11

---

Pourtant ce que je souhaite c'est que l'erreur soit stocké dans la varibale $erreur (pour traitement avec une expression régulière) et non affiché par le navigateur.

Que faire ?

Merci d'avance...

Tu devais pourtant t'y attendre vu que tu fais

Code:

---

$db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

---

:roll:

Solution: soit tu mets ton execute() dans un bloc try/catch (recommandé) soit mets l'error mode à PDO::ERRMODE_SILENT (déconseillé).

Avant que tu ne pose la question, voici comment utiliser un bloc try/catch: http://php.net/manual/en/language.exceptions.php

Salut

Une autre technique, moins optimisée, mais simple, consiste à faire une 1ère requête SELECT pour vérifier si l'adresse mail existe déjà ou pas.

Un simple :

Code:

---

1 2 3

SELECT COUNT(*) FROM duplicate WHERE email = :email

---

Une autre encore, vu que tu utilise InnDB, c'est de faire un trigger coté SQL (MySQL) qui sera déclenché lors d'un INSERT, ce trigger fera une requête SQL (comme un peu ci-dessus), et l'insertion ne se fera que SI l'email n'existe pas.
C'est pas loin d'être la même chose mais en plus optimisé.

Peut être même plus optimisé qu'une gestion des exceptions Php, car l'erreur de duplication sera obtenue que lorsque MySQL aura renvoyée l'erreur à Php.

Ceci dit, je dit bien "peut être" car je suis loin d'être expert pour affirmer ce genre de chose.
Disons que vu que le tout (trigger / l'insertion) se passe coté MySQL, la théorie veut que cela soit assez performant.


Pure suggestions. ;)

Citation:

---

Envoyé par Benjamin Delespierre

soit mets l'error mode à PDO::ERRMODE_SILENT (déconseillé).

---

Ça marche.

Citation:

---

Envoyé par Benjamin Delespierre

soit tu mets ton execute() dans un bloc try/catch (recommandé).
http://php.net/manual/en/language.exceptions.php

---

Je suis interressé par cette solution mais je n'y arrive pas. J'ai regardé le lien. De l'aide SVP !

Merci d'avance...

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

function insertdb($email) { global $db;   $sql = 'INSERT INTO duplicate (email) VALUES (:email)'; $statement = $db->prepare($sql); $statement->bindValue(':email', $email); try { $statement->execute(); $return = $db->lastInsertId(); } catch (Exception $e) { $return = $statement->errorInfo(); } return $return; }

---

Ça marche. Avant de basculer en résolu, j'attends vos remarques et améliorations possibles.

Merci d'avance...

Merci à vous tous pour votre précieuse aide.:ccool:

mis à part le global $db qui me répugne, c'est acceptable niveau sécurité.

Remarque: pour éviter d'avoir recours à une variable globale (ou à un Singleton, ce qui revient exactement au même), tu devrais considérer la possibilité d'injecter l'instance PDO dans les fonctions/méthodes qui en ont besoin.

Dans ton cas ça nous donne:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

function insertdb(PDO $db, $email) { $sql = 'INSERT INTO duplicate (email) VALUES (:email)'; $statement = $db->prepare($sql); $statement->bindValue(':email', $email); try { $statement->execute(); $return = $db->lastInsertId(); } catch (Exception $e) { $return = $statement->errorInfo(); } return $return; }

---

De cette façon, tu es certain que $db sera toujours une instance de PDO, c'est d'ailleurs beaucoup plus flexible car tu peux passer l'instance qui t'intéresse si par exemple tu utilise 2 serveur MySQL (ok c'est rare mais ça arrive).