Afficher les messages d'erreur ou de réussite Class

Version imprimable

Bonjour !!!

Dans ma classe, je n'arrive pas à faire sortir les messages soit d'erreurs ou soit de réussites.

Mon objet en bas du code qui a pour but dans cet exemple, c'est de faire
une mise à jour au ID 20. La mise à jour se fait très bien. Sauf qu'il est
supposé afficher un message de réussite. C'est la même chose pour les
autres actions comme "insert, update et delete". Quelqu'un voit le problème?

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
<?php
header('Content-Type: text/html; charset=utf-8');
 
$bd_dsn = "mysql:host=localhost;charset=utf8;dbname=mabase_bdd";
$bd_dsnUser = "root";
$bd_dsnPwd = "pass";
 
define("TBL_PREFIX","");
 
class Categorie
{
    private $id;
    private $cat;
 
    public function __construct($id = 0) {
        if (empty($id)) {
            $this->id =0;
            $this->cat ="";
            $this->requeteOk = true;
            $this->errormsg = "";
        } else {
            global $bd_dsn;
            global $bd_dsnUser;
            global $bd_dsnPwd;
 
            $sql = new PDO($bd_dsn, $bd_dsnUser, $bd_dsnPwd);
            $query = "SELECT * FROM ".TBL_PREFIX."categorie WHERE id = :primary";
            $c = $sql->prepare($query);
            $c->bindParam(":primary", $id, PDO::PARAM_INT);
            if ($c->execute()) {
                $result = $c->fetch(PDO::FETCH_OBJ);
                if (!empty($result)) {
                    $this->id = $result->id;
                    $this->cat = $result->cat;
                    $this->requeteOk = true;
                    $this->errormsg = "categorie trouvée";
                } else {
                    $this->requeteOk = false;
                    $this->errormsg = "Ce categorie n'existe pas !";
                }
            } else {
                $this->requeteOk = false;
                $this->errormsg = $c->errorInfo();
            }
        }
    }
 
    public function insert()
    {
        global $bd_dsn;
        global $bd_dsnUser;
        global $bd_dsnPwd;
 
        $sql = new PDO($bd_dsn, $bd_dsnUser, $bd_dsnPwd);
        $query ="
            INSERT INTO ".TBL_PREFIX."categorie
            (
                cat
            )
            VALUES
            (
                :cat
        )";
        $sql->beginTransaction();
        try {
            $c = $sql->prepare($query);
            $c->bindParam(":cat", $this->cat, PDO::PARAM_STR , 60);
            if ($c->execute()) {
                $this->id = $sql->lastInsertId();
                $sql->commit();
                $c->closeCursor();
                $this->requeteOk = true;
                $this->errormsg = "Insertion reussie !";
            } else {
                $this->requeteOk = false;
                $this->errormsg = $c->errorInfo();
                $sql->rollback();
                $c->closeCursor();
                $this->id = 0;
            }
        } catch (PDOException $Exception) {
            $this->requeteOk = false;
            $this->errormsg = $c->errorInfo();
            $sql->rollback();
            $c->closeCursor();
            $this->id = 0;
        }
    }
 
    public function update()
    {
        global $bd_dsn;
        global $bd_dsnUser;
        global $bd_dsnPwd;
 
        $sql = new PDO($bd_dsn, $bd_dsnUser, $bd_dsnPwd);
        $query =
            "UPDATE ".TBL_PREFIX."categorie SET
                cat = :cat
            WHERE
                id = :id
        ";
        $sql->beginTransaction();
        try {
            $c = $sql->prepare($query);
 
            $c->bindParam(":id", $this->id, PDO::PARAM_INT);
            $c->bindParam(":cat", $this->cat, PDO::PARAM_STR , 60);
 
            if ($c->execute()) {
                $sql->commit();
                $c->closeCursor();
                $this->requeteOk = true;
                $this->errormsg = "Modification reussie!";
            } else {
                $this->requeteOk = false;
                $this->errormsg = $c->errorInfo();
                $sql->rollback();
                $c->closeCursor();
            }
        } catch (PDOException $Exception) {
            $this->requeteOk = false;
            $this->errormsg = $c->errorInfo();
            $sql->rollback();
            $c->closeCursor();
        }
        return $this->requeteOk;
    }
 
