CLASSE MOMENT V2.0
le_duche
Messages postés
159
Date d'inscription
lundi 13 juin 2005
Statut
Membre
Dernière intervention
26 février 2009
-
17 févr. 2009 à 16:21
le_duche Messages postés 159 Date d'inscription lundi 13 juin 2005 Statut Membre Dernière intervention 26 février 2009 - 26 févr. 2009 à 16:44
le_duche Messages postés 159 Date d'inscription lundi 13 juin 2005 Statut Membre Dernière intervention 26 février 2009 - 26 févr. 2009 à 16:44
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/49289-classe-moment-v2-0
https://codes-sources.commentcamarche.net/source/49289-classe-moment-v2-0
26 févr. 2009 à 16:44
D'un côté c'est vrai que déclarer un tableau, c'est tout propre. Cela-dit en terme de quantité de lignes de code ce n'est pas beaucoup inférieur d'y injecter les donées que mes if.
Mais avec n langues, il y a une portion (n-1)/n des déclarations qui sont faites qui sont inutiles, avec une vraie internationalisation, ça peut devenir très lourd.
Voila donc ce qui justifie ce code très lourd et très laid ^_^
25 févr. 2009 à 15:18
Je n'avais pas non plus capté le MomentPattern: bien foutu ! Juste une petite chose cependant: pourquoi ne pas stocker les noms en différentes langues dans un tableau multidimensionnel ? Ce serait plus simple si tu ajoutes d'autres langues, non ?
25 févr. 2009 à 14:58
Si certains points ne sont pas clairs, c'est toujours bien de le mentionner, je ne compte les comenter, mais bien améliorer ma façon de coder pour de telles séquences.
Pour ce qui est de la justification d'un algorithme, je pense que ça ne doit pas se trouver dans le code, mais bien éventuellement en annexe.
18 févr. 2009 à 11:04
J'ai survolé le code ainsi que la doc. La doc est claire et compréhensible. Le code est propre, semble efficace (je n'ai pas testé). Manquent juste quelques commentaires.
Sinon, vraiment bien !
Bonne prog !
17 févr. 2009 à 16:21
Vous trouverez toute la documentation sur cette source dans le pdf qui accompagne.