Lazy Singleton in einer Multithread-C# -Anwendung

Question

Ich arbeite an einer Multithread-C# -Anwendung, die einen WCF-Webdienst verwendet. Die Verbindung zum Webservice wird ein bestimmtes Timeout haben, das wir definieren können und nach dem es geschlossen wird. Ich suche, um die Verbindung zum Webservice mit Singleton-Klasse zu speichern. Ich versuche, die Instanz zu erhalten, wie folgt:

CLazySingleton ins = CLazySingleton.Instance; 
string connection = CLazySingleton.abc; 

Unterhalb der Code für die Singleton-Klasse ist:

using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Text; 

namespace LazySingleton 
{ 
    public class CLazySingleton 
    { 
     private static readonly Lazy<CLazySingleton> _instance 
      = new Lazy<CLazySingleton>(() => new CLazySingleton()); 
     private static readonly object ThreadLock = new object(); 
     public static string abc; 
     //I will use the service connection object in place of 'abc' in the application 
     //assume that 'abc' is storing the connection object  

     private CLazySingleton() 
     { } 

     public static CLazySingleton Instance 
     { 
      get 
      { 
       if (abc == null) 
       { 
        lock (ThreadLock) 
        { 
         //Make the connection 
         abc = "Connection stored in this variable"; 
         Console.WriteLine("Connection Made successfully"); 
         return _instance.Value; 
        }      
       } 
       else 
       { 
        return _instance.Value; 
       } 
      } 
     } 
    } 
} 

Meine Fragen sind: 1. Würde dieser Code in der Lage sein, Pflege mehrerer nehmen Threads versuchen, die Instanz zur gleichen Zeit zu erhalten? Dies ist derzeit meine größte Sorge. 2. Kann ich eine bessere Lösung dafür haben? 3. Muss ich hier 'lock' verwenden oder verwenden Lazy Ansatz kümmert sich um Multithreads versuchen, die Instanz zu bekommen?

Jede Hilfe wäre willkommen.

Danke!

Answer 1

Einfache Verwendung ThreadSafetyMode

Lazy<MergeSort> ty = new Lazy<MergeSort>(LazyThreadSafetyMode.ExecutionAndPublication); 

Answer 2

Laut Microsoft Lazy Initialization Dokumentation unter dem Abschnitt mit dem Titel "Thread-Safe-Initialisierung":

standardmäßig Lazy Objekte sind Thread-sicher.

Vor diesem Hintergrund muss Ihr abc Feld nicht statisch sein. Da Sie ein Lazy<T> verwenden, um Ihr Singleton zu instanziieren, können Sie Ihre Verbindung im CLazySingleton-Konstruktor sicher initialisieren.

Answer 3

Wäre dieser Code in der Lage, sich um mehrere Threads zu kümmern, die versuchen, die Instanz zur gleichen Zeit zu erhalten?

In Ihrem Szenario kann das Feld "abc" zweimal initialisiert werden. Stellen Sie sich vor, die Variable "abc" ist null. Der erste Thread befindet sich vor der Zuweisung der Werte im Block "Sperren". Der zweite Thread wartet vor der Sperre. Der erste Thread initialisiert also den "abc" und der zweite Thread initialisiert ihn neu (Ihre Überprüfung auf Null ist außerhalb der Sperre, das ist der Grund). Aber vielleicht solltest du davor keine Angst haben.

Kann ich eine bessere Lösung dafür haben?

Ja, Sie können. Lassen Sie mich das im letzten Block dieser Antwort beschreiben.

Muss ich hier 'lock' verwenden oder die Verwendung von Lazy Ansatz kümmert sich um Multithreads versuchen, die Instanz zu bekommen?

Die Erstellung der Value-Eigenschaft in der Lazy-Klasse ist threadsicher. In Ihrem Szenario würde ich den Vorteil der Eigenschaft IsValueCreated von Lazy <> class verwenden. Sie werden weiterhin das ThreadLock-Objekt benötigen.Nur noch eine Sache ist, dass, sobald Sie Eigenschaft Value Lazy <> Klasse zuzugreifen, IsValueCreated Eigenschaft true zurück (das ist der Trick ;-))

using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Text; 

namespace LazySingleton 
{ 
    public class CLazySingleton 
    { 
     private static readonly Lazy<CLazySingleton> _instance 
      = new Lazy<CLazySingleton>(() => new CLazySingleton()); 
     private static readonly object ThreadLock = new object(); 
     public static string abc; 
     //I will use the service connection object in place of 'abc' in the application 
     //assume that 'abc' is storing the connection object  

     private CLazySingleton() 
     { } 

     public static CLazySingleton Instance 
     { 
      get 
      { 
       if (_instance.IsValueCreated) 
       { 
        return _instance.Value; 
       } 
       lock (ThreadLock) 
       { 
        if (abc == null) 
        { 
         abc = "Connection stored in this variable"; 
         Console.WriteLine("Connection Made successfully"); 
        } 
       } 
       return _instance.Value; 
      } 
     } 
    } 
}