    public function delete()
    {
        global $bd_dsn;
        global $bd_dsnUser;
        global $bd_dsnPwd;
 
        $sql = new PDO($bd_dsn, $bd_dsnUser, $bd_dsnPwd);
        $query = "DELETE FROM ".TBL_PREFIX."categorie WHERE id = :primary";
        $c = $sql->prepare($query);
        $c->bindParam(":primary", $this->id, PDO::PARAM_INT);
        $c->execute();
    }
 
 
    public function getId()
    {
        return $this->id;
    }
 
    public function getCat()
    {
        return $this->cat;
    }
 
    public function setId($id)
    {
        $errTab = array("boolOk" => true, "errmsg" => "");
        if (isset($id) && !empty($id))
        {
            $this->id = $id;
        }
        else
        {
            $this->id = 0;
            $this->boolOK = false;
            $this->errormsg = "Le champ id ne peut etre vide";
        }
        return $errTab;
    }
 
    public function setCat($cat)
    {
        $errTab = array("boolOk" => true, "errmsg" => "");
        if (isset($cat) && !empty($cat))
        {
            $this->cat = $cat;
        }
        else
        {
            if (!isset($cat))
            {
                $errTab["errmsg"] = "Le cat ne peut être vide !";
            }
            elseif (empty($cat))
            {
                $errTab["errmsg"] = "Le cat ne peut être vide !";
            }
            elseif(strlen($cat) > 60)
            {
                $errTab["errmsg"] = "Le cat doit être compris entre 1 et 60 caractères !";
            }
            $this->cat = "";
            $this->boolOK = false;
        }
        return $errTab;
    }
 
    static public function getList()
    {
        global $bd_dsn;
        global $bd_dsnUser;
        global $bd_dsnPwd;
 
        $sql = new PDO($bd_dsn,$bd_dsnUser,$bd_dsnPwd);
        $query = "SELECT * FROM ".TBL_PREFIX."categorie";
        $c = $sql->query($query);
        $result = $c->fetchAll(PDO::FETCH_OBJ);
        if(!empty($result))
        { echo "é ";
            return $result;
        }
        return array();
    }
}
 
foreach (Categorie::getList() as $k): echo $k->id .' '. $k->cat; endforeach;
 
$category = new Categorie(20);
$category->setCat("asdf");
$category->update();
 
?>

Merci d'avance.

Ajout: si je fais un echo avant la ligne 114 pour afficher "réussi", ça va l'afficher même si le ID n'existe pas. Alors, quelque chose ne va pas dans mon code.

08/10/2013, 12h04
sabotage

Une requête UPDATE est valide même si aucune ligne n'a été affectée.
Utilise rowCount() pour savoir si une ligne a été modifiée.
08/10/2013, 12h46
dancom5
Tien, j'ai changé :

Code:

$this->errormsg = "Modification réussie";

pour :

Code:

$this->errormsg = ($c->rowCount()===1) ? "Modification réussie" : "Aucune modification";

J'ai ajouté une nouvelle fonction:
Code:

1 2 3 public function message(){ echo $this->errormsg; }
et l'objet:

Code:

$category->message();

Ainsi, ça devrait faire mes affichages partout en modifiant les autres messages d'erreurs.

Merci pour l'indication sabotage.

J'ai modifié mon code pour le mettre au plus basique possible.
J'ai tenter de séparer la connexion bdd par une fonction. négatif.
J'ai tenter de faire une class, négatif aussi.

Citation:

Mon erreur : Fatal error: Call to a member function prepare() on a non-object ligne 43

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
<?php
header('Content-Type: text/html; charset=utf-8');
 
