Discussion :

[Frederik]

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 :

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

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 :

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
 
 
	/**
         * 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?

[ep31]

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

Code :

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

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 :

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

1
2
 
private static volatile DemandeServiceImpl instance = null

[Pill_S]

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

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

[Frederik]

merci beaucoup pour ces informations

[gailuris]

Code :

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

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/

[hwoarang]

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 :

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

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