bloqueo en el interior de bloqueo

Question

me pregunto si esta construcción se producirá un error:

lock(sync) 
{ 
    // something 
    lock(sync) 
    { 
    //something 
    lock(sync) 
    { 
     //something 
    } 
    } 
} 

me he encontrado este código, y parece muy bien, pero tal vez en algunas circunstancias, un error puede ser lanzado?

Answer 1

lock is a wrapper for Monitor.Enter and Monitor.Exit:

The lock keyword calls Enter at the start of the block and Exit at the end of the block. From the former's documentation:

From the documentation for Monitor.Enter:

It is legal for the same thread to invoke Enter more than once without it blocking; however, an equal number of Exit calls must be invoked before other threads waiting on the object will unblock.

Debido a que se emparejan las llamadas a Enter y Exit, su patrón de código tiene bien definido el comportamiento.

Tenga en cuenta, sin embargo, que se lock no garantiza que sea una construcción de excepciones menos:

A ThreadInterruptedException is thrown if Interrupt interrupts a thread that is waiting to enter a lock statement.

Answer 2

De acuerdo con MSDN (ver here y here) este es un comportamiento bien definido y no causa ningún problema.

Answer 3

para explicar el comportamiento por lo que está bien definido y nunca fallará:

Aparte: This answer has better details about how locks actually work

El bloqueo se produce a nivel Thread, así que llamar por segunda vez en el mismo subproceso será redundante . Pensaría que no tendría ninguna penalización de rendimiento (aunque eso dependería de cómo se escriban exactamente las partes internas de .Net, por lo que no puedo garantizarlo)

Muchas veces tendría una función pública que llama a otra función pública en su clase, que ambos necesitan el candado cuando se usan por separado. Si no se permitiera esto, fallaría lo siguiente:

private Dictionary<string, int> database = new Dictionary<string, int>(); 
private object databaseLock = new object(); 
public void AddOrUpdate(string item) 
{ 
    lock (databaseLock) 
    { 
     if (Exists(item)) 
      database.Add(item, 1); 
     else 
      ++database[item]; 
    } 
} 
public bool Exists(string item) 
{ 
    lock (databaseLock) 
    { 
     //... Maybe some pre-processing of the key or item... 
     return database.ContainsKey(item); 
    } 
}