BunoCS
Messages postés15472Date d'inscriptionlundi 11 juillet 2005StatutModérateurDernière intervention25 mars 2024
-
19 août 2009 à 12:04
BunoCS
Messages postés15472Date d'inscriptionlundi 11 juillet 2005StatutModérateurDernière intervention25 mars 2024
-
19 août 2009 à 12:04
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
BunoCS
Messages postés15472Date d'inscriptionlundi 11 juillet 2005StatutModérateurDernière intervention25 mars 2024103 19 août 2009 à 12:04
Hello,
J'ai parcouru vite fait le code et j'ai quelques remarques:
- je trouve que ça manque de commentaires explicatifs. Par exemple, pour la fonction Count, je ne comprends pas à quoi elle sert (case sensitive en plus..)
- pourquoi avoir doubler des fonctions? Size() = Length(), non? D'une manière générale, il faut éviter d'en faire trop, pour limiter les apparitions de bugs
- il y a des endroits où tu peux optimiser. Par exemple, dans la fonction LastIndexOf(), tu peux remplacer ta boucle for par une boucle while()...
19 août 2009 à 12:04
J'ai parcouru vite fait le code et j'ai quelques remarques:
- je trouve que ça manque de commentaires explicatifs. Par exemple, pour la fonction Count, je ne comprends pas à quoi elle sert (case sensitive en plus..)
- pourquoi avoir doubler des fonctions? Size() = Length(), non? D'une manière générale, il faut éviter d'en faire trop, pour limiter les apparitions de bugs
- il y a des endroits où tu peux optimiser. Par exemple, dans la fonction LastIndexOf(), tu peux remplacer ta boucle for par une boucle while()...
Next step: gestion de l'unicode?
Bon travail,
Buno.