Discussion :

[laurentSc]

Bonjour,

j'effectue un try/catch dans le cadre d'une transaction ce qui fait que le commit ne sera effectué que si la requête (un INSERT) sera exécutée sans problème. Cela dit, si une autre erreur PHP se produit, on va aller dans le catch et on fera un rollback.
Mon code est le suivant :

Code php :

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
22
23
24
25
26
27
28
29
30
31
32
 public function save(string $email = null, int $sesa, ?int $id = 0) {
 
        try
        {
            $ppp = new PDOPlusPlus();
 
            $ppp->startTransaction();
 
            $local_address = explode("@",$email)[0];
            $domain = explode("@",$email)[1];
 
            if ( $id ===  0)
            { //si $email pas en bdd, alors insert
                $sql = <<<sql
INSERT INTO em_mail
SET 
    EM_email = {$ppp($local_address)},
    DM_ident = (SELECT DM_ident FROM DM_domain WHERE DM_domain_name = {$ppp($domain)}),
    US_ident = (SELECT US_ident FROM US_user WHERE US_sesa = {$ppp($sesa,
                    'int')})
sql;
                $id = $ppp->insert($sql);
                $ppp->commit();
 
            }
        }
 
 
        catch (\Exception $e) {
                $ppp->rollback();
            } // sur cette table, pas d'update
    }

Une erreur surviendra si $email est null. Si c'est le cas, on passera dans le catch avant l'exécution de la requête et donc un rollback sera exécuté. Cette façon de faire est-elle convenable ?

[jreaux62]

Bonjour Laurent,

on ne doit pas provoquer volontairement une erreur PHP ou SQL.
Ça n'a pas de sens.

C'est comme :

• descendre une côte à vélo, SANS vérifier les freins,
• et attendre de se prendre un gadin pour dire : "Ah, les freins ne vont pas !"

(ça te rappelle des souvenirs ? )


Il faut faire les vérifications nécessaires concernant $email.

• vérifier qu'il est non NULL
• vérifier que c'est bien un email : filter_var($email, FILTER_VALIDATE_EMAIL),
• et si PAS d'erreur, on lance la requête

[CosmoKnacki]

Code :

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

save(string $email = null, int $sesa, ?int $id = 0)

Attention, tu t'emmêles les pinceaux avec tes paramètres de fonction.

Lorsque tu donnes une valeur par défaut à un paramètre c'est que tu veux le rendre optionnel, or les paramètres optionnels d'une fonction/méthode sont toujours à la fin^*, après les paramètres obligatoires. Donc pas la peine de donner une valeur par défaut à $email qui de toute façon (je pense) est obligatoire. De plus dés lors que tu utilises le type "string" pour ce paramètre, en aucun cas celui-ci ne devrait accepter pour valeur null. (qu'il convertira en chaîne vide automatiquement, ou alors tu obtiendras une erreur fatale à la compilation si tu es en mode type strict).

Pour le dernier paramètre $id: es-tu sûr que la valeur par défaut doit-être 0 et pas null puisque tu as mis un ? avant int? Si ce n'est pas le cas et que ce paramètre ne peut jamais être null, alors le ? est inutile.

^*sauf si tu utilises les paramètres nommés de PHP 8 et dans ce cas, l'ordre n'a plus d'importance.

[laurentSc]

ça te rappelle des souvenirs ?

chu devenu expert de cette question

pour les contrôles, j'ai remplacé

Code php :

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

1
2
$local_address = explode("@",$email)[0];
$domain = explode("@",$email)[1];

par

Code php :

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

1
2
3
4
5
6
$explode_email = explode("@",$email??[]);
if (( isset($explode_email) ) && (count($explode_email) == 2) )
{
      $local_address = $explode_email[0];
      $domain = $explode_email[1];
}

Qu'en penses-tu Jérôme ?

Suite aux remarques de Cosmo : public function save(string $email, int $sesa, int $id = 0)

[CosmoKnacki]

Code :

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

1
2
$explode_email = explode("@",$email??[]);
if (( isset($explode_email) ) //...

Quelque soit le scénario $explode_email sera toujours défini. Pourquoi veux-tu passer un tableau à explode?

[laurentSc]

...car je me suis trompé (confusion avec implode) :

Code php :

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

1
2
3
4
5
6
7
 
            $explode_email = explode("@",$email??"");
            if (count($explode_email) == 2)
            {
                $local_address = $explode_email[0];
                $domain = $explode_email[1];
            }

$email??"", c'est pour passer une chaine vide à explode si $email n'existe pas pour qu'il n'y ait pas d'erreur.

Juste après, j'ai aussi mis if ( ( $id == 0) && (isset($local_address)) && (isset($domain)) )

[grunk]

Cela dit, si une autre erreur PHP se produit, on va aller dans le catch et on fera un rollback.

Il suffit de spécialiser le catch qui provoque le rollback.

AVec :

Code :

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

1
2
3
catch (\Exception $e) {
    $ppp->rollback();
}

Tu rollback quelque soit l'exception , puisque tu utilises Exception qui est la base de toutes les exception.

Tu pourrais donc faire :

Code :

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

1
2
3
4
5
6
7
 
catch (\PDOException $e) { // Erreur lié à PDO
    $ppp->rollback();
}
catch (\Exception $e) { // Multiple catch supporté a partir de PHP 7.1
	// Faire autre chose
}