ThreadPool.QueueUserWorkItem z wyrażenia lambda i metody anonimowej

Question

Przechodząc dwa parametry do nowego wątku na puli wątków może być czasami skomplikowane, ale wydaje się, że z wyrażeń lambda i anonimowe metody, mogę to zrobić:

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 
} 

teraz już na pewno uprościć ten przykład, ale te punkty są kluczowe:

• Przedmiotem smyczkowe były przekazywane są niezmienne i dlatego threadsafe
• S1 i S2 są zmienne zadeklarowane w w zakresie bloku try, który zamykam natychmiast po kolejkowaniu pracy do puli wątków, więc zmienne s1 i s2 nigdy nie są modyfikowane po tym.

Czy coś jest nie tak z tym?

Alternatywą jest utworzenie nowej klasy, która implementuje niezmienny typ z trzema elementami: test, s1 i s2. To wydaje się być dodatkową pracą bez żadnych korzyści w tym momencie.

Answer 1

Nie ma w tym nic złego. Kompilator zasadniczo robi automatycznie to, co opisałeś jako swoją alternatywę. Tworzy klasę przechwytującą przechwycone zmienne (test, s1 i s2) i przekazuje instancję delegata do lambda, która jest przekształcana w metodę na anonimowej klasie. Innymi słowy, jeśli wybierzesz alternatywę, skończysz z soemingiem bardzo podobnym do tego, co kompilator właśnie dla ciebie wygenerował.

Answer 2

To miły sposób na zrobienie tego. Nie widzę żadnych wad korzystania z lambdas. Jest prosty i czysty.

Answer 3

W tym konkretnym przypadku nie ma tu nic złego. Stan przekazany do drugiego wątku jest całkowicie ograniczony i żaden z typów nie ma żadnych problemów z powinowactwem wątków.

Answer 4

To, na co patrzysz, jest określane jako zamknięcie. Jako chuckj states, kompilator generuje klasę w czasie kompilacji, która odpowiada elementom, do których uzyskuje się dostęp poza zamknięciem.

Jedyna rzecz, o którą musisz się martwić, to to, czy masz parametry ref lub out. Chociaż łańcuchy są niezmienne, odniesienia do nich (lub dowolnej zmiennej) NIE są.

Answer 5

Jeden potencjalny problem ze wzoru jest to, że bardzo kuszące, aby go rozwinąć w coś bardziej rodzajowe ale mniej bezpieczny tak (scratch Code nie można oczekiwać, że do pracy):

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 
    } 
}