Dodanie do listy w pętli Parallel.ForEach w THREADSAFE sposób

Question

mam kawałek kodu, który działa w ten sposób na liście obiektów OBJ zwane ListofObjects:

List<SomeObject> NewListofObjects<SomeObject>(); 

Parallel.ForEach(ListofObjects, obj => 

//Do some operations here on obj to get a newobj 

NewListofObjects.Add(newobj); 

); 

Teraz jestem z równoległym .Dla pętli i chcę wykonać operację na NewListofObjects. Jednak pojawia się ten błąd, gdy próbuję: "Próba odczytu lub zapisu pamięci chronionej często wskazuje, że inna pamięć jest uszkodzona".

Czy to dlatego, że moja metoda NewListofObjects.Add (newobj) nie jest bezpieczna dla wątków? Jeśli tak, w jaki sposób mogę ją chronić w wątkach?

Answer 1

Czy to dlatego, że moja metoda NewListofObjects.Add(newobj) nie jest bezpieczna dla wątków?

Prawidłowo. To nie jest bezpieczne dla wątków.

Żadna instancja nie ma gwarancji, że będzie bezpieczna dla wątków.

To z MSDN w odniesieniu do List<T> (przejdź do sekcji zatytułowanej "Bezpieczeństwo wątków").

Jeśli tak, w jaki sposób mogę go zabezpieczyć jako gwinty?

Użyj zbieżnej kolekcji, takiej jak ConcurrentBag<T>. Zwróć uwagę, że utracisz możliwość śledzenia kolejności wstawiania przedmiotów.

Answer 2

Możesz użyć bloku locking, jak poniższy kod, aby wstawiać elementy do listy w sposób bezpieczny dla wątków.

var sync = new object(); 
var myNewList = new List<SomeObject>(); 
Parallel.ForEach(myListOfSomethings, a => 
    { 
     // Some other code... 
     var someObj = new SomeObject(); 
     // More other code... 
     lock(sync) 
     { 
      myNewList.Add(someObj); 
     } 
     // Even more code... 
    });