Gabriel Coletta tem razão. List
não é thread-safe.
Sempre que tem código em paralelo tentando escrever a umuma estrutura só, tem que verificar que ao código não conflita.
EmUm exemplo que mostra o cuidado que precisa ter é (isso será em C++, mais é simples):
void AdicionarNumero(int valorNovo)
{
valorCompartilhado += valorNovo;
return;
}
Se tem dois "threads" executando esta função em paralelo, pode ter problemas. O código varlorComparthilhadovalorComparthilhado += valor;
talvez se torna os instruções:
- Guardar valor de
valorNovo
num registro (memória no CPU). - Guardar valor de
valorCompartilhado
num registro. - Sumar os dois registros e guardar o resultado num registro.
- Guardar a suma em
valorCompartilhado
.
Se os dois "threads" chegam a passo 3 aao mesmo tempo, os dois tem oso mesmo valor de valorCompartilhado
guardado. Então os dois fazem suma, um vai guardar o resultado, e depois o outro vai guardar o resultado. Isto quer dizer que um dos resultados será jogado fora.
Se não limitar este código a um "thread" a cada vez, não pode controlar o resultado. Se os dois "threads" chamam AdicionarNumero
ao mesmo tempo com valorCompartilhado == 5
e argumentos de 3
e 1
, pode ficar com resultados de 6
, 8
, ou 9
guardados em valorCompartilhado
.
O jeito de limitar código para que só um "thread" pode entrar a cada vez é com um lock
(como Gabriel Coletta comentou). Tambem pode usar umuma estrutura e um algoritmo que não falha em paralelo, mesmo sem lock
(como ConcurrentBag
).