Calak
Messages postés38Date d'inscriptionmercredi 28 août 2002StatutMembreDernière intervention24 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és41Date d'inscriptionmardi 3 octobre 2006StatutMembreDernière intervention15 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.
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és41Date d'inscriptionmardi 3 octobre 2006StatutMembreDernière intervention15 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és217Date d'inscriptionmercredi 26 juillet 2006StatutMembreDernière intervention16 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és41Date d'inscriptionmardi 3 octobre 2006StatutMembreDernière intervention15 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és488Date d'inscriptionsamedi 5 avril 2003StatutMembreDernière intervention31 mars 20094 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 \
24 janv. 2010 à 05:06
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:
Je ne sais pas ce que vous en pensez ^^
26 févr. 2009 à 19:37
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.
25 févr. 2009 à 17:48
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).
16 févr. 2009 à 21:58
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.
16 févr. 2009 à 08:32
J'en ai besoin, paraît-il ;-((
8 févr. 2009 à 12:42
Je savais pas pour les namespaces avec \ dans la doc sur fr.php.net il y a toujours les :: je vais regarder ça.
8 févr. 2009 à 11:15