On Fri, 09 Dec 2011 15:30:15 +0100, desbois <desbois@codelutin.com> wrote:
On Fri, 9 Dec 2011 14:56:25 +0100, Tony Chemit <chemit@codelutin.com> wrote:
On Fri, 09 Dec 2011 14:33:53 +0100 desbois <desbois@codelutin.com> wrote:
Salut,
J'ai continué les TU sur nuiton-csv, et notamment tester le ModelBuilder. Mais j'ai un doute de son utilité.
Il faut surtout ce concentrer sur tester des choses interessante je crois...
cette api sert bien à importerexporter en csv, donc je vois pas trop l'intérêt de tester la création d'un modèle csv :( Par contre tester que la mécanique d'Importexport fonctionne c'est plus important.
Ben sur l'import, t'as CsvReader derrière donc a priori c'est déjà bon, ici on doit bien tester l'api qui a été créé, dont le ModelBuilder. De plus cela permet d'avoir des cas pratiques d'implémentation pour voir si l'api est judicieuse.
En pratique on pourrait avoir :
CsvModel<Row> model = CsvModel.newModel(';', Row.class); model.addMandatoryColumn("TITLE", "title"); model.addMandatoryColumn("DATE", "date", new Common.DateValue("yyyy-MM-dd")); model.addMandatoryColumn("NUMBER", "number", Common.INTEGER);
Je suis contre cette approche car un modèle ne devrait JAMAIS être responsable de sa construction.
Pour moi (on en discuté hier avec Brendan) y'a un gros pb dans cette api, les choses sont mal nommée et des concepts de builder/model et factories sont bien mélangées.
On doit revoir ça.
Je suis d'accord
je vais maintenant regarder l'Export voir si ça se goupille bien. Je pense qu'on peut avoir une api bien sympa en tout cas, mais ya encore pas mal de chose à voir pour simplifier/fignoler.
Pour le moment faut qu'on arrive à trouver une meilleure manière de faire à plusieurs Donc je ne pense pas que ça soit une bonne qu'une personne seule fasse les modifs que ça soit moi ou toi ;)
Vi en effet, je ne commite pas mes modifs à cause de ca. <troll du vendredi>Et tiens git aurait été cool pour ca !</troll du vendredi>
On pourrait s'organiser une petite session de travail autour de ça.
Je propose (Julien me souffle ça dans l'oreille) qu'on peut mettre ça à l'ordre d'une prochaine réunion dev ;)
Ok ca me va. Je peux être dispo demain si ca te dit :D Qui serait intéressé ?
Oups, dans ma tête j'avais compris coding party. My bad.
Mais de grâce pas de despotisme à vouloir tout chambouler : je me sers de l'api sur echobase, brendant devrait rapidemment s'en servir aussi sur wao donc on change pas tout :)
Bien evidemment je sais que vous l'utilisez ;) j'ai jamais fais par de despotisme, je te prierais d'éviter ce genre de propos déplacé.
_______________________________________________ Nuiton-utils-devel mailing list Nuiton-utils-devel@list.nuiton.org http://list.nuiton.org/cgi-bin/mailman/listinfo/nuiton-utils-devel