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

**dancom5** · 08/10/2013, 07h18

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 :

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

**sabotage** · 08/10/2013, 12h04

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.

**dancom5** · 08/10/2013, 12h46

Tien, j'ai changé :

Code :

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

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

pour :

Code :

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

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

J'ai ajouté une nouvelle fonction:

Code :

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

1
2
3
public function message(){
        echo $this->errormsg;
    }

et l'objet:

Code :

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

$category->message();

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

Merci pour l'indication sabotage.

**dancom5** · 08/10/2013, 15h26

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.

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

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
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>';
 
?>

**sabotage** · 08/10/2013, 15h38

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.

**dancom5** · 08/10/2013, 20h59

Un singleton qui engloberait que la gestion de connexion?

**papajoker** · 08/10/2013, 21h22

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

**dancom5** · 09/10/2013, 01h25

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

Merci pour le liens.

Je vais tester tout de sutie.

**dancom5** · 09/10/2013, 02h22

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

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
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>';
 
?>

**papajoker** · 09/10/2013, 02h50

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

c'est la base de la poo

edit : bien dancom5

**dancom5** · 09/10/2013, 02h51

Tien, il me manquait une fonction:

Code :

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

1
2
3
4
    public function prepare($query)
    {
        return $this->PDOInstance->prepare($query);
    }

ça fonctionne maintenant.

**papajoker** · 09/10/2013, 03h06

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

Code :

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

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();

**dancom5** · 09/10/2013, 03h34

c'est dans mon singleton que je devrais mettre ça?

Code :

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

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.

**papajoker** · 09/10/2013, 03h41

Undefined property: Categorie::$PDOInstance in

a traduire en fr :

propriété $PDOInstance non trouvée dans Categorie !!

**dancom5** · 09/10/2013, 04h15

Propriété indéfini.. mais je ne vois pas mon erreur.

**dancom5** · 09/10/2013, 06h26

Franchement

Code :

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

1
2
3
4
public function lastInsertId($query)
    {
        return $this->PDOInstance->lastInsertId($query);
    }

comme fonction dans mon singleton.

Code :

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

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

Dans ma classe catergorie.

J'ai essayé

Code :

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

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.

**dancom5** · 09/10/2013, 07h01

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 :

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

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

ça fonctionne.

**chico_du_33** · 09/10/2013, 17h33

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

**dancom5** · 09/10/2013, 21h40

Ma requête actuelle:

Code :

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

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 :

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

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.

**dancom5** · 23/10/2013, 09h33

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.

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

PHP & Base de données

Discussions similaires

Partager

Partager