Discussion :

[rawsrc]

Bonjour à tous,
Comment faire pour savoir si une classe dérivée composée que de fonctions statiques implemente une interface spécifique quand la fonction appelée par la dérivée est localisée dans la classe parente.
Je sais pas si c'est super clair, voilà un petit exemple bien plus parlant :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
<?php

interface iTest { }

class base {
public static function create() {
if (self instanceof iTest) echo 'iTest';
}
}

class child extends base implements iTest { }

child::create();

Le problème se situe avec le self qui ne gère pas le LSB
J'ai essayé avec get_called_class() mais là plus question d'objet juste une string. Et à chaque fois le test échoue child n'implémente pas iTest !

[s.n.a.f.u]

Salut,

Ne pourrais-tu pas utiliser le mot clé static : http://php.net/manual/en/language.oo...c-bindings.php (example #2).

Je n'ai pas de php5.3 sous la main, donc pas de tests...

[rawsrc]

Alors j'ai bien essayé avec static mais PHP m'envoie bouler : PHP Parse error: syntax error, unexpected T_INSTANCEOF, expecting T_PAAMAYIM_NEKUDOTAYIM

Voili voilou,

Je suis en train de voir si avec ReflectionClass il n'y aurait pas moyen de bidouiller
Si tu as d'autres idées je suis preneur bien évidemment :-)

J'ai essayé ça : plus de problèmes mais return __CLASS__ me renvoie une string

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
 
 
interface iTest { }
 
class base {	
    public static function create() {
        $r = (static::self() instanceOf iTest);
        if ($r) echo 'iTest';
    }
}
 
class child extends base implements iTest { 
    public static function self() {
        return __CLASS__;
    }
}

Merci

[s.n.a.f.u]

expecting T_PAAMAYIM_NEKUDOTAYIM

Tu as bien mis le double deux-points :: ?

[s.n.a.f.u]

Et je n'ai peut-être pas assez précisé : personellement, j'essaierais un truc de ce genre

Code :

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

 if (static::get_class() == 'iTest') echo 'iTest';

=> il faut bien sûr oublier le instanceof car il n'y a aucune instance en appel statique

[rawsrc]

avec l'essai conseillé, j'obtient le nom de la classe child et rien concernant ses implémentations

[rawsrc]

Bon après plusieurs essais j'ai trouvé une solution au problème :

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
 
interface iTest { }
 
class base {	
   public static function create() {
      $r = new ReflectionClass(static::$SELF);
      if (in_array('iTest', $r->getInterfaceNames())) echo 'iTest';
   }
}
 
class child extends base implements iTest {	
   protected static $SELF = __CLASS__;
}
 
child::create();
 
?>

Bon c'est pas super propre mais pour l'instant je n'ai rien de mieux en magasin

[s.n.a.f.u]

Bon c'est pas super propre mais pour l'instant je n'ai rien de mieux en magasin

C'est quand même pas mal vu le besoin un peu tordu.
Bien joué

[rawsrc]

Besoin tordu ! Elle est bonne celle-là. MOUAHAHAHAHA

Blagues à part, j'ai recherché sur www.php.net cette fonctionnalité et n'ayant rien trouvé, j'en ai profité pour soumettre cette "feature request" à la communauté.

Voili voilou
Bonne nuit

[rawsrc]

Je vous fait part de la réponse de la communauté php concernant le problème ci-dessus.

Pour avoir toutes les interfaces implémentées dans un contexte static utiliser : $interfaces = class_implements(get_called_class());
(class_implements gère pour nous la réflection)
Ce qui renvoie un tableau comme ma soluce utilisant ReflectionClass sauf que là c'est plus propre et pas de $self à trainer donc voici la solution finale :

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
<?php
 
interface iTest { }
 
class base {	
   public static function create() {
      if (in_array('iTest', class_implements(get_called_class()))) echo 'iTest';
   }
}
 
class child extends base implements iTest { }
 
child::create();
 
?>

Je pense malgré tout qu'il serait mieux d'avoir un système unifié comme instanceof pour tout. Allez soyons fous : pour englober tous les contextes : je propse de remplacer instanceof par uses et puis il faudrait aussi que get_called_class() renvoie autre chose qu'une string.

Code :

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

1
2
3
<?php
if (get_called_class() uses iTest) echo 'iTest'
?>

Bref ça ne sert à rien mais cela m'a permis d'étrenner mon nouveau clavier MOUAHAHA

[s.n.a.f.u]

Merci pour le suivi

[ThomasR]

Bonjour,

Juste pour info, ce que tu désirais faire à l'origine n'avais pas de sens car tu essayais de chercher une instance sur une méthode statique, ce qui n'existe pas.

e pense malgré tout qu'il serait mieux d'avoir un système unifié comme instanceof pour tout

instanceof compare des instances, pas des types.

[rawsrc]

Bonjour

ce que tu désirais faire à l'origine n'avais pas de sens

Je le savais dès le départ qu'avec instanceof ça n'allait pas être gagné vu que la classe child n'était qu'un conteneur et n'avait pas d'instance. Enfin à l'utilisation child a bien une existence car get_called_class() renvoie bien la résolution statique à la volée. La logique aurait voulu que qu'elle que soit la manière d'exister on puisse interroger de la même manière les propriétés de cette existence. Mais bon, je sais parfois que la logique...

M'est d'avis que tous ces procédés : instanceof, ReflectionClass ou le sucre syntaxique (class_implements->array) sont bordéliques. C'est pour cela que je parlais d'unification.