cptpingu
Messages postés3837Date d'inscriptiondimanche 12 décembre 2004StatutModérateurDernière intervention28 mars 2023123 23 déc. 2010 à 21:28
Je parlais de faire des .hxx, regarde mes sources, j'utilise ce genre de procédé (donc .cc, .hh et .hxx ou .cpp, .hpp, .hxx).
Dans ton cas, vu que la plupart des classes sont très concises, ce n'est pas nécessaire (je l'aurais peut être fait juste pour "SimpsonIntegral", mais c'est franchement pour pinailler :p).
(10 pour la propreté, c'est cette qualité là qui doit être montré à des débutants).
ordiman85
Messages postés41Date d'inscriptionsamedi 4 mars 2006StatutMembreDernière intervention19 mars 2010 23 déc. 2010 à 20:58
Merci pour ces remarques. Et merci pour le conseil du "using namespace", désormais je ne serai plus un abuseur de cette atrocité ! Etant très attaché à la propreté du code, si tu as d'autres conseils, je suis preneur !
"Essaie de séparer le code de sa définition"
Je suppose que tu parles de séparer les déclarations et l'implémentation, malheureusement je n'ai pas trouvé de solution, puisque mes structures sont des templates.
PS : je viens de modifier la source, avec pas mal de modifs dont le mot 'bibliothèque' ;)
cptpingu
Messages postés3837Date d'inscriptiondimanche 12 décembre 2004StatutModérateurDernière intervention28 mars 2023123 23 déc. 2010 à 15:07
C'est propre, commenté en Doxygen, et le code est élégant. C'est du bon travail.
Quelques petites remarques:
- Library se traduit par bibliothèque et non librairie (bookstore en anglais). Ça pique un peu aux yeux, c'est dommage.
- Évite les "using namespace", voir: http://0217021.free.fr/portfolio/axel.berardino/articles/bon-usage-using-namespace - Essaie de séparer le code de sa définition, avec des .hxx. Néanmoins, comme ton code est concis, je comprends que tu n'es pas fait la séparation. J'aurais sûrement fait pareil.
C'est rare d'avoir du code intéressant, et celui-ci en fait partie.
23 déc. 2010 à 21:28
Dans ton cas, vu que la plupart des classes sont très concises, ce n'est pas nécessaire (je l'aurais peut être fait juste pour "SimpsonIntegral", mais c'est franchement pour pinailler :p).
(10 pour la propreté, c'est cette qualité là qui doit être montré à des débutants).
23 déc. 2010 à 20:58
"Essaie de séparer le code de sa définition"
Je suppose que tu parles de séparer les déclarations et l'implémentation, malheureusement je n'ai pas trouvé de solution, puisque mes structures sont des templates.
PS : je viens de modifier la source, avec pas mal de modifs dont le mot 'bibliothèque' ;)
23 déc. 2010 à 15:07
Quelques petites remarques:
- Library se traduit par bibliothèque et non librairie (bookstore en anglais). Ça pique un peu aux yeux, c'est dommage.
- Évite les "using namespace", voir: http://0217021.free.fr/portfolio/axel.berardino/articles/bon-usage-using-namespace
- Essaie de séparer le code de sa définition, avec des .hxx. Néanmoins, comme ton code est concis, je comprends que tu n'es pas fait la séparation. J'aurais sûrement fait pareil.
C'est rare d'avoir du code intéressant, et celui-ci en fait partie.