Je plaide coupable pour certains des chefs d'accusation énoncés ici. Vous avec gagné 2 coups de fouets. encore ? :( tony Chemit a écrit :
Salut les lutins,
désolé de faire mon rabat-joie une fois de plus mais je suis un peu étonné du code que je trouve dans les générateurs de ToPIA :
1) y'a du code dupliqué à gogo, on fait n fois le même test, par example pour vérifier une propriété sur un attribut, à quoi ça sert ?
2) du code qui devrait être identique (donc déjà dupliqué...) mais pas écrit pareil pour avoir le même résultat ? à quoi ça sert ?
3) du code en commentaire qui a mon avis devrait être supprimé (sinon ça sert à quoi le versionning svn ?)
4) des if-else à l'infini et cela est d'un compliqué à savoir où on en est (surtout avec le formattage à la Eclipse...)
5) les tag values ne sont même pas en constantes et donc on est obligé de parcourir le code des générateurs pour voir quel tagValue existe :je trouve ça trop violent, trop crade et à l'encontre d'une quelconque maintenance du code. A ce propose j'ai commencé dès que je vois un truc du genre
attr.getTagValue("lazy") à définir la constante dans GeneratorUtil et à l'utiliser :
attr.getTagValue(GeneratorUtil.TAG_LAZY)
6) du code sans javadoc (et J'y suis pour quelque chose (lol... et mea culpa))
Je trouve déjà assez dur de lire les classe de template alors je pense qu'on devrait faire quelque chose pour harmoniser tout cela, mais peut-être pas tout de suite (passage a JPA).
Je dois (ou on doit bientôt passer sur JPA et donc revoir tous les générateurs), j'espère qu'on prendra soin de rendre nos générateurs plus compréhensibles et moins (comment dirais-je sans blesser personne LM-atelier-esque...)
De manière générale je trouve qu'on ne prette vraiment pas attention à la propreté de notre code et que coder à l'arrache est une hérésie alors que y'a plein de très bonnes choses et idées dans notre code, alors un peu de rigueur dans le codage serait cool...
Tout ce dont je vous ai énoncé dans les points précédents sont pour moi le beaba d'un codeur et je ne vous parle pas des best-pratices de sharengo ;) mais du simple bon sens...
Ne vous sentez en aucun cas aggresser par mes propos, c'est pas le but, juste ça serait bien que ça avance dans le bon sens.
Tony.
ps: et c'était pas une gueulante, juste un constat :) _______________________________________________ Dev mailing list Dev@list.codelutin.com http://list.codelutin.com/cgi-bin/mailman/listinfo/dev
-- Société Code Lutin http://www.codelutin.com tel : 02 40 50 29 28 fax : 09 59 92 29 28