Discussion :

[tnarol]

Bonjour,

Je voulais savoir s'il y aurait un problème particulier à synchroniser sur un objet retourné par une fonction.

Un peu de contexte pour ce besoin. J'utilise le "Initialization-on-demand holder idiom"

L'instance que je veux initialiser à la demande m'est donc renvoyée par quelque chose du style :

Code :

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

1
2
3
        public static Something getInstance() {
                return LazyHolder.INSTANCE;
        }

Et plus tard pour quelque chose d'autre que l'initialisation, j'ai besoin de synchroniser l'instance.

D'où ma volonté de faire :

Code :

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

1
2
3
4
 
synchronized(getInstance()) {
...
}

[thelvin]

Disons que c'est pas très robuste en cela que le jour où ce ne sera plus un singleton, cette syntaxe synchronisera sur des objets complètements différents et donc, ne synchronisera pas.

En regardant le code, on ne "voit" pas que la synchronisation a lieu sur un objet précis, ce qui est moins maintenable. Il vaudrait mieux récupérer une instance, la garder dans un coin, se la partager autant que possible, et synchroniser là-dessus.

Mais techniquement, ça marche aussi bien que sur un objet obtenu autrement qu'avec un appel de fonction.