class Categorie
{
    private $id;
    private $cat;
 
    public function __construct($id=0,$cat="") {
        $this->id=$id;
        $this->cat=$cat;
        $this->errormsg="";
    }
 
    public function connect() {
        $db_host = "localhost";
        $db_name = "bdd";
        $db_user = "root";
        $db_pass = "pass";
 
        try {
            $sql = new PDO("mysql:host=" . $db_host . ";dbname=" . $db_name, $db_user, $db_pass);
        } catch(PDOException $e) {
            die($e);
        }
    }
 
    public function insert()
    {
        $sql = $this->connect();
        $query = " INSERT INTO categorie (cat) VALUES (:cat) ";
        $c = $sql->prepare($query);
        $c->bindParam(":cat", $this->cat, PDO::PARAM_STR , 60);
        $c->execute();
        $c->closeCursor();
        $this->errormsg = ($c->rowCount()===1) ? "Insertion réussie" : "Aucune insertion";
    }
 
    public function update()
    {    
        $sql = $this->connect();
        $query = " UPDATE categorie SET cat = :cat WHERE id = :id ";
        $c = $sql->prepare($query);
        $c->bindParam(":id", $this->id, PDO::PARAM_INT);
        $c->bindParam(":cat", $this->cat, PDO::PARAM_STR , 60);
        $c->execute();
        $c->closeCursor();
        $this->errormsg = ($c->rowCount()===1) ? "Modification réussie" : "Aucune modification";
    }
 
    //ici c'est OK
    public function delete()
    {
        $sql = $this->connect();
        $query = " DELETE FROM categorie WHERE id = :primary ";
        $c = $sql->prepare($query);
        $c->bindParam(":primary", $this->id, PDO::PARAM_INT);
        $c->execute();
        $c->closeCursor();
        $this->errormsg = ($c->rowCount()===1) ? "Effacement réussie" : "Aucune Effacement";
    }
 
    public function getId()
    {
        return $this->id;
    }
 
    public function getCat()
    {
        return $this->cat;
    }
 
    public function setId($id)
    {
        $this->id = $id;
        return $id;
    }
 
    public function setCat($cat)
    {
        $this->cat = $cat;
        return $cat;
    }
 
    //ici c'est OK
    static public function getList()
    {
        $sql = $this->connect();
        $query = " SELECT * FROM categorie ";
        $c = $sql->query($query);
        $result = $c->fetchAll(PDO::FETCH_OBJ);
        if(!empty($result))
        { 
            return $result;
        }
        return array();
    }
 
    public function message(){
        echo $this->errormsg;
    }
 
}
 
$category = new Categorie();
 
//$category->setCat("");
$category->update();
$category->message();
echo '<br>';
 
foreach (Categorie::getList() as $k): echo $k->id .' '. $k->cat; endforeach;
echo '<br>';
 
?>

08/10/2013, 15h38
sabotage

Ton erreur ici est simple : la variable $sql définie dans la méthode connect() n'existe pas en dehors de connect().
On pourrait corriger ça mais intégrer la connexion à la base de donnée à cette classe, et encore pire, appeller la construction de l'objet PDO à chaque requête, sont des erreurs de conception.

Renseigne toi plutôt sur la construction d'un singleton.
08/10/2013, 20h59
dancom5

Un singleton qui engloberait que la gestion de connexion?
08/10/2013, 21h22
papajoker

Pour un singleton,
sujet vu et revu dans le forum

http://www.developpez.net/forums/d12...ton-pdo-class/

Mais le singleton(seul) est passé de mode; même déconseillé si l'on désire utiliser les toutes dernières techniques poo. Je te rassures, dans ton cas, pas d problème, même conseillé (autres techniques très lourdes).
09/10/2013, 01h25
dancom5

Je vais suivre vos conseils qui sont toujours intéressants et appréciables :P

Merci pour le liens.

Je vais tester tout de sutie.

