transaction : faire un return si rollback

Version imprimable

Bonjour,

utiliser une transaction sert à ne modifier la bdd que si l'ensemble des requêtes se déroule bien. OK. Par contre, la fonction (ou méthode) qui effectue la transaction a une valeur de retour qui est utilisée par le code qui l'appelle. Je me dis donc que si on fait un rollback, il faut mettre return NULL; dedans. Est-ce exact ?

Voici un exemple de code :

Code:

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
  try
        {
            $ppp = new PDOPlusPlus(); //super classe de rawsrc (enrobage de pdo)
 
            $ppp->startTransaction();
 
            if ( $id == 0 )
            {
                $sql = <<<sql
INSERT INTO US_user (US_firstname ,US_lastname,US_sesa) 
    VALUES(
           {$ppp($firstname,'str')}, 
           {$ppp($lastname, 'str')}, 
           {$ppp($sesa, 'str')}
          ) ;
sql;
 
                $id = $ppp->insert($sql);
                return $id;
 
 
                $ppp->commit();
            }
        }
 
 
        catch (\Exception $e) {
            $ppp->rollback();
            return NULL; //pour que le code appelant comprenne que la bdd n'a pas été modifiée
        }

27/07/2021, 14h45
CosmoKnacki

Oui par exemple, mais tu peux aussi te caler sur les fonctions build-in de PHP qui renvoient un false (comme strpos) et réservent le null plutôt lorsque la fonction ou méthode est censée retourner un objet (par exemple DateTime::createFromFormat avec un format illisible).

Après c'est toi qui voit quelles conventions tu choisis pour ton code.

Pourquoi utilises-tu une transaction alors qu'il n'y a qu'une seule requête?
27/07/2021, 15h25
laurentSc
Merci pour ta réponse, et OK pour privilégier false quand on peut. Selon moi, false est un booléen. Si la fonction est censée retourner un entier, false ou NULL ?

Citation:

Envoyé par CosmoKnacki

Pourquoi utilises-tu une transaction alors qu'il n'y a qu'une seule requête?

Même si ici, un INSERT, soit il réussit, soit il échoue et alors la bdd n'est pas modifiée, pour être sûr de ne pas faire d'erreur, je préfère adopter partout la même technique.
A noter que la technique des transactions, je viens de la découvrir et que au préalable, en plus de la classe US_user, j'envisageais une classe US_user_error, et que en cas d'erreur détectée (avant de faire les requêtes SQL, j'analyse les données), au lieu d'écrire dans US_user, j'écrivais dans US_user_error.

EDIT : en regardant mon code :
Code:

1 2 3 4 $id = $ppp->insert($sql); return $id; $ppp->commit();
comme le return est placé avant le commit, je me demande si on va exécuter la requête. Qu'en penses-tu ?
27/07/2021, 15h39
CosmoKnacki

Moi je pencherais pour false (comme pour strpos).

Effectivement, j'avais pas vu, le return doit évidemment être après le commit() sans quoi rien ne sera modifié dans la bdd.
27/07/2021, 15h49
grunk

Citation:

Même si ici, un INSERT, soit il réussit, soit il échoue et alors la bdd n'est pas modifiée, pour être sûr de ne pas faire d'erreur, je préfère adopter partout la même technique.

Le but des transactions c'est avant tout de maintenir l'intégrité relationnelle des données. Ca a donc de l'intérêt que quand tu as plusieurs requêtes.
Ca peut aussi avoir un intérêt d'un point de vue performance dans certains cas bien précis (insertion en masse)

C'est bien de vouloir standardiser tes pratiques mais il ne faut pas utiliser des techniques juste pour faire comme ailleurs , il faut que ca ai du sens.
27/07/2021, 17h57
laurentSc

Merci pour vos réponses à vous 2. Cette discussion est terminée, donc je la clos, mais j'en ai créé une nouvelle car j'ai un autre problème qui n'a rien à voir : https://www.developpez.net/forums/d2.../#post11748854