Discussion :

[hocinema]

Bonjour,

J'ai le Singleton suivant:

Code :

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

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

[sir_gcc]

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 :

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

[djo.mos]

Bonjour,

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.

[Grimly_old]

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.

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