Singleton monothread indépendant

Version imprimable

10/09/2009, 16h29
celes

Singleton monothread indépendant

Bonjour à tous,

Je travaille actuellement sur une classe Design Pattern (ou Singleton).

Il se peut que plusieurs threads aient besoin d'utiliser ce Singleton mais de façon indépendante.

En fait, je ne voudrais pas qu'ils se partagent le singleton mais qu'il y ai un singleton unique par thread.

J'ai essayé en essayant de récupérer l'id du thread en cours (numéro unique) et donc de recréer le singleton par thread mais alors, je ne sais plus quand le supprimer et je ne sais pas trop si c'est sûr.

Avez vous une idée ?

Merci.
10/09/2009, 17h10
jacky01

Salut,

Pourquoi ne pas créer de destructeur paramétrer avec l'id du thread afin de détruire seulement l'objet concerné.(biensure ton objet devra avoir une proprioté du genre

Code:

int id_thread

)
Après pour l'appel du destructeur à toi de trouvé le moment idéal, soit une fois que tu est sur de ne plus avoir besoin de ton objet ou alors en fin de vie de ton thread.

Ce n'est qu'une proposition, je ne l'ai jamais mis en place.
10/09/2009, 17h17
npuzin
Je n'ai pas le code en tete, mais tu peux peut etre utiliser la memoire du Thread avec ce genre de code:
Code:

1 2 3 LocalDataStoreSlot myTLSValue; myTLSValue=Thread.GetNamedDataSlot("SleepTime"); int tlsValue=(int)Thread.GetData(myTLSValue);
11/09/2009, 10h39
celes

Merci pour vos réponses.
Je n'ai pas bien compris ton bout de code npuzin même si j'ai bien l'impression que c'est ce que je recherche. C'est à dire, créer le singleton seulement dans la mémoire du thread pour qu'il ne soit pas partagé avec les autres threads. Mais peux-tu être un peu plus explicite ?
Merci ;)

En fait, ce que j'ai surtout peur c'est que de créer un dictionary avec <int id_thread, object instanceSingleton>, va devenir trop lourd en temps de calcul surtout si on prends l'habitude d'utiliser notre singleton par la classe.

Si on utilise notre singleton de cette manière :
Singleton.instanceSingleton
alors il y aura dans mon Getter, une recherche dans le dictionaire en fonction de l'id du thread....

Alors que si on utilise :
Object monSingleton = Singleton.instanceSingleton
et qu'on se sert de monSingleton dans la méthode ensuite, cela diminuera considérablement le temps de calcul car l'appel au dictionaire se fera qu'une seule fois lorsqu'on en as besoin.

Sauf que je ne peux pas être sur de la façon dont sera utilisé ce singleton !

Mon problème c'est que j'aimerai savoir s'il serait possible de directement limiter ce singleton au thread sans avoir à créer un objet lourd (avec id_thread, une liste ou un dictionnaire, etc.).
11/09/2009, 11h26
Guulh

Me semble que s'il y en a plusieurs, c'est plus un singleton :aie:
Et l'accès à un dico est en o(1). Même si tu avais un millier de thread, la différence par rapport à une référence directe est trèèèès négligeable par rapport au reste de ton code.

A titre indicatif, pourquoi veux-tu avoir une instance différente de cette classe par thread ? Parce que là, c'est plus un singleton, chaque instance ayant bien sûr ses propres membres.

Voici comment tu peux declarer ton singleton.

Au lieu d'instancier ton singleton et l'enregistrer dans une variable static comme on fait d'habitude, tu places ton instance dans la memoire du thread.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
    public class MySingleton
    {        
        private MySingleton()
        {}
        public static MySingleton Instance
        {
            get
            {
                LocalDataStoreSlot mySingletonInstanceSlot = 
                    Thread.GetNamedDataSlot("MySingletonInstance");
                MySingleton instance = (MySingleton) Thread.GetData(mySingletonInstanceSlot);
                if (instance == null)
                {
                    // Put the new singleton instance onto the thread local storage
                    instance = new MySingleton();
                    Thread.SetData(mySingletonInstanceSlot, instance);
                    return instance;
                }
                else // returning existing instance from the thread local storage
                {
                    return instance;
                }
            }
        }
    }

Après petit test vite fait, il y a peut-être plus simple :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
  public class ThreadStaticSingleton
  {
    [ThreadStatic]
    private static ThreadStaticSingleton _instance;
 
    private ThreadStaticSingleton()
    {
    }
 
    public static ThreadStaticSingleton Instance
    {
      get
      {
        if (_instance == null)
        {
          _instance = new ThreadStaticSingleton();
        }
 
        return _instance;
      }
    }
 
    public void PrintThreadId()
    {
      Console.WriteLine(Thread.CurrentThread.ManagedThreadId);
    }
  }

Et pour vérifier :

Code:

1
2
3
4
5
6
7
8
9
      var threads = new List<Thread>();
 
      for (int i = 0; i < 10; ++i)
      {
        threads.Add(new Thread(ThreadStaticSingleton.Instance.PrintThreadId));
      }
 
      threads.ForEach(x => x.Start());
      threads.ForEach(x => x.Join());

Ça affiche bien des ids de threads différents et vu que du coup l'instance statique est locale à chaque thread, plus besoin de verrou.

Je ne me souvenais plus de si l'attribut ThreadStatic marchait bien, apparemment si :)

15/09/2009, 09h43
npuzin

Bien vu :ccool:
21/09/2009, 08h42
celes

Merci beaucoup pour votre aide, ça devrait fonctionner comme sur des roulettes !

Et juste pour répondre à Guulh, j'ai commencé à penser à faire un singleton après avoir vu dans un code existant qu'une variable était passée à toutes les méthodes du projet. Je vous dit pas le bordel.
J'ai donc supprimé toutes ces variables et créé un singleton.

Sauf que ce même code est utilisé de différentes manières. Par un WinForm par exemple mais aussi à travers une interface web. Cette interface ouvre un thread à chaque fois que le code est appelé. Et là, paf... Chaque thread utilise le même singleton à tord ;)