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. 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. 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; 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. A vos avis :) -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com