[Expert]Singleton =? instancié une seule fois

Bonjour,

Pré-requis:
J'ai un fichier de configuration régulièrement consulté auquel j'accéde à partir de multiple thread. Afin d'éviter la lecture à répétition de ce fichier, je cré un singleton. Mais est il réellement lu et charger en mémoire une seule fois?

Une instance d'un objet implémentant le pattern singleton est unique. Ca, je n'en doute pas. Si j'y accède à partir de plusieurs thread, la classe ne sera monté qu'une seule fois en mémoire.

Mais une fois cette instance est en mémoire, que fait le garbage collector si je ne m'en sert pas pendant plusieurs heures? Je crains que cette instance ne soit supprimée et que le fichier sera rechargé lors de la prochaine lecture.

Si quelqu'un maitrise bien le garbage collector, je suis preneur.


Arnaud

hello,

L'implementation d'un Singleton passe generalement par une variable static qui reference ton instance unique.
Cette reference est une reference 'dure', montee en memoire a l'endroit de la heap ou les definition des classe sont chargees.
si je ne me trompe pas, comme ces definitions de classes reste dans la heap, il y aura toujours une reference 'dure' et donc le GC ne liberera pas la memoire.

Salut,


Tant que ton singleton est référencé dans son attribut static, le GC n'y touchera pas ! Donc à moins que tu ais bidouiller le pattern singleton cela ne devrait pas arriver.


Le gros problème que tu peux avoir, si tu as un code comme cela :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

final class Singleton {   private static Singleton INSTANCE;   private Singleton() { // ... }   public static Singleton getInstance() { if (INSTANCE==null) { INSTANCE = new Singleton(); } return INSTANCE; } }

---

:arrow: Si plusieurs threads appelle "en même temps" la méthode getInstance(), tu pourrais avoir plusieurs création de singleton entre le moment où il est créé et celui où il est affecté à INSTANCE...


Il y a des solutions complexes utilisant la synchronisation sans que cela ne soit trop couteux (et de mauvaise solution utilisant mal la synchronisation :aie:), mais dans 99% des cas on peut s'en passer en s'assurant que la méthode sera appelé par un seul et unique thread.

Soit en appelant getInstance() au tout début de l'application (avant même de créer les autres threads).
Soit en initialisant en ligne l'instance lors du chargement de la classe (c'est le ClassLoader qui nous assurera l'unicité de l'opération) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

final class Singleton {   private static final Singleton INSTANCE = new Singleton();   private Singleton() { // ... }   public static Singleton getInstance() { return INSTANCE; } }

---

a++

j'ajoute une précision sur l'unicité et le GC. La référence statique est référencée en dure par la classe, donc le singleton ne sera pas GC *tant* que la classe ne sera pas garbage collectable. La pluspart du temps çà veux dire jamais de GC dessus. *mais* si le classloader qui a instancié la classe est libéré (ce qui arrive dans les webapplication) alors le singleton sera nettoyé. On parle plus là d'un singleton par classloader.

En pratique, c'est un singleton quoi :D

Je vous remerci de vos réponses.

Cdlt,

Arnaud