static::methode() comportement étrange

Version imprimable

Bonjour,
Cette question fait suite à la discussion acces-classe-fille-non-instanciee.
Suite à cette discussion, j'ai mis en pratique le static::method(). Cela fonctionne globalement bien, mis à part un cas que je ne comprend pas: J'expose ici un cas de synthèse qui selon moi représente bien le cas que je rencontre:
J'ai 3 classes: Classe1, ClasseMere et ClasseFille.
Bien entendu, ClasseFille est une sous Classe de ClasseMere.
Dans Classe1, une méthode methodeClasse1 appelle une methode statique de ClasseFille qui est définie dans ClasseMere : methodeClasseMere().
Cette même méthode fait appel (ou devrait faire appel) à une méthode statique de ClasseFille grace à static::methodeClasseFille().
Et là, c'est le drame car php m'explique que :

Citation:

Fatal error: Call to undefined method Classe1::methodeClasseFille() in blabla\ClasseMere.php on line 86

Ci dessous du code plus concret
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 abstract class ClasseMere { abstract static function methodeClasseFille(); function methodeClasseMere() { static::methodeClasseFille(); } } class ClasseFille extends ClasseMere { static function methodeClasseFille() { print("hello"); } } class Classe1 { function methodeClasse1() { ClasseFille::MethodeClasseMere(); } } $objet=new Classe1(); $objet->methodeClasse1();
Je cherche à ce que methodeClasseMere appelle ClasseFille::MethodeClasseFille() et non Classe1::MethodeClasseFille(). Lorsque je lance un get_called_class() depuis methodeClasseMere, j'obtiens aussi Classe1. Donc, je ne comprend vraiment pas ce qui se passe. J'ai essayé d'aller voir sur http://php.net/manual/fr/language.oo...c-bindings.php. Je n'y ai pas trouvé le problème que j'évoque.
Merci d'avance pour votre aide

11/10/2010, 14h27
stealth35

Code:

static function methodeClasseMere()

par contre : abstract + static, ca se fais pas

Code:

Strict Standards: Static function ClasseMere::methodeClasseFille() should not be abstract

Salut

Tu fais référence à un topic, et tu n'as pas tenu compte d'une remarque qui disait que la classe Parente ne devrait pas faire appel à une classe Enfant.
Théoriquement, la classe Parente ne devrait pas faire ce qui relève de l'Enfant, sinon la Parente devient totalement dépendante de la Fille.

Si on suit le raisonnement du code, on évoque au départ la classe Fille, qui elle appelle une méthode de la classe Mère dont cette classe Mère qui devrait appeler une méthode de la classe Fille du départ.

Ceci ne reviendrait il pas à évoquer une classe Fille appelant SA méthode (ClasseFille::methodeClasseFille) ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 abstract class ClasseMere { abstract function methodeClasseFille(); } class ClasseFille extends ClasseMere { function methodeClasseFille() { print("hello"); } } class Classe1 { function methodeClasse1() { $fille = new ClasseFille(); $fille->methodeClasseFille(); } } $objet=new Classe1(); $objet->methodeClasse1();
En quoi ceci ne conviendrait il pas dans ton cas ?

S'il y a des traitements communs devraient faire avant ou après que la Méthode methodeClasseFille() soit appelée, peut être faudrait il créer une méthode dédiée à ça dans la classe Mère.
Une classe Fille héritant des méthodes Parentes, on peu les appeler quand on veut (avant ou après).

11/10/2010, 14h59
a066043

Bonjour RunCodePhp,
Merci de t'intéresser à ma question.
Effectivement, cela conviendrait. C'est effectivement comme cela qu'était structuré mon code auparavant.
Cependant, mon objectif est de factoriser le code et c'est pour cela que j'ai créé la méthode methodeClasseMere(). Celle ci contient l'ensemble du code qui est valable pour toutes les classes filles et devrait appeler ce qui est spécifique de chaque classe par la méthode methodeClasseFille().
C'est pour cela que j'ai structuré le code de cette manière, chose que je ne parvenais pas à faire en statique auparavant et que je peux faire grâce à static::blabla() mais qui comme vous le voyez ne fonctionne pas dans le cas particulier évoqué.
11/10/2010, 15h00
stealth35

Citation:

Envoyé par a066043

C'est pour cela que j'ai structuré le code de cette manière, chose que je ne parvenais pas à faire en statique auparavant et que je peux faire grâce à static::blabla() mais qui comme vous le voyez ne fonctionne pas dans le cas particulier évoqué.

si mais t'avais oublier de mettre un static
11/10/2010, 15h12
a066043

Réponse à stealth35

