04-09-2011, 09:58 AM
Je te conseille de passer à l'anglais tout de suite, c'est plus approprié pour le développement et encore plus pour Ruby on Rails, qui utilise beaucoup l'inférence de noms. 
Généralement, on n'écrit pas explicitement le return : Ruby retourne automatiquement la dernière expression appelée.
Je n'aime pas trop cette méthode "afficher" pour tes personnes : je te conseille plutôt d'utiliser to_string.
Ensuite, je ne trouve pas la définition d'un itérateur pertinente : faire directement @address_book.contacts.each est infiniment plus naturel.
Enfin, je te recommande d'utiliser l'interpolation de chaîne plutôt que la concaténation (la coloration syntaxique est meilleure dans les éditeurs).
Après, je ne sais pas trop si ce genre d'exercice te sera vraiment utile dans la mesure où tu écriras souvent des modèles/contrôleurs qui te fourniront un socle différent (par exemple, tes constructeurs ne prendront jamais une liste nom, prénom, etc. mais un hash d'attributs).
Généralement, on n'écrit pas explicitement le return : Ruby retourne automatiquement la dernière expression appelée.
Je n'aime pas trop cette méthode "afficher" pour tes personnes : je te conseille plutôt d'utiliser to_string.
Ensuite, je ne trouve pas la définition d'un itérateur pertinente : faire directement @address_book.contacts.each est infiniment plus naturel.
Enfin, je te recommande d'utiliser l'interpolation de chaîne plutôt que la concaténation (la coloration syntaxique est meilleure dans les éditeurs).
@numero + " " + @rue + "\n" + @code_postal + " " + @ville + ", " + @pays
"#{@numero} #{@rue}\n#{@code_postal} #{@ville}, #{@pays}"
Après, je ne sais pas trop si ce genre d'exercice te sera vraiment utile dans la mesure où tu écriras souvent des modèles/contrôleurs qui te fourniront un socle différent (par exemple, tes constructeurs ne prendront jamais une liste nom, prénom, etc. mais un hash d'attributs).