Discussion :

[Rony Rauzduel]

Bonjour,

Dans une classe je dispose de 2 méthodes dans lesquelles, j'invoque à chaque fois une méthode statique permettant le chargement d'un fichier. Ce qui me dérange c'est d'invoquer à chaque fois cette méthode statique qui charge un fichier.
Peux-on invoquer une seule et une seule fois cette méthode dans la classe?

Code java :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
private JSONObject mJsonObj = new JSONObject();
 
public List<ColumnModel> getColumns() throws JSONException {
        try {
            mJsonObj = FileReaderUtil.loadJSONFile(this.getEntityName() + ".json");
 
}
 
 public Sort getDefaultSort() throws IOException {
        mJsonObj = FileReaderUtil.loadJSONFile(this.getEntityName() + ".json");
}

Merci d'avance.
Transact.

[thelvin]

Bien sûr, il suffit que ta variable soit private static final, et initialisée en appelant la méthode statique en question.

Les méthodes n'ont plus qu'à se servir de la variable au lieu d'appeler la méthode statique à chaque fois.

[Rony Rauzduel]

Merci pour ta réponse, par contre l'inconvénient est que ce fichier contient des données au format json qui seront amenées à évoluer.
En final static cela ne posera t-il pas un problème lors de la modification du fichier car les valeurs ajoutées ne seront pas prise en compte?

[thelvin]

Quand tu dis "évoluer" tu veux dire, pendant que l'application est en marche, sans redémarrer ?
Dans ce cas oui, lire le fichier une seule fois va empêcher de prendre en compte des modifications... Et cela que tu le fasses avec ou sans l'utilisation de static. C'est le fait de le faire une seule fois qui est gênant. En gros tu aurais même pas dû chercher à appeler la méthode une seule fois puisque tu n'as pas le droit de le faire.

Si tu veux dire qu'il peut être modifié entre deux démarrages de l'application, je vois pas en quoi l'emploi de static ou pas static serait gênant. Ce n'est pas le compilateur qui va initialiser la variable, et donc, elle va devoir être initialisée à chaque démarrage, en chargeant donc le fichier, prenant ainsi en compte d'éventuelles modifications.

[joel.drigo]

Oui, c'est sûr que si le fichier change et qu'on veut prendre en compte ces changements lors de l'exécution, on est plus dans le cas d'un besoin de chargement unique. On peut éventuellement tester à chaque fois qu'on accède à la réprésentation mémoire du fichier le lastmodified et le comparer à la valeur qu'il avait lors du dernier chargement, et le recharger le cas échéant. On ne peut plus passer par une variable statique, mais un singleton est tout indiqué pour résoudre le cas. Il y a aussi la solution à base de WatchService.

[Nico02]

Salut,

j'ai l'impression que ce que tu cherches à faire est un peu contradictoire. Car effectivement si le contenue de ton fichier est susceptible de changer dans le temps, il est normal de vouloir le recharger à chaque fois pour ne pas passer à coter d’éventuelles modifications. Après oui tu vas surement recharger ton fichier plus de fois que nécessaire si tu utilises souvent ces fonctions..

Mais si tu n'a aucun moyen de savoir précisément quand ton fichier à été modifié, je ne vois pas trop comment synchroniser le rechargement..

[Rony Rauzduel]

En static cela ne pose pas de problème ok mais qu'en est-il en final?

[Rony Rauzduel]

Je suis tout à fait d'accord avec toi Nico02 mais ce qui me dérangeait c'était de charger le fichier à chaque fois dans des méthodes différentes.
Mais bon je vais rester sur la solution initiale de ne pas mettre static
Merci encore pour vos réponses.

[tchize_]

En static cela ne pose pas de problème ok mais qu'en est-il en final?

staic indique que c'est une variable de classe et pas une variable d'instance. Il n'y en a qu'une pour l'application, en gros.

final permet au compilateur de vérifier que la valeur est initialisée exactement une et une seule fois.


C'est distinct de static, tu peux très bien utiliser les deux en même temps, ce sont des concepts différents.

[joel.drigo]

Salut,

Tout dépend ce que fait la méthode au juste. Le cas général est d'appeler la méthode lors de l'initialisation de la variable

Code :

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

1
2
3
4
5
6
7
8
9
public class Exemple {
 
    public final static Image IMAGE;
    private static Image chargerFichier() {
          ...
          return ...;
    }
 
}

La méthode chargeFichier() ne se appelée qu'une seule fois, lors du premier new Exemple() invoqué.

L'autre solution est d'utiliser ce qu'on appelle un Singleton. Voir les tutoriels sur Developpez, comme par exemple : Le singleton ou Design Patterns / Le singleton.

Un moyen simple de faire un singleton sûr et efficace en multi-thread est de faire une Enum (Une Enum est forcément static) à une valeur :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
public enum Image {
 
     IMAGE;
 
     public final Image image;
 
     private Image() {
         image = chargeFichier();
     }
 
     private static Image chargerFichier() {
          ...
          return ...;
    }
 
}

Si tu as plusieurs images, tu peux faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
public enum Images {
 
     IMAGE_XXXX(".../.../XXXX.png"),
     IMAGE_YYYY(".../.../YYYY.png")
     ;
 
     public final Image image;
 
     private Image(String path) {
         image = chargeFichier(path);
     }
 
     private static Image chargerFichier(String path) {
          ...
          return ...;
    }
 
}

Ensuite, dans le code, pour l'utiliser :

Code :

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

label = new JLabel(Images.IMAGE_XXXX.image);

Chaque fichier image ne sera chargé qu'une seule fois pendant une exécution de l'application.