Meilleur implémentation de singleton sur un serveur tomcat

Bonjour...

Actuellement nous développons une application n-tiers avec tomcat.

Afin d'accéder aux service nous passons par des factories.

Le principe de la factory est de renvoyer un singleton.

Il a été implémenté de la façon suivante :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

/**          * Singleton          */ private static DemandeService instance = new DemandeServiceImpl(); /**          * Singleton          * @return          */ public static DemandeService singleton() { return instance; }

---

je ne suis pas trop habitué à cette implémentation et je préfère l'implémentation classique :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

/**          * Singleton          */ private static DemandeServiceImpl instance = null /**          * Singleton          * @return          */ public static DemandeService singleton() { if(instance == null) { instance = new DemandeServiceImpl(); } return instance; }

---

Dans la mesure ou c'est le serveur qui instancie la factory la première version du code peut-être valide non?

La première version est bonne (ajouter un final optionnellement sur instance)
Concernant la seconde,
il faut ajouter un bloc synchronized :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

public static DemandeService singleton() { if(instance == null) { synchronized (instance) { if(instance == null) instance = new DemandeServiceImpl(); } } return instance; }

---

pour éviter que deux accès concurrents ne fassent appel au constructeur simultanément.
On peut aussi en plus déclarer instance comme étant volatile :

Code:

---

1 2	private static volatile DemandeServiceImpl instance = null

---

Attention aux champs static+final dans un serveur d'application, cela peut conduire à des fuites mémoires dans certains cas...

:arrow: http://wiki.apache.org/tomcat/Memory...ClassVariables

merci beaucoup pour ces informations :)

Citation:

---

Envoyé par ep31

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

public static DemandeService singleton() { if(instance == null) { synchronized (instance) { if(instance == null) instance = new DemandeServiceImpl(); } } return instance; }

---

pour éviter que deux accès concurrents ne fassent appel au constructeur simultanément.

---

Petite précision : en java, il existe des cas (je ne sais plus exactement lequel) où 2 threads peuvent quand même passer sur le new DemandeServiceImpl();

Une bonne lecture sur ce sujet : http://christophej.developpez.com/tu...n/multithread/

Citation:

---

Envoyé par gailuris

Petite précision : en java, il existe des cas (je ne sais plus exactement lequel) où 2 threads peuvent quand même passer sur le new DemandeServiceImpl();

Une bonne lecture sur ce sujet : http://christophej.developpez.com/tu...n/multithread/

---

Pour info, l'explication (qui est egalement donnée dans le lien) :
L'implementation de la JVM d'un appel à instance = new Singleton(); peut etre :

Code:

---

1 2 3

mem = allocate() ; instance = mem ; ctorSingleton(instance);

---

Autrement dit, si on suppose que les 2 premieres intructions s'executent puis qu'un thread plus prioritaire prend la main, alors le premier if (qui n'est pas synchronisé) va etre faux et notre thread prioritaire va garder la main et utiliser une instance non initialisée de Singleton...

Voir le lien...