Discussion :

[flyeric]

Bonjour à toutes et à tous,

Je me pose la question qui est dans le titre.

Lors de l'utilisation d'une constante (dans une class Utilities par exemple) qui restera définitivement immuable:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
private static final int NB_HEURES_JOURS = 24;
...
public static int getNbHeuresJours()
{
    return NB_HEURES_JOURS;
}

Dans une autre classe, je vais donc récupérer ma constante comme ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

private int heuresParJour = Utilities.getNbHeuresJours();

Vu que sur terre une journée est faite de 24 heures, 24 ne changera jamais mais jamais.
Dans ce cas précis et puisque 24 ne changera jamais, serait-ce une pratique déconseillée ou mauvaise de simplifier en supprimant le getter et en passant ma constante en public dans ma classe Utilities comme ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
public static final int NB_HEURES_JOURS = 24;
...

Et en la récupérant ailleurs comme ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

private int heuresParJour = Utilities.NB_HEURES_JOURS;

Merci beaucoup pour vos avis de bonnes pratiques et bonnes journée à toutes et à tous,
Eric.

[joel.drigo]

Salut,

Vu que sur terre une journée est faite de 24 heures, 24 ne changera jamais mais jamais.

Peu importe, c'est surtout le fait que la variable soit static et final. Autant faire ça et ne pas alourdir avec une méthode.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

public static final int NB_HEURES_JOURS = 24;

Cependant, il y aurait une raison qui pourrait pousser à faire une méthode et à faire la static final private. Lors de la compilation, la valeur d'une public static final est copiée là où elle est utilisée. Si tu modifies la valeur et que tu ne recompiles pas les classes qui l'utilisent, elles continueront à utiliser l'ancienne valeur, ce qui causera des dysfonctionnements. Tu peux avoir envie ou besoin de l'éviter.

A noter que ce n'est pas forcément vrai qu'un nombre d'heures pas jour soit 24. A cause de l'heure d'hiver/heure d'été (le DST, Daylight saving time), et donc une méthode pourrait avoir son intérêt, par exemple, avec 3 constantes dont le choix est fait par la date et la locale.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

public static int getNombreHeureJour(Date date, Locale locale)

D'ailleurs, si le but est de faire des calculs de temps entre deux dates, il vaut mieux passer par les API Java, qui gèrent tout ça (package java.time).

[flyeric]

Bonjour Joel et merci beaucoup pour ton explication.

Non, ce n'est pas pour jouer avec les dates. En fait, c'était juste un exemple mais comme j'ai plusieurs constantes immuables, je me disais justement que de les déclarer public static final m'éviterait d'avoir des getter superflux sauf dans le cas que tu as précisé.
Alors merci beauoup pour ta réponse.

Bonne journée,
Eric.