Discussion :

[Goupo]

Bonjour,

Tout d'abord voilà ma classe :

Code :

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

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?

[in]

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

[Goupo]

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

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.

[Goupo]

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

Code :

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

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

[Clorish]

Code :

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

 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.

[CheryBen]

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 :

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

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;        
    }

[Goupo]

Les traces indiquent que parametres est null... !

Code :

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

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 :

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

 parametreOracle.connexion.adresse = jdbc:oracle:thin:@(DESCRIPTION=(ADDRESS_LIST=(ADDRESS=(COMMUNITY=***)(PROTOCOL=***)(Host=***)(Port=***))(ADDRESS=(COMMUNITY=***)(PROTOCOL=***)(Host=***)(Port=***)))(CONNECT_DATA=(SID=***)))

[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 ), 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 ...

[CheryBen]

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 :

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

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.

[Goupo]

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

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
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);
}

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.

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.

[CheryBen]

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.

[Goupo]

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.

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 :

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

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
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;
    }
}

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

[natha]

Il est où dans l'arborescence de ton projet ce fichier "jcerbere.properties" ?

[in]

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

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

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

[Glob]

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

Eclipse powah

[Goupo]

Ah ouai d'accord...

À 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!

[natha]

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.