[ Singleton ] Threads

Version imprimable

03/05/2008, 00h17
hocinema

[ Singleton ] Threads
Bonjour,

J'ai le Singleton suivant:
Code:

1 2 3 4 // Singleton avec champ final public class Singleton1 { private static final Singleton1 INSTANCE= new Singleton1(); Private Singleton1 () { ---- }
Comment éviter la création de plusieurs objets dans le cas de Multithreading. Si deux Thread accède au singleton en meme temps, comment éviter qu'il soit instancié deux fois?

Merci

Ce que tu écris me semble convenir, même en environnement multi-threadé.
Pour éviter le problème d'accès simultané à la méthode getInstance() d'un singleton (ce qui pourrait poser problème si l'instanciation se faisait dans cette méthode), on peut écrire un truc dans le genre :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
public class MonSingletonAMoi {
 
     // l'instance unique est créée au chargement de la classe dans la JVM  donc pas de     
     // soucis même si plusieurs thread appellent la méthode getInstance() en même temps
     private static final MonSingletonAMoi instance = new MonSingletonAMoi();
 
     // le constructeur privé pour empêcher les instanciations "manuelles"
     private MonSingletonAMoi() {}
 
     // retourne l'instance unique de la classe MonSingletonAMoi et on est sur qu'avant l'appel
     // à cette méthode, l'instance existe déjà. Donc on a bien une seule instance dans une JVM
     public static MonSingletonAMoi getInstance() {
         return instance;
     }
 }

Ça me parait être une bonne méthode.

03/05/2008, 02h18
djo.mos

Bonjour,

Citation:

Envoyé par hocinema

Comment éviter la création de plusieurs objets dans le cas de Multithreading. Si deux Thread accède au singleton en meme temps, comment éviter qu'il soit instancié deux fois?

Aucun risque: le champ INSTANCE est statique, il est donc initilisé au chargement de la classe, ce qui ne se fait qu'une seule fois.

Une autre technique serait l'approche différée, où on ne crée le singleton que la première fois qu'il est demandé, donc dans la méthode getInstance(). Seulement, faut faire attention à marquer cette méthode comme synchronized.
05/05/2008, 13h16
Grimly_old

Citation:

faut faire attention à marquer cette méthode comme synchronized

Légèrement dangereux en multithreading si on s'y prends mal (ç peux ralentir pas mal le programme voire le rendre assimilable à un monothread), pour ceux qui ont du mal avec le mot synchronized, essayez d'appeler une fois la méthode getInstance() avant de lancer vos threads.
05/05/2008, 13h42
Tommy31

La création d'un singleton en multi-threadé est assez touchy en java. La solution du synchronized est sympa mais bien couteuse si on fait beaucoup d'accès. L'autre solution c'est d'utiliser un ThreadLocal().

Tout est décrit ici: http://christophej.developpez.com/tu...n/multithread/

Il y est expliqué aussi pourquoi le très connu double check locking est erroné.