révision du code

J'ai fait une révision du code qui ressemble plus à un singleton.
Fatal error: Call to undefined method SPDO::prepare() in

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
<?php
header('Content-Type: text/html; charset=utf-8');
 
class SPDO
{
    private $PDOInstance = null;
    private static $instance = null;
    const DEFAULT_SQL_USER = 'root';
    const DEFAULT_SQL_HOST = 'localhost';
    const DEFAULT_SQL_PASS = '';
    const DEFAULT_SQL_DTB = 'bdd';
 
    private function __construct()
    {
        $this->PDOInstance = new PDO('mysql:dbname='.self::DEFAULT_SQL_DTB.';host='.self::DEFAULT_SQL_HOST,self::DEFAULT_SQL_USER ,self::DEFAULT_SQL_PASS);
    }
 
    public static function getInstance()
    {
        if(is_null(self::$instance))
        {
            self::$instance = new SPDO();
        }
        return self::$instance;
    }
 
    public function query($query)
    {
        return $this->PDOInstance->query($query);
    }
}
 
class Categorie
{
    private $id;
    private $cat;
 
    public function __construct($id=0,$cat="") {
        $this->id=$id;
        $this->cat=$cat;
        $this->errormsg="";
    }
 
    public function insert()
    {
        $query = " INSERT INTO categorie (cat) VALUES (:cat) ";
        $c = SPDO::getInstance()->prepare($query);
        $c->bindParam(":cat", $this->cat, PDO::PARAM_STR , 60);
        $c->execute();
        $c->closeCursor();
        $this->errormsg = ($c->rowCount()===1) ? "Insertion réussie" : "Aucune insertion";
    }
 
    public function update()
    {    
        $query = " UPDATE categorie SET cat = :cat WHERE id = :id ";
        $c = SPDO::getInstance()->prepare($query);
        $c->bindParam(":id", $this->id, PDO::PARAM_INT);
        $c->bindParam(":cat", $this->cat, PDO::PARAM_STR , 60);
        $c->execute();
        $c->closeCursor();
        $this->errormsg = ($c->rowCount()===1) ? "Modification réussie" : "Aucune modification";
    }
 
    //ici c'est OK
    public function delete()
    {
        $query = " DELETE FROM categorie WHERE id = :primary ";
        $c = SPDO::getInstance()->prepare($query);
        $c->bindParam(":primary", $this->id, PDO::PARAM_INT);
        $c->execute();
        $c->closeCursor();
        $this->errormsg = ($c->rowCount()===1) ? "Effacement réussie" : "Aucune Effacement";
    }
 
    public function getId()
    {
        return $this->id;
    }
 
    public function getCat()
    {
        return $this->cat;
    }
 
    public function setId($id)
    {
        $this->id = $id;
        return $id;
    }
 
    public function setCat($cat)
    {
        $this->cat = $cat;
        return $cat;
    }
 
    //ici c'est OK
    static public function getList()
    {
        $query = " SELECT * FROM categorie ";
        $c = SPDO::getInstance()->query($query);
        $result = $c->fetchAll(PDO::FETCH_OBJ);
        if(!empty($result))
        { 
            return $result;
        }
        return array();
    }
 
    public function message(){
        echo $this->errormsg;
    }
 
}
 
$category = new Categorie();
 
//$category->setCat("");
$category->update();
$category->message();
echo '<br>';
 
foreach (Categorie::getList() as $k): echo $k->id .' '. $k->cat; endforeach;
echo '<br>';
 
?>

09/10/2013, 02h50
papajoker

Oui normal, il faut ajouter la methode SPDO public prepare
exactement comme "query()"

c'est la base de la poo :)

edit : bien dancom5 :)

Tien, il me manquait une fonction:
Code:

1 2 3 4 public function prepare($query) { return $this->PDOInstance->prepare($query); }
ça fonctionne maintenant.

il manque dans methode insert
$this->id= SPDO::lastInsertId();

