Rechanger la méthode de détection des clefs i18n dans le code Java
Hello, Je voudrais revenir en arrière et détecter les clefs i18n via des regex. L'utilisation d'un arbre de compilation dans notre cas c'est trop lourd, et ça n'apporte rien. L'évolution à défaire: https://forge.nuiton.org/issues/1586; J'avais demandé à l'époque de la création du ticket d'avoir un impact sur les performances, cela n'avait pas été fait et nous le subissons désormais à chaque build. Pour les prochaines fois, merci d'y penser. tony. -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 http://www.codelutin.com email: chemit@codelutin.com twitter: https://twitter.com/tchemit
Le 14/05/2014 15:09, Tony Chemit a écrit :
Je voudrais revenir en arrière et détecter les clefs i18n via des regex.
L'utilisation d'un arbre de compilation dans notre cas c'est trop lourd, et ça n'apporte rien. Ca détecte mieux il me semble les bonnes chaines i18n et ça résous le cas compliqué de la détection des clés inutiles (commentaires multilignes et code commenté).
Et c'est pendant la phase de compilation, si ça prend tu temps, ce n'est pas un problème. -1 pour moi globalement -- Éric Chatellier - www.codelutin.com - 02.40.50.29.28
On Wed, 14 May 2014 15:35:09 +0200 Eric Chatellier <chatellier@codelutin.com> wrote:
Le 14/05/2014 15:09, Tony Chemit a écrit :
Je voudrais revenir en arrière et détecter les clefs i18n via des regex.
L'utilisation d'un arbre de compilation dans notre cas c'est trop lourd, et ça n'apporte rien. Ca détecte mieux il me semble les bonnes chaines i18n et ça résous le cas compliqué de la détection des clés inutiles (commentaires multilignes et code commenté).
L'utilisation de l'arbre permet juste de détecter une méthode bien nommé, on n'est pas sûr non plus que c'est une méthode de la classe I18n, donc là encore ce n'est pas assez safe!!! Par contre quel est l'intérêt de mettre des chaines i18n dans des commentaires ? pour moi ne pas mettre de clefs i18n dans des commentaires Pas non plus une bonne pratique de mettre des clefs i18n en multiligne, code illisble selon moi, ... ne pas le faire est un plus.
Et c'est pendant la phase de compilation, si ça prend tu temps, ce n'est pas un problème.
non c'est un process avant la compilation et excuses moi de ne pas partager ton désintéret des temps de build. Je ne vois pas ce qui justifie le fait que pendant un build c'est normal que ça soit long ? Merci d'étayer ton propos car avec un simple "... si ça prend tu temps, ce n'est pas un problème..." ça vole pas très haut comme argumentaire :( En plus je crois qu'il n'y a que toi qui pense ça! que les temps de build c'est pas important... Le fait de revenir en mode recherche texte me semblait un bon compromis, certes le parsing java est mieux mais trop couteux à mon gout, le build reste bloqué dessus de longues secondes, tout ça pour juste détécter du i18n, le jeu n'en vaut pas la chandelle. -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 http://www.codelutin.com email: chemit@codelutin.com twitter: https://twitter.com/tchemit
Le 14/05/2014 15:52, Tony Chemit a écrit :
certes le parsing java est mieux mais trop couteux à mon gout, le build reste bloqué dessus de longues secondes, tout ça pour juste détécter du i18n, le jeu n'en vaut pas la chandelle.
Perso, j'ai du mal à me décider. D'ordinaire je préfère un truc fiable et lent à un truc moins fiable mais rapide mais j'avoue que là oui ça ralenti pas mal le build pour pas des masses de plus-values. On pourrait faire un condorcet pour voir si les gens préfèrent de façon évident l'une ou l'autre solution ? -- Brendan Le Ny, Code Lutin bleny@codelutin.com (+33) 02 40 50 29 28
participants (3)
-
Brendan Le Ny -
Eric Chatellier -
Tony Chemit