Re: [Lima-devel] [Lima-commits] r3425 - in trunk/lima-swing/src/main: java/org/chorem/lima/enums java/org/chorem/lima/ui/accountsreports java/org/chorem/lima/ui/celleditor java/org/chorem/lima/ui/combobox java/org/chorem/lima/ui/common java/org/chorem/lima/ui/financialtransactionsearch resources/i18n
Autant pour le private -> protected, je suis d'accord, par contre pour la disparition du final, je me demande pourquoi ? Au contraire, je trouve que ça a plus de sens... Arnaud Le 16/05/2012 14:41, echatellier@users.chorem.org a écrit :
public enum ComboBoxDatesEnum {
FISCAL_PERIOD(_("lima.enum.comboboxperiod.fiscalperiod")), FINANCIAL_PERIOD(_("lima.enum.comboboxperiod.financialperiod")), PERIOD(_("lima.enum.comboboxperiod.period"));
- private final String description; + protected String description;
ComboBoxDatesEnum(String description) { this.description = description;
On Mon, 21 May 2012 09:19:36 +0200 Arnaud Thimel <thimel@codelutin.com> wrote:
Autant pour le private -> protected, je suis d'accord, par contre pour la disparition du final, je me demande pourquoi ? Au contraire, je trouve que ça a plus de sens...
une enum = une constante alors mettre des choses en protected ça sert à rien car tu pourras jamais surcharger ce champs de nulle -part Donc -1 pour un protected car ça n'a aucun sens dans ce context, je préfère un private final. Je sais que dans une librairie on est assez adepte de mettre des protected partout à mon grand regret (je préfèrerais qu'on utilise mieux l'encapsulation pour ne pas exposer les champs partout). Autant dans une application finale on est pas tenu de faire ça partout,... Et encore moins dans une énum.
Arnaud
Le 16/05/2012 14:41, echatellier@users.chorem.org a écrit :
public enum ComboBoxDatesEnum {
FISCAL_PERIOD(_("lima.enum.comboboxperiod.fiscalperiod")), FINANCIAL_PERIOD(_("lima.enum.comboboxperiod.financialperiod")), PERIOD(_("lima.enum.comboboxperiod.period"));
- private final String description; + protected String description;
ComboBoxDatesEnum(String description) { this.description = description;
_______________________________________________ Lima-devel mailing list Lima-devel@list.chorem.org http://list.chorem.org/cgi-bin/mailman/listinfo/lima-devel
-- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com
Le 21/05/2012 09:19, Arnaud Thimel a écrit :
Autant pour le private -> protected, je suis d'accord, par contre pour la disparition du final, je me demande pourquoi ? Au contraire, je trouve que ça a plus de sens... Dans une enum ca a du sens, oui. De plus il n'y a pas de setter.
Mais le mot clé "final", je ne supprime systématiquement, et je ne l'utilise jamais sauf si le language l'oblige par moment. Comme "private", j'aime pas ce mot clé. -- Éric Chatellier <chatellier@codelutin.com> Tel: 02.40.50.29.28 http://www.codelutin.com
participants (3)
-
Arnaud Thimel -
Eric Chatellier -
Tony Chemit