ThreadPool.QueueUserWorkItem mit einem Lambda-Ausdruck und anonymer Methode

Question

Passing zwei Parameter zu einem neuen Thread auf dem Threadpool kann manchmal kompliziert sein, aber es scheint, dass mit Lambda-Ausdrücke und anonymen Methoden, kann ich dies tun:

public class TestClass 
{ 
    public void DoWork(string s1, string s2) 
    { 
     Console.WriteLine(s1); 
     Console.WriteLine(s2); 
    } 
} 

try 
{ 
    TestClass test = new TestClass(); 
    string s1 = "Hello"; 
    string s2 = "World"; 
    ThreadPool.QueueUserWorkItem(
     o => test.DoWork(s1, s2) 
     ); 
} 
catch (Exception ex) 
{ 
    //exception logic 
} 

Nun, ich habe dieses Beispiel sicherlich vereinfacht, aber diese Punkte sind der Schlüssel:

• Die String-Objekte THREAD
• Die s1 und s2 Variablen sind unveränderlich und daher übergeben wird erklärt w ith im Rahmen des try-Blocks, den ich sofort nach dem Einreihen der Arbeit in den Thread-Pool verlasse, so dass die Variablen s1 und s2 danach nie geändert werden.

Ist da etwas nicht in Ordnung?

Die Alternative besteht darin, eine neue Klasse zu erstellen, die einen unveränderlichen Typ mit drei Mitgliedern implementiert: test, s1 und s2. Das scheint nur eine zusätzliche Arbeit ohne Nutzen zu sein.

Answer 1

Es ist nichts falsch daran. Der Compiler erledigt im Wesentlichen automatisch das, was Sie als Ihre Alternative beschrieben haben. Es erstellt eine Klasse für die erfassten Variablen (test, s1 und s2) und übergibt eine Delegat-Instanz an das Lambda, das in eine Methode für die anonyme Klasse umgewandelt wird. Mit anderen Worten, wenn Sie mit Ihrer Alternative fortfahren würden, würden Sie mit etwas sehr Ähnlichem enden, was der Compiler gerade für Sie generiert hat.

Answer 2

Es ist eine nette Art, es zu tun. Ich sehe keine Nachteile bei der Verwendung von Lambdas. Es ist einfach und sauber.

Answer 3

Für dieses spezielle Beispiel, nein ist hier nichts falsch. Der Status, den Sie an den anderen Thread übergeben haben, ist vollständig enthalten und keiner der Typen enthält Probleme mit der Thread-Affinität.

Answer 4

Was Sie betrachten, wird als eine Schließung bezeichnet. Als chuckj states generiert der Compiler eine Klasse zum Zeitpunkt der Kompilierung, die den Elementen entspricht, auf die außerhalb des Abschlusses zugegriffen wird.

Die einzige Sache, um die Sie sich sorgen müssen, ist, wenn Sie ref oder out Parameter haben. Während Strings unveränderlich sind, sind die Referenzen auf sie (oder eine Variable) NICHT.

Answer 5

Ein mögliches Problem mit dem Muster ist, dass es sehr es in etwas mehr generisch, aber weniger sicher so zu erweitern verlockend (Kratzern Code- es funktioniert nicht erwarten):

public static void QueueTwoParameterWorkItem<T1, T2>(T1 value1, T2 value2, workDelegate<T1,T2> work) 
{ 
    try 
    { 
     T1 param1 = value1; 
     T2 param2 = value2; 
     ThreadPool.QueueUserWorkItem(
      (o) => 
      { 
       work(param1, param2); 
      }); 
    } 
    catch (Exception ex) 
    { 
     //exception logic 
    } 
}