Bonjour,
Ok, il me semble que tu m'avais déjà fais cette remarque.
J'ai donc essayé de rendre la méthode concernée non abstract. J'ai même supprimé le abstract de la classeMere, mais cela n'a rien changé. Il me dit toujours que la méthode n'existe pas sur Classe1:cry:
Par ailleurs, pour ma culture, sais tu quelle est la raison qui interdit d'avoir cette méthode en abstract. J'arriverai à m'en passer s'il le faut mais ceci-dit, à partir du moment où tu appelle une méthode d'une classe fille depuis la classe mère, il parait nécessaire de s'assurer que la méthode de la classe fille existe. Or, il me semble bien que abstract soit utile pour cela.

Citation:

Envoyé par a066043

Bonjour,
Ok, il me semble que tu m'avais déjà fais cette remarque.
J'ai donc essayé de rendre la méthode concernée non abstract. J'ai même supprimé le abstract de la classeMere, mais cela n'a rien changé. Il me dit toujours que la méthode n'existe pas sur Classe1:cry:
Par ailleurs, pour ma culture, sais tu quelle est la raison qui interdit d'avoir cette méthode en abstract. J'arriverai à m'en passer s'il le faut mais ceci-dit, à partir du moment où tu appelle une méthode d'une classe fille depuis la classe mère, il parait nécessaire de s'assurer que la méthode de la classe fille existe. Or, il me semble bien que abstract soit utile pour cela.

parce que faut pas confondre abstract et interface ;)
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 interface ClasseFilleInterface { public static function methodeClasseFille(); } abstract class ClasseMere { public static function methodeClasseMere() { static::methodeClasseFille(); } } class ClasseFille extends ClasseMere implements ClasseFilleInterface { public static function methodeClasseFille() { print("hello"); } } class Classe1 { public function methodeClasse1() { ClasseFille::MethodeClasseMere(); } } $objet=new Classe1(); $objet->methodeClasse1();

11/10/2010, 15h29
a066043

Résolu

Bon, alors tout d'abord merci Stealth35 de m'avoir encore permis de règler un problème.
Je m'explique: Je n'avais pas bien lu ton message ou au moins la partie qui réglait mon problème:

Citation:

Code :

static function methodeClasseMere()

par contre : abstract + static, ca se fais pas

En fait, j'ai effectivement omis de déclarer static la méthode methodeClasseMere. Cela a du troubler monsieur Php.
Maintenant, cela fonctionne.
Pour revenir à la question du abstract + static, je suis d'accord avec le fait que abstarct et interface ne soient pas pareil, mais j'ai bien besoin dans une classe mère qui ne soit pas une interface d'obliger ses filles à avoir certaines méthodes. Comment faire??????

Citation:

mais j'ai bien besoin dans une classe mère qui ne soit pas une interface d'obliger ses filles à avoir certaines méthodes. Comment faire??????

C'est abstract normalement.
Mais en faisant quelques essais vite fait (je ne maitrise pas tout ça non plus), il faut juste déclarer les méthodes abstract au niveau de la classe Mere, et c'est au niveau des classes Filles où ce trouveront les traitements à faire.

Si par contre on crée des méthodes non abstraites au niveau de la classe Mère, alors il y aura rien qui obligera au Filles d'avoir ces mêmes méthode.

Citation:

Envoyé par stealth35

par contre : abstract + static, ca se fais pas

Là par contre je ne vois pas pourquoi ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 abstract class Transport { abstract static function roues(); } class Voiture extends Transport { public static function roues() { echo '4 roues'; } } class Velo extends Transport { public static function roues() { echo '2 roues'; } } Voiture::roues(); // Affiche : 4 roues Velo::roues(); // Affiche : 2 roues
Si je fais cet essai, ça fonctionne très bien.
Au niveau conception, je ne vois pas non plus qu'est ce qui n'irait pas ?

Par contre, et même pour des raisons de Factorisation, j'ai du mal à concevoir qu'une classe Fille soit obligée de passer par la classe Mère pour appeler sa propre méthode.
Dans ton exemple, faire ceci reviendrait au même :

Code:

ClasseFille::methodeClasseFille();

La seule chose que je vois, c'est qu'il y aurait dans la classe Mère ClasseMere
::methodeClasseMere() des traitements autres que l'appel à static::methodeClasseFille()

