[PHP5.3]PACKAGE CONFIG

Signaler
Messages postés
488
Date d'inscription
samedi 5 avril 2003
Statut
Membre
Dernière intervention
31 mars 2009
-
Messages postés
38
Date d'inscription
mercredi 28 août 2002
Statut
Membre
Dernière intervention
24 janvier 2010
-
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

Messages postés
38
Date d'inscription
mercredi 28 août 2002
Statut
Membre
Dernière intervention
24 janvier 2010

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 ^^
Messages postés
41
Date d'inscription
mardi 3 octobre 2006
Statut
Membre
Dernière intervention
15 mars 2009

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.
Messages postés
276
Date d'inscription
dimanche 22 juillet 2001
Statut
Modérateur
Dernière intervention
5 décembre 2013

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).
Messages postés
41
Date d'inscription
mardi 3 octobre 2006
Statut
Membre
Dernière intervention
15 mars 2009

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.
Afficher les 7 commentaires