Discussion :

[ancrou]

Bonjour,

Peut on passer des paramètres dans un Singleton ?

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
    private BLL(String Path) {
        PATH= Path;
    }
 
    private BLL() {
    }
 
     public static BLL getInstance() {
         if (INSTANCE == null){
             synchronized(BLL.class){             
                     INSTANCE = new BLL();
                 }
         }
         return INSTANCE;
     }
 
     public static BLL getInstance(String Path) {
         if (INSTANCE == null){
             synchronized(BLL.class){             
                     INSTANCE = new BLL(Path);
                 }
         }
         return INSTANCE;
     }

ou préférez vous une methode Set qui va initialiser mes attributs sans faire de passage de parametre dans le constructur ?

[Kanithael]

J'aurais pour ma part tendance à dire ni l'un, ni l'autre...

Dans un cas, dans ton exemple de code, le passage de paramètre ne fonctionnera qu'une seule fois, à l'initialisation. Tu vas me dire, c'est exactement le fonctionnement souhaité... Ce qui est génant, c'est quand même que si quelqu'un fait un autre appel avec un autre paramètre, ça ne marchera pas (comme c'est un singleton, il n'a aucun moyen de savoir si ça a déjà été instancié ou pas)

Et faire un Set à un singleton, c'est aussi dangereux parce que le set impacte tout le monde...

Moi j'aurais tendance à dire pour éviter toute ambiguité, de ne laisser qu'une seule méthode et de supprimer le getInstance sans paramètre :

Code :

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

1
2
3
4
5
6
7
8
9
     public static BLL getInstance(String Path) {
         if (INSTANCE == null){
             synchronized(BLL.class){             
                     INSTANCE = new BLL();
                 }
         }
         INSTANCE.setPath(Path);
         return INSTANCE;
     }

Mais quand même, pas très pratique pour un singleton...

[Djobird]

Ca me semble également une mauvaise idée. De tout évidence ton singleton doit lire un fichier, tu peux donc mettre le chemin dans un fichier de conf par exemple.
Au passage ton synchronised est mal placé et ne sert ici à rien

[ancrou]

Oui mes chemins aux fichiers sont dans un conf.properties. Mais c'est la class Main qui ouvre et lit les conf
bien ou pas bien

Je développe en couche : BLL DAL
On m'a demandé de faire des Singletons
Le Main instancie la BLL et la BLL instancie la DAL...

mon synchronised est plassé comme dans un tutorel de Singletons

[Djobird]

Pour le problème de synchronised, si deux thread appelle getInstance() alors qu'il est encore null, il est possible qu'il rentre tous les deux dans le bloc "if (instance == null)".
Pour empêcher ca soit tu rajoutes un synchronised autour du bloc if, (ce qui est du double check locking ou un nom approchant) sachant que cette solution comporte un risque (très faible mais existant quand même) de ne pas marcher (à cause de la gestion du cache par Java, je te renvoie vers un article parlant du problème de singleton sur le site ).
La solution la plus sécurisée (mais la plus couteuse) est de synchroniser toute la fonction.

Sinon pour les fichiers de conf, j'ai pour habitude d'utiliser un singleton pour le lire justement Je ne sais pas si c'est la facon la plus propre, mais elle a l'intérêt de me permettre d'accéder au propriété du fichier plus facilement.

[ancrou]

Pour le problème de synchronised, si deux thread appelle getInstance() alors qu'il est encore null, il est possible qu'il rentre tous les deux dans le bloc "if (instance == null)".
Pour empêcher ca soit tu rajoutes un synchronised autour du bloc if, (ce qui est du double check locking ou un nom approchant) sachant que cette solution comporte un risque (très faible mais existant quand même) de ne pas marcher (à cause de la gestion du cache par Java, je te renvoie vers un article parlant du problème de singleton sur le site ).
La solution la plus sécurisée (mais la plus couteuse) est de synchroniser toute la fonction.
.

Javais mal lu le tuto
Je vais opter pour une solution coûteuse.

Sinon pour les fichiers de conf, j'ai pour habitude d'utiliser un singleton pour le lire justement Je ne sais pas si c'est la facon la plus propre, mais elle a l'intérêt de me permettre d'accéder au propriété du fichier plus facilement.

Ah oui, allez un singleton de plus


Merci