On Wed, 13 Apr 2011 16:00:06 +0200 Benjamin POUSSIN <poussin@codelutin.com> wrote:
Salut,
Voila quelques modifications que je vois pour la prochaine release, mais certaines ont des impacts sur d'autres projets, il faut donc que ca ne pose pas de probleme:
1) mettre la classe en final pour proscrire la creation de sous classe (car pour elle c mal) 2) ajouter dans le contrat de OptionDef la methode getAliases():String[] 3) passer en public quelques methodes qui sont actuellement protected (installSaveUserAction, loadDefaultOptions, loadActions) 4) ajouter un joli constructeur qui prend tout en argument et lance aussi le parsing
Je reviens sur ce dernier point, je pense qu'il est mieux que le parse soit explicitement appeler. Dans certain cas, on pourrait vouloir que seules les infos passées dans le code soient prises en compte et pas du tout l'envrionnement or le parse lit les valeurs de tout ce qu'il peut (fichier, env system, env java, ...). Donc pas de parse automatique mais qu'il retourne l'instance AppConfig. Pour mettre a ceux qui aime les multi-instruction par ligne de faire: ApplicationConfig conf = new ApplicationConfig().parse(args);
Les deux ajouts qui pose potentiellement probleme sont les 2 premiers. Le 2eme me parait important. Le 1er n'est pas obligatoire, mais forcerait les gens a ce poser la question de comment utiliser ApplicationConfig.
-- Benjamin POUSSIN -------------------- tél: +33 (0) 2 40 50 29 28 email: poussin@codelutin.com http://www.codelutin.com