Discussion :

[Koubi]

Bonjour à tous,
Je suis un petit nouveau sur ce forum et je me permets de vous soumettre mes réflexions sur le singleton.

Imaginons un cas d'utilisation :
J'ai un "manager" qui me permet d'accéder à ma BDD pour effectuer des actions.
Afin de protéger ce fonctionnement en mode multi-Thread, je fais de ce manager un singleton.
J'ai donc un synchronized sur la méthode statique GetInstance() de ma classe.
Je fais appel à cette méthode depuis une autre classe de la manière suivante :
MonManager manager = MonManager.getInstance();

Ensuite, dans ce manager j'ai une même méthode ajouteSiPasAssez() qui :
1)lire une valeur en BDD
2)test si cette valeur est inférieure à X
3)si c'est le cas, elle ajoute 50

J'ai donc besoin que cette méthode ne soit accéder que par un seul Thread à la fois.
Dois-je mettre un synchronized dessus ou alors le fait d'utiliser un singleton suffit ?

Je fais
MonManager manager = MonManager.getInstance();
A quelle moment cet appel rend la main sur le moniteur ?
Dès que je sors de la méthode getInstance ?
Et si je fais ça :
MonManager.getInstance().ajouteSiPasAssez();
Il rend la main avant ou après avoir fini la méthode ajouteSiPasAssez() ?

[Debernad]

Dois-je mettre un synchronized dessus ou alors le fait d'utiliser un singleton suffit ?

Il faut en effet ajouter un synchronized sur la méthode ajouteSiPasAssez() de ton Manager

A quelle moment cet appel rend la main sur le moniteur ?
Dès que je sors de la méthode getInstance ?

Oui , dés que tu sors de la méthode getInstance(), un autre thread peut executer la méthode getInstance() ...

Il rend la main avant ou après avoir fini la méthode ajouteSiPasAssez() ?

Il rend la main avant

[Koubi]

Ok merci beaucoup pour ces explications.
Donc si je comprends bien, le design pattern singleton me garantie uniquement d'avoir une instance unique de classe : pas du tout que ce singleton ne soit utilisé par un seul Thread en même temps. C'est bien ça ?

Donc je vais mettre des synchronized sur les méthodes qui doivent être protégées.
Par contre, si j'ai 2 JVM (2 Tomcat par exemple) qui exécutent mon application, il est possible que 2 Threads (1 de chaque JVM) accède à ma BDD en même temps, même si j'ai protégé par un synchronized.
Y a t'il une solution ?

[dlemoing]

La solution a un nom : l'isolation des transactions.
Rendre toutes tes méthodes synchronized n'est pas forcement une bonne idée, réfléchis aux impacts en terme de performance.

[viena]

Des verrous en base
Si ton probleme concerne l'acces à la base, il faut commencer par le gérer au niveau de la base je pense... (ou de l'acces avec les transactions comme dit précédemment)

[Koubi]

Ok. Merci pour ces explications qui me remettent bien les idées en place.

Question subsidiaire :
2 Threads récupère une référence vers le même singleton. Les 2 lancent en même temps la même méthode.
Vont-ils partager le même espace mémoire pour l'exécution de cette méthode ?
Autrement dit, si j'ai des variables dont la portée est limitée à cette méthode, y a-t-il un risque d'accès concurents sur ces variables entre les Threads ?
Je pense que non mais j'aimerais bien avoir une confirmation et une explication plus théorique que mon intuition...

[herve91]

Tu penses bien Chaque thread, heureusement, dispose de sa propre pile d'exécution.