Discussion :

[bouchette63]

Bonjour,

Je viens vous demander quelques conseils de code

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 :

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

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

[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 ) te répondras assez rapidement

[adiGuba]

Salut,

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 :

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

[bouchette63]

Merci pour ta réponse fouillée adiGuba
Je vais regarder le scope "application" de plus près et le mettre en oeuvre.

[dude]

il est aussi possible de stocker tes properties dans le web.xml (tag <context-param>) et de les recuperer par

Code :

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

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 :

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

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

est preferable a

Code :

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

1
2
File fichierConfig = new File(application.getRealPath("WEB-INF/conf/app-droits.properties"));
FileInputStream input = new FileInputStream(fichierConfig);

[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

[adiGuba]

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