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é ?
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é.
Le terme despotisme te parait déplacé ? Il me semble que j'ai assez entendu de phrase du genre : "Celui qui fait a toujours le dernier môt : ce qui est du pur et simple despotisme..." ou comment imposer ces idées sans la moindre considération du point de vue des autres... donc désolé si la phrase te dérange, elle est adressé à toi aussi qu'aux autres développeurs et elle ne me parait pas du tout déplacée. -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com