On Fri, 04 Jan 2013 15:08:27 +0100 Yannick Martel <martel@codelutin.com> wrote:
Salut !
Ca me parait pas une mauvaise idée. Splitter la classe serait une bonne chose (avec ou sans changement de module).
Passer dans un module propre, ça peut être aussi intéressant, mais est-ce vraiment utile ? A part une meilleure visibilité (ce qui n'est pas rien déjà ;), qu'y a-t-il à y gagner ?
Je reste mitigé :\ Il ne faudrait pas non plus ouvrir une brèche vers le "tout-en-minimodule", le simple fait de revoir le packaging suffisant dejà à y voir plus clair, non ?
pour moi non, va voir nuiton-profiling : y'a une classe dedans, donc pour la brèche c'est déjà fait... Surtout ce que j'y vois c'est avoir un artifact qui a un but, parce que pour moi ApplicationConfig (nuiton-config) et bien délimité et ne devrait pas être mélangé avec nuiton-utils qui est un foure tout de classes plus au moin pas du tout utiles :( ApplicationConfig est bien, je trouve dommage que ça soit au milieu d'un bordel sans nom :( Et c'est plus juste qu'une classe utilitaire vu qu'il a y'a d'autre truc autour. Avoir son propre module permettrait d'améliorer sa visibilité et pouvoir plus le mettre en valeur, perso je veux utiliser ApplicationConfig mais pas nuiton-utils... et idem je voudrais bien sortir d'autres modules (nuiton-beans, nuiton-converter, ...)
Le 2013-01-04 10:09, Tony Chemit a écrit :
Hello,
Je pense qu'il serait bien de créer un nouveau module pour y mettre tout ce qui touche à ApplicationConfig (et surtout pouvoir éclater la classe ApplicationConfig car là une classe de plus de 2700 lignes, je dis bof :().
Pour le package je voyais org.nuiton.util.config, histoire de ne pas avoir de collision avec la classe dépréciée qu'on laisserait dans l'ancien nuiton-utils
Qu'en pensez-vous ?
tony. _______________________________________________ Nuiton-utils-devel mailing list Nuiton-utils-devel@list.nuiton.org http://list.nuiton.org/cgi-bin/mailman/listinfo/nuiton-utils-devel
-- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com