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
 
//$db est une instance de PDO
    public function __construct($db) {
        $this->db = $db;
    }
 
//    liste les famille de produits
    private function getListeFamilles() {
        $select = $this->db->prepare('select id_fp, famille from famille_prix');
        $select->execute();
        ...
    }
 
    public static function setListeFamilles() {
        $this->getListeFamilles();
    }

Quand j'appelle la méthode setListeFamilles() dans un fichier secondaire, j'obtiens ce message d'erreur:

Fatal error: Using $this when not in object context...

qui correspond à l'appel de la méthode getListeFamilles() dans setListeFamilles()
Pourquoi ?
merci,

[stealth35]

y'ap s de $this en static que du self (ou static)

[grunk]

Pour étayer un peu la réponse précédente. $this fait référence à ton instance de classe courante.
Pour utiliser une méthode statique pas besoin d'instancier la classe , donc $this n'a pas lieu d'être.

[yann18]

Oui, je pense. C'est pourquoi dans le code secondaire, j'avais bien écrit

Code :

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

Static_Class::setListeFamilles();

Il y a quelque chose qui m'échappe...

[grunk]

Qu'est ce qui t'échappe ?

A parti du moment ou ta méthode est static tu peux pas faire référence à l'instance de la classe (this)
Tu dois donc faire :

Code :

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

1
2
3
public static function setListeFamilles() {
        self::getListeFamilles();
    }

ou encore

Code :

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

static::getListeFamilles()

si jamais le besoin s'en ressent : http://www.php.net/manual/fr/languag...c-bindings.php

Quoi qu'il en soit dans ton cas ça va te poser problème puisque tu semble avoir initialiser un objet bdd à l'instanciation de ta classe. Objet auquel tu n'auras pas accès dans tes méthode static.

[yann18]

Quoi qu'il en soit dans ton cas ça va te poser problème puisque tu semble avoir initialiser un objet bdd à l'instanciation de ta classe. Objet auquel tu n'auras pas accès dans tes méthode static.

Oui, il est clair que je ne peux pas envisager la chose sous cette angle.
Et pour en finir avec cela, getListeFamilles() doit être également en static (?)
Je pensais malgré tout que PHP était capable d'établir le lien entre une méthode "simple" et static (je ne sais pas si je m'exprime bien). Mauvaise compréhension de ma part...
merci !

[rawsrc]

Salut,

Tu peux conserver cette approche (static) en modifiant un peu ta classe :
Regardes :

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
<?php
 
class Foo {
 
   /**
    * Renvoie une connexion vers la base de données
    * @return \PDO
    * @static
    */
   static private function pdo() {
      return Cnx::pdo('db_main');
   }
 
   /**
    * @return array
    * @static
    */
   static function getListeFamilles() {
      $select = self::pdo()->prepare('select id_fp, famille from famille_prix');
      $select->execute();
      # ...
   }
}
 
?>

Tout ce que tu as à faire, c'est d'avoir un classe qui te fournit le PDO au besoin. Ici c'est Cnx::pdo() qui gère toutes mes connexions.
En plus, très souvent une classe travaille presque toujours sur la même connexion donc l'approche static est plutôt bien vue.