Discussion :

[yann18]

Bonjour/ bonsoir,

J'ai le code suivant:

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
//        Si pas d'erreur
        if (empty($this->errors)) {
//            insertion du compte dans la base
            global $PDO;
            $insert = $PDO->prepare('INSERT INTO cpte (nom, prenom, id, pass, section)
                VALUES(:nom, :prenom, :id, :pass, :section)');
//            Référencement des donnée à inclure dans la requête
            $insert->bindValue(':nom', $_SESSION['newCompte']['nom'], PDO::PARAM_STR);
            $insert->bindValue(':prenom', $_SESSION['newCompte']['prenom'], PDO::PARAM_STR);
           ...
//
            try {
                $result = $insert->execute();
                return TRUE;
            } catch (PDOException $e) {
                header('Location:'.$auth->erreurPage);
            }
        } else {
            return FALSE;
        }

Est-ce que la dernière portion de code (à partir du try) vous paraît correct ? Ou dois-je encore apporter une vérification supplémentaire sur $result ?
merci,

[Benjamin Delespierre]

PDOStatemen::execute renvoie un booléen, ce serait dommage de ne pas s'en servir pour savoir si la requête à été effectuée avec succès. De plus, si tu regardes la doc de cette méthode, tu t'apercevra qu'elle ne lèves pas d'exception.

Tu fais un return TRUE même si la requête à échoué ? c'est vraiment le comportement que tu souhaite ?

Également, pour que PDO lèves des exceptions il faut lui donner un paramètre spécifique lors de la création de l'instance: http://www.php.net/manual/en/pdo.construct.php

[yann18]

merci!