¿Es seguro utilizar este hilo de Parallel.ForEach()?

Question

Esencialmente, estoy trabajando con esto:

var data = input.AsParallel(); 
List<String> output = new List<String>(); 

Parallel.ForEach<String>(data, line => { 
    String outputLine = ""; 
    // ** Do something with "line" and store result in "outputLine" ** 

    // Additionally, there are some this.Invoke statements for updating UI 

    output.Add(outputLine); 
}); 

de entrada es un objeto List<String>. La instrucción ForEach() realiza algún procesamiento en cada valor, actualiza la interfaz de usuario y agrega el resultado al outputList. ¿Hay algo intrínsecamente incorrecto en esto?

Notas:

• orden de salida es poco importante

Actualización:

base en la retroalimentación que he recibido, he añadido un manual para lock la instrucción output.Add, así como también el código de actualización de la interfaz de usuario.

Answer 1

Sí; List<T> no es seguro para subprocesos, por lo que agregarlo ad-hoc de subprocesos arbitrarios (muy posiblemente al mismo tiempo) está condenado. En su lugar, debe usar una lista de seguridad para subprocesos o agregar el bloqueo manualmente. O tal vez hay un Parallel.ToList.

Además, si es importante: el pedido de inserción no estará garantizado.

Esta versión es segura, sin embargo:

var output = new string[data.Count]; 

Parallel.ForEach<String>(data, (line,state,index) => 
{ 
    String outputLine = index.ToString(); 
    // ** Do something with "line" and store result in "outputLine" ** 

    // Additionally, there are some this.Invoke statements for updating UI 
    output[index] = outputLine; 
}); 

aquí estamos utilizando index para actualizar un índice de matriz diferente por llamada paralelo.

Answer 2

The documentation dice lo siguiente acerca de la seguridad de los subprocesos de List<T>:

estáticos públicos (Shared en Visual Basic) de este tipo son seguros para subprocesos. No se garantiza que ningún miembro de instancia sea seguro para subprocesos.

Una lista (de T) puede admitir varios lectores al mismo tiempo, siempre que la colección no se modifique. Enumerar a través de una colección no es intrínsecamente un procedimiento seguro para subprocesos. En el raro caso en que una enumeración contenga uno o más accesos de escritura, la única forma de garantizar la seguridad de la secuencia es bloquear la colección durante toda la enumeración. Para permitir que la colección sea accedida por múltiples hilos para lectura y escritura, debe implementar su propia sincronización.

Por lo tanto, es output.Add(outputLine)no flujos seguros y hay que garantizar la seguridad de rosca a sí mismo, por ejemplo, envolviendo la operación de adición en un comunicado lock.

Answer 3

¿Hay algo intrínsecamente incorrecto en esto?

Sí, todo. Nada de esto es seguro. Las listas no son seguras para actualizarse en varios subprocesos al mismo tiempo, y no puede actualizar la UI desde ningún subproceso que no sea el subproceso de interfaz de usuario.