Code:

1
2
3
4
5
6
7
8
9
10
public function insert()
    {
        $query = " INSERT INTO categorie (cat) VALUES (:cat) ";
        $c = SPDO::getInstance()->prepare($query);
        $c->bindParam(":cat", $this->cat, PDO::PARAM_STR , 60);
        $c->execute();
$this->id= SPDO::lastInsertId();
        $c->closeCursor();
        $this->errormsg = ($c->rowCount()===1) ? "Insertion réussie" : "Aucune insertion";
    }

et bien sur aussi faire une methode public SPDO::lastInsertId();

09/10/2013, 03h34
dancom5
c'est dans mon singleton que je devrais mettre ça?
Code:

1 2 3 4 5 public function lastInsertId() { return $this->PDOInstance->lastInsertId(); } }
Avec j'obtiens comme erreur :
Notice: Undefined property: Categorie::$PDOInstance in
et
Fatal error: Call to a member function lastInsertId() on a non-object in

Merci pour les précisions.
09/10/2013, 03h41
papajoker

Citation:

Undefined property: Categorie::$PDOInstance in

a traduire en fr :

propriété $PDOInstance non trouvée dans Categorie !!
09/10/2013, 04h15
dancom5

Propriété indéfini.. mais je ne vois pas mon erreur.
09/10/2013, 06h26
dancom5
Franchement :(
Code:

1 2 3 4 public function lastInsertId($query) { return $this->PDOInstance->lastInsertId($query); }
comme fonction dans mon singleton.

Code:

$c = SPDO::lastInsertId($query);

Dans ma classe catergorie.

J'ai essayé

Code:

return $this->PDOInstance->lastInsertId();

Non plus. Et rien n'y fait.. si j ai fait de quoi pas correcte.. faite-moi savoir svp.

Si je ne mets pas lastInsertId, ça va fonctionner.
09/10/2013, 07h01
dancom5

Citation:

Envoyé par papajoker

il manque dans methode insert
$this->id= SPDO::lastInsertId();

[code]public function insert()
...
$this->id= SPDO::lastInsertId();
...

Si je remplace par ceci:

Code:

$this->id = SPDO::getInstance()->lastInsertId();

ça fonctionne.
09/10/2013, 17h33
chico_du_33

au lieu de redéfinir toutes les méthodes de PDO dans SPDO, utilise __call, tu gagneras du temps !

Ma requête actuelle:

Code:

1
2
3
4
5
6
7
8
9
10
11
static public function getList()
    {
        $query = " SELECT * FROM categorie ";
        $c = SPDO::getInstance()->query($query);
        $result = $c->fetchAll(PDO::FETCH_OBJ);
        if(!empty($result))
        { 
            return $result;
        }
        return array();
    }

J'ai tenter d'ajouter un critère avec WHERE et BIND, je n'y arrive pas.

Je cherche à pouvoir établir des critères.

et ici:

Code:

1
2
3
4
5
6
7
8
9
10
    public function update()
    {    
        $query = " UPDATE categorie SET cat = :cat WHERE id = :id ";
        $c = SPDO::getInstance()->prepare($query);
        $c->bindParam(":id", $this->id, PDO::PARAM_INT);
        $c->bindParam(":cat", $this->cat, PDO::PARAM_STR , 60);
        $c->execute();
        $c->closeCursor();
        $this->errormsg = ($c->rowCount()===1) ? "Modification réussie du ID $this->id" : "Aucune modification";
    }

J'ai tenter d'afficher à Modification réussie.... $this->cat, ça n'affiche rien.
Je cherche à afficher le catalogue.

En fin de compte, c'est que je cherche à avoir la possibilité d'avoir plusieurs types d'affichages soit par critères.

23/10/2013, 09h33
dancom5

J'ai tout revu depuis un moment. Puisque mon but principal était de réaliser un CRUD de base en utilisant une class et un singleton. Je fais un nouveau topic la dessus.