Discussion :

[WebDream]

Bonjour, je pose le contexte de mon code.

J'ai une classe abstraite sqlObject :

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
<?php
 
abstract class sqlObject {
 
	public function getField($champ) {
 
		return $this->$champ;
 
	} // getField
 
	public static function getAllObjects() {
 
		global $db
 
		$sql = "SELECT * FROM ".TABLE_NAME;
 
		$rows = $db->query($sql);
 
		return $rows;
 
	} // getAllObjects
 
} // sqlObject

et une classe "Section" qui hérite de la classe sqlObject, dont voici un extrait :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
<?php
 
class Section extends sqlObject {
 
	const TABLE_NAME = 'section';
 
	private $id_section;
	private $titre_section;
 
	// [...]
 
} // Section

Dans mon code je souhaite récupérer toutes mes "sections" en utilisant une méthode statique héritée de cette façon :

Code :

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

$sections = Section::getAllObjects();

et j'obtiens :

Code :

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

Fatal error: Call to undefined method Section::getallobjects()

La méthode ne semble pas héritée...

Qu'est-ce qui ne va pas dans mon code ?

[Eusebe]

Bonjour,

Je suis étonné que dans ton message d'erreur la fonction soit en minuscules... Alors que dans ton test elle contient des majuscules. As-tu bien testé Section::getAllObjects(); et pas Section::getallobjects() ?

[WebDream]

Certain.

J'ai mis ma fonction en minuscule dans ma classe pour voir si le problème venait de là mais ça ne change rien. Donc je ne sais pas si le fait que le message d'erreur me donne ma fonction en minuscule ou pas, mais le problème ne semble pas venir de là.

[Eusebe]

Pourtant, si j'exécute le code suivant, obtenu après avoir rapidement simplifié le code que tu as fourni, je n'obtiens pas de message d'erreur, et la méthode getAllObjects est bien appelée :

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
<?php
 
abstract class sqlObject
{
    public static function getAllObjects()
    {
        return 'ok';
    } // getAllObjects
 
} // sqlObject
 
class Section extends sqlObject
{
}
 
$sections = Section::getAllObjects();
 
echo $sections;

[mathieu]

essaye ce code seul dans un fichier
il fonctionne bien chez moi (PHP 5.2.5) et affiche "bon"

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
abstract class P 
{
    static public function s()
    {
        echo 'bon';
    }
}
 
class E extends P
{
 
}
 
E::s();

[Bisûnûrs]

Je suis étonné que dans ton message d'erreur la fonction soit en minuscules... Alors que dans ton test elle contient des majuscules. As-tu bien testé Section::getAllObjects(); et pas Section::getallobjects() ?

Les noms de fonctions ne sont pas sensibles à la casse, alors lors d'erreur PHP a tendance à les mettre entièrement en minuscules, même si elles sont déclarées et appelées avec des majuscules.

@WebDream : Ton code, tel que tu nous le montres, fonctionne parfaitement (parse errors corrigées).

[WebDream]

Désolé, effectivement j'avais deux version de la classe Section, et j'ai modifié la mauvaise... :S


Par contre ça ne marche toujours pas même si je n'ai plus la même erreur (j'ai corrigé le point virgule manquant) :

Notice: Use of undefined constant TABLE_NAME - assumed 'TABLE_NAME' in /home.50/t/l/a/tlabs/classes/sqlObject.php on line 15

Lors de l'appel à la méthode getAllObjects, il ne trouve pas la constante TABLE_NAME car elle n'est pas déclarée dans la classe parente...

Vous comprenez le problème ?

[Eusebe]

Bonjour,

il ne faut pas que tu utilises une constante mais une propriété.

Par exemple (non testé) :

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
abstract class sqlObject
{
    protected static $tableName;
 
    public static function getAllObjects()
    {
        global $db;
        $sql = "SELECT * FROM ".self::tableName;
        $rows = $db->query($sql);
        return $rows;
    }
}
 
class Section extends sqlObject
{
	protected static $tableName = 'section';
}

[WebDream]

En fait j'avais déjà essayé avec self::$tableName (il faut le "$" devant pour une propriété statique) mais ça ne marche pas car il cherche la variable $tableName définie dans la classe de la fonction getAllObjects... (or elle est définie dans la classe fille...)

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1' in /home.50/t/l/a/tlabs/classes/Db.php:54 Stack trace: #0 /home.50/t/l/a/tlabs/classes/Db.php(54): PDO->query('SELECT * FROM ')

N'y-a-t-il pas de solution pour passer un paramètre de la classe fille dans une méthode de la classe mère ?

[kaymak]

si tu es en php5.3 il faut utiliser le mot clé static. Autrement, il faut soit utiliser une instance, soit donner le nom de la classe fille ne paramètre.

Ce topic est en complet rapport avec ton nouveau problème http://www.developpez.net/forums/d62...atic-heritage/

bye