Discussion :

[okoweb]

Bonjour à tous,

Code :

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

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

[okoweb]

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

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
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é

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

[Benjamin Delespierre]

Tu devais pourtant t'y attendre vu que tu fais

Code :

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

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

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

[RunCodePhp]

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 sql :

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

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.

[okoweb]

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

Ça marche.

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

[okoweb]

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

[okoweb]

Merci à vous tous pour votre précieuse aide.

[Benjamin Delespierre]

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 :

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

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