Purtroppo non funziona così: se usi il lock in quel modo garantisci esclusivamente che due thread non possano eseguire quella foreach contemporaneamente, devi anche gestire il lock in ogni operazione di aggiunta/rimozione di elementil.

Inoltre, il lock limita il parallelismo, quindi rischi che un thread resti in attesa per parecchio tempo (e quando devi salvare gli oggetti, come nell'esempio che mi hai fatto, il tempo è proprio tanto!!)

Consiglio: se puoi, rivedi la tua architettura