Properties Null PointerException

Version imprimable

Voir 40 message(s) de cette discussion en une page

Bonjour,

Tout d'abord voilà ma classe :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public class properties implements I_properties {
    private Properties  parametres;
 
    public void properties() throws FileNotFoundException, IOException{
        parametres = new Properties();
        FileInputStream in = new FileInputStream("jcerbere.properties");
        parametres.load(in);
        in.close();
    }    
 
    public String getParametres(String code){
        //affectation des valeurs par défaut configuré dans le properties
        return parametres.getProperty("parametreOracle.connexion.adresse");        
    }

Lorsque j'appelle getParameters, j'ai une NullPointerException sur return parametres.getProperty(...); Quelqu'un voit-il pourquoi?

28/04/2008, 16h03
in

Citation:

Envoyé par Goupo

Quelqu'un voit-il pourquoi?

ben parametres est null. Tu n'as pas d'exception quand tu fais ton new properties () ?

soit dit en passant, le nom des classes "doit", par convention, commencer par une Majuscule ...
28/04/2008, 16h09
Goupo

Non, pas d'exception pour new properties();

Citation:

Envoyé par in

soit dit en passant, le nom des classes "doit", par convention, commencer par une Majuscule ...

Je sais bien mais je suis pas auteur de cette classe.

A propos, voici un court extrait de mon fichier jcerbere.properties :
Code:

1 2 3 bd.outil.outype_traitement = MAJ/C bd.profil_utilisateur.pu_date_fin = 01/01/00 bd.profil_utilisateur.pudelegation = N

28/04/2008, 16h33
Clorish

Citation:

Envoyé par Goupo

Code:

return parametres.getProperty("parametreOracle.connexion.adress

L'extrait ne mentionne nulle part cette propriété.
Soit l'extrait sert a rien ;) soit tu ne l'a pas definie, donc il est normal que tu ne la trouve pas et donc renvoie "null" en retour de methode getProperty.

Bonjour, c'est étrange, il n'y a pas de raison que parametre soit null...du moins à première vue...ce serait plutôt la méthode getParametres qui renvoit null et l'exception se situe après. (parametreOracle.connexion.adresse n'est pas trouvé dans le fichier)
Essait d'ajouter des traces pour vraiment vérifier (ou utilise un debugger) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 public String getParametres(String code){ String result = null; if(parametres == null) { System.out.println("parametres null"); } else { result = parametres.getProperty("parametreOracle.connexion.adresse"); if(result == null) System.out.println("result null"); } return result; }

28/04/2008, 16h41
Goupo
Les traces indiquent que parametres est null... ! 8O
Code:

1 2 3 parametres null parametres null parametres null
A propos de mon extrait qui sert à rien voilà la ligne qui concerne la property :

Code:

parametreOracle.connexion.adresse = jdbc:oracle:thin:@(DESCRIPTION=(ADDRESS_LIST=(ADDRESS=(COMMUNITY=***)(PROTOCOL=***)(Host=***)(Port=***))(ADDRESS=(COMMUNITY=***)(PROTOCOL=***)(Host=***)(Port=***)))(CONNECT_DATA=(SID=***)))
28/04/2008, 16h49
in

Fait voir la portion de code où tu crées ton objet properties ...

D'ailleurs, il sert à quoi le paramètre "code" de getParametres ? Elle est bizarre cette méthode ...

sinon, petite question (je ne me souviens plus de la réponse :oops:), t'es sûr que tu peux mettre des espaces avant et après le signe = dans le fichier properties ? [EDIT] oui a priori on peut ...

Si parametres est null c'est parce que tu dois avoir une exception dans ton constructeur.
As-tu catché correctement les exceptions? ou tu continue comme si de rien n'était?
Essai :
Code:

1 2 3 4 5 6 7 public void properties() throws FileNotFoundException, IOException{ parametres = new Properties(); FileInputStream in = new FileInputStream("jcerbere.properties"); parametres.load(in); in.close(); System.out.println("parametres initialisé : " + parametres!=null); }
Si cette trace n'apparaît pas, c'est qu'une exception est levée avant et que tu as mal géré derrière.

Citation:

Envoyé par in

Fait voir la portion de code où tu crées ton objet properties ...
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 try { String paramAdress = (new properties()).getParametres("parametreOracle.connexion.adresse"); String paramUser = (new properties()).getParametres("parametreOracle.connexion.utilisateur"); String paramPass = (new properties()).getParametres("parametreOracle.connexion.passe"); DriverManager.registerDriver(new oracle.jdbc.driver.OracleDriver()); con = DriverManager.getConnection(paramAdress, paramUser, paramPass); ok = true; } catch (SQLException e) { System.out.println("echec d'ouverture:" + e.getMessage()); ok = false; } return (ok); }
Citation:

Envoyé par in

il sert à quoi le paramètre "code" de getParametres?

En fait j'ai remplacé code par "parametreOracle.connexion.adresse" pour voir si ce n'était pas le paramètre code qui posait problème.

Citation:

Envoyé par in

t'es sûr que tu peux mettre des espaces avant et après le signe = dans le fichier properties ?

Je sais pas, je suis pas auteur de cette classe.

28/04/2008, 17h01
CheryBen

8O En faisant N new Properties tu vas lire et charger N fois ton fichier en mémoire. Garde l'instance en variable et réutilise la.
Je ne vois pas de catch IOException et FileNotFOundException.

Citation:

Envoyé par morph41

