Discussion :

[BakaOnigiri]

Bon j'arrive après la bagarre, mais je me souviens d'une version qui avait été proposée (par Adiguba je crois bien) lors d'une discussion animée autour du Singleton thread-safe avec lazy loading.

Ca donnait un truc comme ça et de ce que j'avais compris, cela résolvait parfaitement le problème:

Code java :

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
public class MaClasse {
 
    private static class InstanceHolder {
 
        private InstanceHolder() {
        }
 
        static MaClasse instance = new MaClasse();
    }
 
    public static MaClasse getInstance() {
        return InstanceHolder.instance;
    }
 
   private MaClasse() { }
 ....

Si ce n'est pas le cas .. j'espère qu'une bonne âme aura l'amabilité de me détromper .. que j'arrête d'utiliser ça partout

Bulbo

C'est d'ailleurs (presque) comme çà qu'es codé le template Netbeans pour 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
public class MaClasse
{
 
	private MaClasse()
	{
	}
 
	public static MaClasse getInstance()
	{
		return MaClasseHolder.INSTANCE;
	}
 
	private static class MaClasseHolder
	{
 
		private static final MaClasse INSTANCE = new MaClasse();
	}
}

[alladdin]

Parce que ce code n'arrivera pas jusqu'au while tant que instance est null

On ne peux pas avoir un état mémoire ou instance est partiellement initialise (donc non null) et isCompleted a true ?

Sinon tout le reste est très clair, merci .

[tchize_]

si, mais dans ce cas là si instance est toujours null, vous allez rentrer dans le if (instance==null) et donc dans le synchronized et donc de toutes facçons vous vous en battez de l'état "isComplete"