Dans tel cas peut être qu'au niveau de la classe fille il faudrait appeler la même méthode Parente :
Exemple :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 class Transport { public function roues() { // Traitement propre au parent echo 'Transport à '; } } class Voiture extends Transport { public function roues() { parrent::roues(); // On appel les traitement du parent echo '4 roues'; } } $voiture = new Voiture(); $voiture->roues();
Devrait afficher : Transport à 4 roues

Citation:
Envoyé par RunCodePhp
Là par contre je ne vois pas pourquoi ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 abstract class Transport { abstract static function roues(); } class Voiture extends Transport { public static function roues() { echo '4 roues'; } } class Velo extends Transport { public static function roues() { echo '2 roues'; } } Voiture::roues(); // Affiche : 4 roues Velo::roues(); // Affiche : 2 roues
Si je fais cet essai, ça fonctionne très bien.
Au niveau conception, je ne vois pas non plus qu'est ce qui n'irait pas ?
Strict Standards: Static function Transport::roues() should not be abstract

11/10/2010, 17h25
RunCodePhp

Citation:

Envoyé par stealth35

Strict Standards: Static function Transport::roues() should not be abstract

Ca date depuis quand ça ? (Je suis en Php5.3.0)
Je suis en strict pourtant et je n'ai pas cette erreur :cry:

Bon, jusqu'à lors j'ai jamais mis de méthode abstraites en statique, à part cet essai, donc tout va bien.
C'est bon à savoir en tout cas. ;)
11/10/2010, 17h35
stealth35

Citation:

Envoyé par RunCodePhp

Ca date depuis quand ça ? (Je suis en Php5.3.0)
Je suis en strict pourtant et je n'ai pas cette erreur :cry:

depuis php 5.2 c'était comme ca, a la version 5.3.0 c'est revenu et c'est reparti après
11/10/2010, 17h40
stealth35

tout ca c'est une histoire de visibilité, comme tu peux pas faire du abstract private ;)
11/10/2010, 17h47
RunCodePhp

Citation:

Envoyé par stealth35

depuis php 5.2 c'était comme ca, a la version 5.3.0 c'est revenu et c'est reparti après

:aie:

C'est Php6 qui doivent leur faire perdre la raison :mrgreen:

Mais qu'est ce qu'il y a de tordu à déclarer des méthodes abstraites en statique selon toi ? (comme la Transport par exemple)
De mon coté je ne parviens pas à voir où est le problème.
11/10/2010, 18h15
stealth35

Citation:

Envoyé par RunCodePhp

:aie:

C'est Php6 qui doivent leur faire perdre la raison :mrgreen:

Mais qu'est ce qu'il y a de tordu à déclarer des méthodes abstraites en statique selon toi ? (comme la Transport par exemple)
De mon coté je ne parviens pas à voir où est le problème.

parce que ca n'a de sens de déclarer une méthode abstraite statique parce que une méthode statique appartient toujours à a la classe. donc si tu la déclares comme abstraite, elle est ignoré et elle appartient plus à la classe ainsi qu'a chaque sous-classe ;)
11/10/2010, 19h31
RunCodePhp

Citation:

Envoyé par stealth35

donc si tu la déclares comme abstraite, elle est ignoré et elle appartient plus à la classe ainsi qu'a chaque sous-classe ;)

Je vois ce que tu veux dire, mais je t'avouerais que je vais plus l'admettre qu'autre chose.
Ce n'est pas d'une limpidité extrême tout ça :mrgreen:
On dit que l'Objet c'est abstrait, c'est peu de l'dire ...

Merci pour ces explications ;)
11/10/2010, 19h47
stealth35

Citation:

Envoyé par RunCodePhp

Je vois ce que tu veux dire, mais je t'avouerais que je vais plus l'admettre qu'autre chose.
Ce n'est pas d'une limpidité extrême tout ça :mrgreen:
On dit que l'Objet c'est abstrait, c'est peu de l'dire ...

Merci pour ces explications ;)

oui c'est spé, je sais pas pourquoi, pour moi ca parait logique, mais j'arrive pas a l'expliquer simplement, pense au abstract private c'est pareil

y'a eu des demandes pour faire des classes statique : http://wiki.php.net/rfc/static-classes
11/10/2010, 20h45
RunCodePhp

Citation:

Envoyé par stealth35

pense au abstract private c'est pareil

C'est bizarre, car là c'est nettement plus limpide.
Vu qu'il n'est pas permis d'instancier une classe abstraite, une portée privée fera que la méthode ne pourra jamais être atteinte.

Un des ces 4 cette histoire de statique sera mieux comprise mais pour le moment c'est encore flou.
Disons que je comprends que déclarer une méthode statique au niveau de la classe abstraite, pourquoi pas, mais que cela n'empêcherait pas de le faire au niveau des classes Enfants.
C'est cette restriction au niveau des classes Enfants qui me dérange.