Le Sat, 21 Aug 2010 18:26:01 +0200, Tony Chemit <chemit@codelutin.com> a écrit :
Le Wed, 18 Aug 2010 20:57:55 +0200, fdesbois <fdesbois@codelutin.com> a écrit :
Bonjour à tous,
Quelques petits retours sur les transformers créés dans la branche :
Sur ToPIA plutot que de décomposer les 3 parties d'une entité en trois transformeurs (Interface, Abstract, Impl), il nous a paru plus judicieux de fusionner les 3 dans un seul transformeur pour plus fortement couplé la génération, les méthodes sont les mêmes, des imports communs, ... Ici, rajouter une méthode impliquera un copier coller dans les 3 transformeurs...
C'est sur que cela rajoute du contenu dans un seul transformeur mais cela me semble plus maintenable. +10.
De plus ça évite d'avoir trois transformeurs qui ne se ressemblent pas (je sais pas comment on en était arrivé là dans ToPIA) mais ces trois transformeurs utilisait pas toujours la même logique et ça au niveau de la maintenance c'est bien le bordel.
en plus écrire 3 (n fois?) la même choses j'ai jamais aimé et encore moins parcourir n fois le même modèle...
Il faudrait qu'on passe aussi le DAO sur ce modèle dans ToPIA.
Vi même si pour le moment il n'y a pas la même structure que les entités (Interface, Abstract, Impl) car il n'y a pas d'interface, juste le Abstract et le Impl. Ce qui est utilisé dans le DAOHelper est une classe vide qui hérite du Impl sans généricité. Ce sera donc plus simple pour la fusion :)
Autre remarque, au sujet des imports, une bonne partie est automatisé au moment des appels au JavaBuilder en utilisant les fqn (full qualified name), seul les contenus des body ou valeur d'attribut statique doivent obligatoirement être importé à côté. Bon ce n'est juste une remarque, d'avoir tout les imports à un seul endroit reste assez visuel.
Pour le coup du MétaTransformer, l'idée sur la liste EUGene me semble intéressante, d'ailleurs je vais aussi y répondre.
Voilà, c'était des petites remarques en passant
-- Florian Desbois <fdesbois@codelutin.com> SSLL Code Lutin http://www.codelutin.com tel : 02 40 50 29 28 Réseau Libre-Entreprise