Le Fri, 02 Apr 2010 10:29:07 +0200, Florian Desbois <fdesbois@codelutin.com> a écrit :

Le jeudi 01 avril 2010 à 20:22 +0200, Tony Chemit a écrit :

...

Le Wed, 31 Mar 10 13:00:08 +0200, "nuiton.org Release Notification" <noreply@nuiton.org> a écrit :

...

o Template pour générer le squelette des méthodes de service pour une appli finale [ServiceTransformer] Issue: 412. Thanks to Florian DESBOIS. Resolved by fdesbois.

Je pense qu'on devrait un peu revoir cette template :

1) L'utilisation d'un objet context est peut-être trop spécifique, on devrait plutôt laisser des méthodes abstraites :

- openTransaction() - closeTransaction() - treateError(...)

Cela permettrait d'avoir quelque chose de plus neutre et de ne pas forcer à utiliser une objet de context.

A la charge ensuite de l'implantation d'écrire ce qui va bien dedans. Si cela paraît trop compliqué pour certains, ils peuvent toujours écrire un service abstrait qui le fait et faire hériter les services abstraits générés de ce nouveau squelette service...

De manière générale, j'aime pas trop l'idée de forcer à utiliser des objets alors que le besoin n'est pas nécessaire... On introduit des couplages forts qui n'ont pas lieu d'être.

Cela me semble honnête, mes besoins étant un peu trop spécifique pour être générique dans ToPIA.

...

2) Lors de la génération, c'est pas génial dans le service abstrait de ne pas avoir toutes les méthodes abstraites tout en haut, on est un peu obligé (lorsqu'on veut lire la classe) de tout parcourir pour voir toutes les méthodes abstraites, pas génial.

En attente pour le moment... C'est juste du confort. (et c'est plus chiant a écrire dans la template, à moins de mettre un système d'ordonnancement des méthodes générés dans EUGene).

non c'est pas du confort! Ecrire du code généré propre n'est du confort... Et faire 2 boucles dans une templates, c'est pas bien chiant sauf si t'as un poil dans la main :)

...

3) La gestion des exceptions dans les signatures sont pas optimisées : il ne sert à rien de remettre de spécifier des Exceptions moins générales, un exemple :

void maMethode() throws Exception, IOException;

peut s'écrire plus simplement :

void maMethode() throws Exception;

Ok, je vais corriger cela.

...

4) le nom des méthodes du context sont pas très normalisées :

- context.beginTransaction(...) - context.doCatch(...) - context.doFinally(...)

Je préfère un

beginTransaction closeTransaction treateError ou treateException

car doCatch ou doFinally ça me paraît un brin bizarre.

Il est clair que ces noms (provenant de Pollen) sont un peu vague. Je vais corriger ca.

...

A vos avis :)

-- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com