Redondance code chargement fichier

Version imprimable

Bonjour,

Je viens vous demander quelques conseils de code :P

Dans mon projet sous Eclipse, j'ai quatres classes qui procédent au chargement d'un même fichier properties. Ce dernier est chargé et utilisé seulement dans la fonction ActionForward de ces quatre classes. Je procède de cette façon :
Code:

1 2 3 4 5 // On récupère le fichier de confuguration sur les droits Properties options = new Properties(); File fichierConfig = new File(servlet.getServletContext().getRealPath("WEB-INF/conf/app-droits.properties")); options.load(new FileInputStream(fichierConfig));
Est-ce que c'est la meilleur manière de procéder, c'est à dire charger le fichier à chaque passage dans la fonction (comme je fais), ou peut-on l'optimiser ?

Merci pour votre aide

28/07/2006, 15h45
in

je ne suis pas un connaisseur, mais je pense que si le fichier est petit tu gagneras à le cahrger une seule fois pour toute.

Par contre s'il est énorme et que tu ne t'en sers pas souvent, ton existant est peut être mieux.

Ca dépend aussi de l'architecture de tes classes.

bref, je pense qu'un pro (suivez mon regard :roll: ) te répondras assez rapidement

Salut,

Citation:

Envoyé par bouchette63

Est-ce que c'est la meilleur manière de procéder, c'est à dire charger le fichier à chaque passage dans la fonction (comme je fais), ou peut-on l'optimiser ?

Le plus gros problème pour moi c'est que tu ne fermes pas ton FileInputStream...

Concernant la lecture si ton fichier est de petite taille ce n'est pas vraiment dérangeant car ce n'est pas énormément coûteux... mais tu pourrais utiliser le scope "application" pour y stocker ton Properties si tu ne veux pas avoir à le relire à chaque fois... par exemple :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
class Config {
 
    public static final String PROPERTIES_KEY = "com.monsite.www.Properties";
 
    public static Properties getProperties(ServletContext application) throws IOException {
        Properties properties = null;
        // On recherche les properties dans le scope application
        Object object = application.getAttribute(PROPERTIES_KEY);
 
 
        if (object instanceof Properties) {
            // S'il s'agit bien d'un objet Properties,
            // on le récupère directement :
            properties = (Properties) object;
        } else {
            // Sinon on le charge depuis le fichier
            properties = new Properties();
            File fichierConfig = new File(application.getRealPath("WEB-INF/conf/app-droits.properties"));
            FileInputStream input = new FileInputStream(fichierConfig);
            try {
                properties.load(input);
            } finally {
                input.close();
            }
            // Et on le stocke dans le scope application
            // (pour les futurs appels)
            application.setAttribute(PROPERTIES_KEY, properties);
        }
        return properties;
    }
 
}

a++

28/07/2006, 16h03
bouchette63

Merci pour ta réponse fouillée adiGuba :D
Je vais regarder le scope "application" de plus près et le mettre en oeuvre.
28/07/2006, 16h32
dude
il est aussi possible de stocker tes properties dans le web.xml (tag <context-param>) et de les recuperer par

Code:

getServletContext().getInitParameter("name");

en terme de performance, ca devrait être bien meilleur que de lire un fichier, et ca laisse le container gerer les properties.

par contre, si tu veux rester comme tu es, je pense qu'un

Code:

InputStream input = getServletContext().getResourceAsStream("/WEB-INF/conf/app-droits.properties")

est preferable a
Code:

1 2 File fichierConfig = new File(application.getRealPath("WEB-INF/conf/app-droits.properties")); FileInputStream input = new FileInputStream(fichierConfig);
28/07/2006, 16h45
bouchette63

Oula merci à toi dude, mais maintenant je suis complétement perdu.
Je ne sais pas lequel utilisé ? :?

Personne ne sait quel est le plus performant pour un petit fichier properties ???

Merci

Edit : je vais choisir par moi-même, ne vous traquassez pas la tête ;)
28/07/2006, 16h58
adiGuba

Citation:

Envoyé par bouchette63

Oula merci à toi dude, mais maintenant je suis complétement perdu.
Je ne sais pas lequel utilisé ? :?

getInitParameter() ne permet pas de lire un fichier properties, mais de lire des propriétées qui sont stocké dans ton web.xml... Je ne te le recommande que si tu as un très petit nombre de valeur à utiliser...

Concernant getRessourceAsStream(), ce n'est qu'un moyen plus simple et plus propre d'ouvrir un fichier dans le projet Web. Donc tu peux très bien l'utiliser...

a++
28/07/2006, 17h01
bouchette63

Encore merci pour ces nouveaux eclaircissements adiGuba :P

Je vais donc rester sur le scope application.

Edit : Merci pour ces tests dude, c'est vrai qu'avec ces derniers, on peut voir la différence de performance entre les méthodes utilisées. Notamment avec getRealPath et getResourceAsStream. :P

je viens de faire un petit test rapide sur Tomcat 5 avec une property name=valeur

Code:

1
2
3
      for (int i = 0; i < 10000; i++) {
         value = getServletContext().getInitParameter("name");
      }

s'execute en 15ms

Code:

1
2
3
4
5
6
7
      for (int i = 0; i < 10000; i++) {
         InputStream is = getServletContext().getResourceAsStream("/WEB-INF/conf.properties");
         Properties p = new Properties();
         p.load(is);
         is.close();
         value = p.getProperty("name");
      }

s'execute en 1266ms

Code:

1
2
3
4
5
6
7
8
      for (int i = 0; i < 10000; i++) {
         File fichierConfig = new File(getServletContext().getRealPath("WEB-INF/conf.properties"));
         FileInputStream is = new FileInputStream(fichierConfig);
         Properties p = new Properties();
         p.load(is);
         is.close();
         value = p.getProperty("name");
      }

s'execute en 4641ms