[Singleton] En faire une bonne utilisation

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() ?

Citation:

---

Envoyé par Koubi

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

Citation:

---

Envoyé par Koubi

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

Citation:

---

Envoyé par Koubi

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

---

Il rend la main avant

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 ?

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.

Des verrous en base :roll:
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)

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

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