Class accès BD : Call to undefined method db::query()

Version imprimable

Bonjour,

j'ai une fonction que je souhaite transformer en classe (c'est la première)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
function libatt($table,$pdo) {
        $sql = "select Att_Lib as lib from attribut where Att_Table = '$table' order by Att_ordre";
        try {
            $result = $pdo->query($sql);
            $nblg=$result->rowCount();   
        }
        catch (PDOException $e)
        {
            $error = 'Erreur Recup liste attribut : '.$table . $e->getMessage();
            include '../include/error.php.html';
            exit();
        }
        if ($nblg == 0) { $lib= ''; }
        else { 
            foreach ($result as $row)
            {
                $lib[]=$row['lib'];
            }
        }
        return $lib;
}

Elle fonctionne parfaitement, le $pdo = new PDO($ConnexionString, $username,$password);.

J'ai donc créer une classe (merci a ceux qui m'on aider avec les $this)

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
class db {
    private $pdo;
    private $instance=null;
    private $options = array(PDO::MYSQL_ATTR_INIT_COMMAND    => "SET NAMES utf8" );
    private $host = 'localhost';
    private $db_name = 't2i_erp';
    private $user = 'root';
    private $pass ='';    
    private $dns = '';
 
    public function __construct() {
        try {
            $this->dns = "mysql:host=$this->host;dbname=$this->db_name";
            $this->instance = new PDO($this->dns,  $this->user,  $this->pass,  $this->options);
            $this->instance->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        }
        catch (PDOException $e)
        {
            $error='Impossible de se connecter à la base de donnée '.$e->getMessage(). '. '.  $dns.'. '.  $this->user.'. '.  $this->pass;
            include'error.php.html';
            exit();
        }
    }
 
    public function getinstance() {
        return $this->instance;
    }
}

J'ai fait un test avec $npdo=newdb();.

Maintenant pour transformer ma fonction en class

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
class attribut {
    protected $db;
    var $lib =null;
 
    function __construct($db=null) {
        if ($db == null) { 
            $this->db = new db();
            $this->db->getinstance();
        } 
        else { 
            $this->db = $db;
        }
    }
    public function libatt($table) {
        $sql = "select Att_Lib as lib from attribut where Att_Table = $table order by Att_ordre";
        try {
            $result = $this->db->query($sql);
            $nblg=$result->rowCount();   
        }
        catch (PDOException $e)
        {
            $error = 'Erreur Recup liste attribut : '.$table . $e->getMessage();
            include '../../include/error.php.html';
            exit();
        }
        if ($nblg == 0) { $this->lib= ''; }
        else { 
            foreach ($result as $row)
            {
                $this->lib[]=$row['lib'];
            }
        }
        return $this->lib;
    }
}

et cela plante sur la ligne $result = $this->db->query($sql);

Code:

1
2
$attlib=new attribut();
$ltype = $attlib->libatt('contrat');

avec le message

Citation:

Call to undefined method db::query()

comme si mon db n'était pas un object de type PDO.

Merci pour votre aide.

06/04/2013, 11h09
RunCodePhp
Salut

A mon avis l'erreur serait que tu n'aurais pas inclus le fichier contenant le code de ta classe db, du coup la propriété db de la classe attribut aurait comme valeur NULL.

Juste pour tester ça, suffit d'afficher la valeur de db, genre :
Code:

1 2 $attlib = new attribut(); var_dump($attlib->db);
Par ailleurs, tu devrais consulter le forum PDO, et rechercher des exemples de code (tu en trouveras c'est certain) sur le comment créer un singleton avec PDO.
A mon sens ce sera plus pratique, mais surtout je ne serais pas étonné que cela correspondrait mieux à ce que tu compte faire.

Bonjour,

Comme je ne sais pas ce que c'est un singleton PDO, j'aurais du mal a savoir de quoi tu parle.

J'ai fait une recherche sur internet et j'ai trouvé les exemples suivants dont je me suis inspiré :
http://codereview.stackexchange.com/...fety-and-logic
http://stackoverflow.com/questions/1...across-classes
http://stackoverflow.com/questions/2...pdo-in-classes
http://www.atelierphp.net/commons/sh...db_lecture.php

Mon besoin est "simple", je veux créer un class db pour la connexion et ensuite faire une class par objet (table mysql). Et donc ne plus avoir le $pdo dans les appels de fonctions (entre autre).

Je n'ai peut être pas pris les bons exemples, mais j'ai eu beaucoup de mal a trouvé des exemples qui correspond à mon besoin (ou j'ai pas fait la bonne recherche google). Je passe mon temps sur Google depuis 1 mois que j'ai débuté.

pour finir var_dump($attlib->db)

Code:

Fatal error: Cannot access protected property attribut::$db

merci

J'ai modifié protected en public et j'obtiens
Code:

1 2 3 4 5 6 7 8 9 10 11 12 object(db)[5] private 'pdo' => null private 'instance' => object(PDO)[6] private 'options' => array (size=1) 1002 => string 'SET NAMES utf8' (length=14) private 'host' => string 'localhost' (length=9) private 'db_name' => string '_erp' (length=7) private 'user' => string 'root' (length=4) private 'pass' => string '' (length=0) private 'dns' => string 'mysql:host=localhost;dbname=_erp' (length=35)

06/04/2013, 12h39
tornade69
Merci beaucoup,

c'est le mot singleton qu'il me manquait !!!

en fait j'avais une double erreurs :
Code:

1 2 3 4 class db { .... public $instance=null; ...
au lieu de private initialement, et ensuite dans la class attribut
Code:

1 2 try { $result = $this->db->instance->query($sql);
Et donc maintenant cela semble fonctionner

Info trouvée sur un autre site.

Merci à toi