On Thu, 23 Aug 2012 16:29:17 +0200 Benjamin POUSSIN <poussin@codelutin.com> wrote:
Salut,
Dans ApplicationConfig "loadDefaultOptions(Class<O> optionClass)" est depreciee mais je ne vois pas trop en quoi elle derange ?
On force les utilisateur a faire
loadDefaultOptions(MesOption.class.getEnumConstants())
ou lieu de
loadDefaultOptions(MesOption.class)
Pour moi je ne vois pas trop de raison de la deprecier, elle appel juste l'autre, elle ne sera pas dur a maintenir, elle existait avant (et etait utilisée), donc autant garder la compatibilité dans ce cas ?
je la laisserais bien. (donc j'enleverais deprecated).
Quelqu'un est contre ? oui je suis contre :( voir mon commentaire dans le code.
si tu veux le laisser alors il faudrait alors écrire et renommer la méthode donc ... public <O extends OptionDef & Enum<O>> void loadDefaultOptions(Class<O>
---------------8<---------------8<---------------8<---------------8<--------------- /** * Load default options of enum pass in param (enum must extend {@link OptionDef}) * * @param optionClass to load * @param <O> type of enum extend {@link OptionDef} * @deprecated since 2.4.8, prefer use now {@link #loadDefaultOptions(OptionDef[])} */ @Deprecated public <O extends OptionDef> void loadDefaultOptions(Class<O> optionClass) {
loadDefaultOptions(optionClass.getEnumConstants());
olaola, je suis une énumération, et si un jour je suis pas une énumération que va-t-il se passer ???
---------------8<---------------8<---------------8<---------------8<---------------
De plus j'aime pas trop ce xxx.getEnumConstants(), si ta classe est une enum un xxx.values() me va plus (les gouts les couleurs et tout le reste...) -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com