8O En faisant N new Properties tu vas lire et charger N fois ton fichier en mémoire. Garde l'instance en variable et réutilise la.
Je ne vois pas de catch IOException et FileNotFOundException.

Arg, mais quelle horreur ce code.
J'ai ajouté un bloc try/catch et viré le trhows (je préfère) et j'ai ajouté un champ properties à ma classe DataBase. J'ai aussi mis des majuscules à mes noms de classe.

Citation:

Envoyé par Pogo

Mon code étant désormais normalisé et mon problème toujours d'actualité. Je crois qu'il vaut mieux que je poste un nouveau topic pour que ce soit plus clair pour tout le monde.

Edit : En fait, ce n'était pas une bonne idée : MultiPost.

Si quelqu'un voit d'où pourrait venir mon problème, il est donc obligé de me le dire ici. Voilà les classes "Refactorées".

Méthode appelant la méthode PropertiesCerbere.getParameters().

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
public boolean ouvre() {
        boolean ok = false;
 
        try {
            String paramAdress = properties.getParametres("parametreOracle.connexion.adresse");
            String paramUser = properties.getParametres("parametreOracle.connexion.utilisateur");
            String paramPass = properties.getParametres("parametreOracle.connexion.passe");
 
            DriverManager.registerDriver(new oracle.jdbc.driver.OracleDriver());
            con = DriverManager.getConnection(paramAdress, paramUser, paramPass);
            ok = true;
        }
        catch (SQLException e) {
            System.out.println("echec d'ouverture:" + e.getMessage());
            ok = false;
        }
        return (ok);
    }

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
33
public class PropertiesCerbere implements I_properties {
    private Properties  parametres;
 
    public void properties() {
        try {       
            parametres = new Properties();
            FileInputStream in = new FileInputStream("jcerbere.properties");
            parametres.load(in);
            in.close();
        }
        catch (FileNotFoundException fnfe) {
            System.out.println("Le fichier properties est introuvable!");
        }
        catch (IOException ioe) {
            System.out.println("Erreur IO");
        }
    }    
 
    public String getParametres(String code){
        //affectation des valeurs par défaut configuré dans le properties
        //return parametres.getProperty("parametreOracle.connexion.adresse");
        String result = null;
        if(parametres == null) {
            System.out.println("parametres null");
        }
        else {
            result = parametres.getProperty("parametreOracle.connexion.adresse");
            if(result == null) 
                System.out.println("result null");
        }
        return result;
    }
}

29/04/2008, 08h53
Glob

Hello.

C'est censé être un constructeur, la méthode properties() qui renvoie "void"?

Habituellement mes constructeurs ne renvoient rien, sinon ils sont (sauf erreur de ma part) considérés comme des méthodes.

:roll:
29/04/2008, 09h10
natha

Il est où dans l'arborescence de ton projet ce fichier "jcerbere.properties" ?
29/04/2008, 09h17
in

Citation:

Envoyé par Glob

C'est censé être un constructeur, la méthode properties() qui renvoie "void"?

+1, il faut déclarer un constructeur que tu appelles par new.

Dans ton nouveau code, ton objet n'est pas initialisé quand tu t'en sers.
Dans ton ancien code, tu appelais le constructeur par défaut, paramètres était donc null. Je suppose que tu croyais appeler ta méthode qui retourne void ... ?

Enlève le void pour créer un "vrai" constructeur et fait un new pour initialiser tes objets ...
29/04/2008, 09h20
CheryBen

Je crois effectivement que c'est sensé être un constructeur (si on regarde le 1er post ils ont le même nom).
Par contre vu que tu as changé le nom de classe, il faut changer le nom du constructeur.
29/04/2008, 09h24
natha

Euh oui c'est exact, il semble que la "méthode" properties() ne soit jamais appelée. Il faudrait que ça soit le constructeur.

Voilà une bonne raison d'utiliser les conventions de nommage de Sun pour les classes, à savoir utiliser une Majuscule, ce qui permet de mieux "lire" ce genre de problème.
29/04/2008, 09h25
Glob

À noter qu'un bon IDE tel qu'Eclipse signale un warning lorsqu'une méthode a un nom de constructeur.

Eclipse powah :king:
30/04/2008, 11h27
Goupo

:scarymov:Ah ouai d'accord...

Citation:

À noter qu'un bon IDE tel qu'Eclipse signale un warning lorsqu'une méthode a un nom de constructeur.

A noter également qu'un bon développeur, et pas forcément si bon que ça, n'aurait pas fait ce genre d'erreur,
qu'un autre bon développeur, et pas forcément si bon que ça, aurait su voir que PropertiesCerbere n'avait pas de constructeur...

Ceci-dit merci! Mon problème est résolu, même s'il en cachait un autre mais ça, c'est une autre histoire!

A la prochaine pour de nouvelles aventures!
30/04/2008, 11h35
natha

Citation:

Envoyé par Goupo

qu'un autre bon développeur, et pas forcément si bon que ça, aurait su voir que PropertiesCerbere n'avait pas de constructeur...

Un bon développeur ne peut pas non plus deviner ce que tu as fait dans le reste du code.
Par exemple si tu avais appelé la méthode properties() ailleurs pour initialiser ton objet, il n'y aurait pas eu de problème.
Personne ne se balade dans ta tête pour trouver le problème. Chaque développeur a "ses propres manies". D'où la difficulté parfois de répondre à des problèmes apparemment simple sur un forum.
Je vous prierais donc de ne pas dénigrer l'aide qui vous est apportée.

Voir 40 message(s) de cette discussion en une page