[PHP5.3]PACKAGE CONFIG

webdeb Messages postés 488 Date d'inscription samedi 5 avril 2003 Statut Membre Dernière intervention 31 mars 2009 - 8 févr. 2009 à 11:15
Calak Messages postés 38 Date d'inscription mercredi 28 août 2002 Statut Membre Dernière intervention 24 janvier 2010 - 24 janv. 2010 à 05:06
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/49214-php5-3-package-config

Calak Messages postés 38 Date d'inscription mercredi 28 août 2002 Statut Membre Dernière intervention 24 janvier 2010
24 janv. 2010 à 05:06
Dis moi si je me trompe, mais ligne 274, ta méthode "merge":
  // Ligne 274...
  public function merge($oMerge)
 {
 $sClass = get_called_class();
 $oReflexion = new ReflectionClass($sClass);

 if(! $oReflexion->isSubclassOf($this) )
 {
 throw new Exception('Vous devez passer en paramètre une classe enfant de MapIterator.', E_ERROR);
 }
  //...


Ce n'est pas la classe passée en paramètre que tu vérifie, mais la classe courrante. Hors, la classe courante sera toujours du même type que la classe courante ;)

J'aurais plutôt mis:
  //...
  public function merge($oMerge)
 {
 $oReflexion = new ReflectionObject($oMerge);

 if(! $oReflexion->isSubclassOf($this) )
 {
 throw new Exception('Vous devez passer en paramètre une classe enfant de MapIterator.', E_ERROR);
 }
  //...


Je ne sais pas ce que vous en pensez ^^
cs_dorian91 Messages postés 41 Date d'inscription mardi 3 octobre 2006 Statut Membre Dernière intervention 15 mars 2009
26 févr. 2009 à 19:37
Merci pour ton commentaire sa fait plaisir.
Oui c'est vrai j'aurais pu mettre des exceptions spécialisées mais j'ai eu une petite flem ^^
Je mettrais la source à jour quand j'aurais complètement terminé mon package config.
Utilisateur anonyme
25 févr. 2009 à 17:48
Non, je t'assure, ta lib est très bien faite et découpée, y'à rien à en redire sinon bravo :)

J'ai pris le temps de bien lire tous les fichiers ;)

Bonne continuation

PS : c'est du chipottage, mais tu peux créer des classes d'héritage de chaque type d'exception gérant le log ou les messages multi-langues (je tombe bien bas pour trouver un truc à améliorer).
cs_dorian91 Messages postés 41 Date d'inscription mardi 3 octobre 2006 Statut Membre Dernière intervention 15 mars 2009
16 févr. 2009 à 21:58
Lol jadu
Courage je ne connais pa ton niveau mais on a tous commencé je pense en faisant du code un peu dégeu ^^
Perso j'ai beaucoup appris en regardant d'autre code source.
Hum 27 téléchargements et po de commentaires, j'en déduis que ma classe est parfaite ^^
Je comptais sur quelques commentaires pour faire des améliorations ou corriger des bugs.
jadu Messages postés 217 Date d'inscription mercredi 26 juillet 2006 Statut Membre Dernière intervention 16 août 2018
16 févr. 2009 à 08:32
je vais donc regarder attentivement cette source pour bien comprendre comment coder proprement.
J'en ai besoin, paraît-il ;-((
cs_dorian91 Messages postés 41 Date d'inscription mardi 3 octobre 2006 Statut Membre Dernière intervention 15 mars 2009
8 févr. 2009 à 12:42
Merci pour le commentaire ça fait plaisir ^^.
Je savais pas pour les namespaces avec \ dans la doc sur fr.php.net il y a toujours les :: je vais regarder ça.
webdeb Messages postés 488 Date d'inscription samedi 5 avril 2003 Statut Membre Dernière intervention 31 mars 2009 4
8 févr. 2009 à 11:15
Excellente source et proprement codée. Par contre, tes namespaces ne sont plus à jour. Il va falloir que tu penses à changer les :: par \
Rejoignez-nous