Optimisation de l'appel d'une méthode de chargement d'un fichier

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:

---

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.

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.

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?

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:

---

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:

---

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:

---

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:

---

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

---

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

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.

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.

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..

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

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.

Après à l'aide d'un singleton tu peux toujours faire appel à ta méthode de chargement au moment ou tu vas faire ton getInstance(), mais bon ça ne fait que masquer tous les appels à ta méthode de lecture, ça ne résous pas le problème de fond.

Cdt.

Citation:

---

Envoyé par transact

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.

Bonjour à tous,

Je reviens encore avec mon problème d'optimisation de l'appel d'une méthode.
En effet invoquer n fois une méthode dans une classe me dérange qui plus est cette méthode s'occuper de charger en mémoire un fichier.

Y-aurait-il une technique permettant d'invoquer une et une seule fois la méthode dans la classe?

Merci d'avance.
Transact.

Ben oui, on te l'a donné plus haut cette méthode: tu stocke son résultat dans une variable et tu utilise par la suite cette variable. Mais comme on te l'a dit, ce n'est pas valable si tu dois t'adapter aux changements du fichier car, forcément, tu ne le chargera qu'une fois et pas N fois, vu que c'est ton but.