Qual è il modo corretto per bloccare zone di codice

Question

Cosa c'è di meglio:
avere un'ampia area di codice in istruzione lock
o
ad avere piccole serrature in un'ampia area ..
scambi in questo campione non sono changable? .

lock (padLock) 
{ 
    foreach (string ex in exchanges) 
    { 
    sub.Add(x.ID, new Subscription(ch, queue.QueueName, true)); 
......... 
} 

o

foreach (string ex in exchanges) 
{ 
    lock (padLock) 
    { 
    sub.Add(x.ID, new Subscription(ch, queue.QueueName, true)); 
    } 
..... 

Answer 1

Il blocco più ampio - meno si ottiene dal multithreading e viceversa Quindi, l'uso dei blocchi dipende completamente dalla logica. Bloccare solo cose e luoghi che cambiano e devono correre da un solo filo in un tempo

Se si blocca per l'utilizzo della raccolta sub - utilizzare il blocco più piccolo, ma se si eseguono più simultanee foreach loop in parallelo

Answer 2

buona pratica sarebbe quella di bloccare solo quella zona che si desidera essere eseguita da un solo thread in un dato momento

Se quella zona è tutta ciclo foreach poi primo approccio va bene

ma se quell'area è solo una linea come hai mostrato io secondo approccio poi vai per il secondo approccio

Answer 3

In In questo caso particolare, l'opzione migliore è la prima, perché altrimenti si sta solo perdendo il blocco/lo sblocco del tempo poiché è comunque necessario eseguire l'intero ciclo. Quindi non ci sono molte opportunità per il parallelismo in un ciclo che esegue comunque operazioni atomiche individuali.

Per ulteriori consigli generali sulle dimensioni sezione critica controllare questo articolo: http://software.intel.com/en-us/articles/managing-lock-contention-large-and-small-critical-sections/

Answer 4

Non si può giudicare in modo efficace, che è "giusto" con il dato frammenti di codice. Il primo esempio dice che non è OK per le persone vedere sub con solo una parte del contenuto degli scambi. Il secondo esempio dice che è OK per le persone vedere sub con solo una parte del contenuto degli scambi.

Answer 5

Penso che ci siano due domande diverse:
1. Quale sarebbe corretto?
2. Quale potrebbe offrire prestazioni migliori?

La domanda sulla correttezza è complicata. Dipende dalle strutture dei dati e da come intendi il lucchetto per proteggerli. Se l'oggetto "sub" non è thread-safe, hai sicuramente bisogno del blocco grande.

La domanda sul rendimento è più semplice e meno importante (ma per qualche ragione, penso che tu ti concentri maggiormente su di esso).
Molte piccole serrature possono essere più lente, perché fanno solo più lavoro. Ma se riesci a eseguire una grande porzione del codice di loop senza bloccare, ottieni una certa concorrenza, quindi sarebbe meglio.