02-09-2015, 09:36 AM
Coté code, tu peux alléger fusionner les if ( ... instanceof ...) suivit d'un return : return (... instanceof... && <<le return existant);
Egalement, pourquoi avoir redéfini les __set(), surtout sans rien y faire? Perso, je n'y aurai pas touché, à défaut, j'aurai levé une exception (là, "$myValueObject->mesgenoux = 2" ne lèvera aucune erreur, mais ne fera rien).
Coté business, les rues sont comparées par nom... Mais quid de deux rues du même nom dans deux villes différentes? Elles seront considérées comme 'isEqual()'...
Egalement, pourquoi avoir redéfini les __set(), surtout sans rien y faire? Perso, je n'y aurai pas touché, à défaut, j'aurai levé une exception (là, "$myValueObject->mesgenoux = 2" ne lèvera aucune erreur, mais ne fera rien).
Coté business, les rues sont comparées par nom... Mais quid de deux rues du même nom dans deux villes différentes? Elles seront considérées comme 'isEqual()'...