[ApplicationConfig] depreciation de loadDefaultOptions(Class)
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 ? ---------------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()); } /** * Load default given options. * * @param options options to load * @param <O> type of enum extend {@link OptionDef} * @since 2.4.8 */ public <O extends OptionDef> void loadDefaultOptions(O[] options) { // load default option (included configuration file name : important) for (OptionDef o : options) { if (o.getDefaultValue() != null) { setDefaultOption(o.getKey(), o.getDefaultValue()); } } } ---------------8<---------------8<---------------8<---------------8<--------------- -- Benjamin POUSSIN -------------------- tél: +33 (0) 2 40 50 29 28 email: poussin@codelutin.com http://www.codelutin.com
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
participants (2)
-
Benjamin POUSSIN -
Tony Chemit