Discussion :

[tornade69]

Bonjour,

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

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

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

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

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

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

avec le message

Call to undefined method db::query()

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

Merci pour votre aide.

[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 :

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

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.

[tornade69]

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 :

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

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

merci

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

Code :

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

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)

[tornade69]

Merci beaucoup,

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

en fait j'avais une double erreurs :

Code :

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

1
2
3
4
class db {
....
    public $instance=null;
...

au lieu de private initialement, et ensuite dans la class attribut

Code :

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

1
2
try {
            $result = $this->db->instance->query($sql);

Et donc maintenant cela semble fonctionner

Info trouvée sur un autre site.

Merci à toi