[PHP5] ORPHYX PLUGIN SYSTEM

Commentaire de neigedhiver le 04/11/2007 12:46:26

Salut,

Merci pour ton commentaire.

"Quand on passe un nom de plugin et une racine, tu fous ça directement dans tes instances sans vérifier que ce plugin n'y est pas déjà activé. Pourquoi ?"
=> Héhé... Parce que j'ai oublié de faire la vérification :o)

"Et ensuite, tu retournes l'instance seulement si on ne passe pas de nom MAIS qu'on passe une racine. Là encore, que vient foutre la racine au milieu?"
=> Non, c'est le contraire ! Je ne retourne l'instance que si on demande son nom, sans spécifier de racine !
Ta question m'embrouillait, mais tu as du te mélanger les noeils :

  public static function getInstance($sName, $sRoot=NULL)

Si on passe un nom en argument, on retourne le pluginManager qui correspond à ce nom. Le chemin, il le connait déjà.
C'est pour ça que j'ai zappé la vérification dont tu soulignes, à juste titre, l'oubli : c'est parce que je considérais (à tort) qu'un pluginManager s'instancie une fois avec les deux paramètres, et qu'après, on en récupère l'instance en n'y passant forcément qu'un seul (le nom). Or, c'est pas sécurisé, parce que je préjuge ainsi que le développeur sera sage et fera bien comme on lui dit.
Je vais donc rajouter ce test cet après-midi.

Commentaire de Teclis01 le 05/11/2007 00:23:25

Je me permet d'ajouter un commentaire pour pouvoir noter (c'est chiant ça! ;) )
Donc un 10/10 pour la propreté et l'utilité que cela peut avoir

Commentaire de malalam le 06/11/2007 08:05:21 administrateur CS

@neige =>
# Root not specified, and instance exists
elseif(!is_null($sRoot) && isset(self::$aInstances[$sName])) {
    return self::$aInstances[$sName];
}

C'est ça que je ne comprends toujours pas :-)
!is_null($sRoot) ça veut dire : si la racine n'est PAS === null, donc si elle est précisée, puisque par défaut dans l'appel de ta fonction elle est à null ?
Donc ce cas-ci, c'est si on passe les 2 arguments.
Dans le premier cas aussi. A mon avis, t'as un "!" en trop, non?

Commentaire de malalam le 06/11/2007 13:23:54 administrateur CS

Lol,

SI je passe $sName, ET que je ne passe pas $sRoot à ta méthode, je me retrouve dans le dernier throw, c'est ça qui me gène :
# # Root not specified, and instance exists
# elseif(!is_null($sRoot) && isset(self::$aInstances[$sName])) {
# return self::$aInstances[$sName];
# }
Tu dis "Root not specified" mais ton code dit le contraire : "!is_null($sRoot)" : "!is_null", pas "is_null", donc quand $sRoot n'est PAS null, donc, quand on l'a spécifié dans l'appel à ta méthode.
On ne rentre jamais dans cette condition, parce qu'elle est identique à la 1ère.