Interbloqueado utilizado para incrementar/imitar un booleano, ¿es esto seguro?

Question

Me pregunto si este código que un desarrollador compañero (que se ha ido) está bien, creo que quería evitar poner un candado. ¿Hay una diferencia de rendimiento entre esto y solo usar un bloqueo directo?

private long m_LayoutSuspended = 0; 
    public void SuspendLayout() 
    { 
     Interlocked.Exchange(ref m_LayoutSuspended, 1); 
    } 

    public void ResumeLayout() 
    { 
     Interlocked.Exchange(ref m_LayoutSuspended, 0); 
    } 

    public bool IsLayoutSuspended 
    { 
     get { return Interlocked.Read(ref m_LayoutSuspended) != 1; } 
    } 

¿Estaba pensando que algo así sería más fácil con un candado? De hecho, será utilizado por varios hilos, por lo que se decidió el uso de bloqueo/enclavamiento.

Answer 1

Personalmente me gustaría usar un booleano volátil:

private volatile bool m_LayoutSuspended = false; 
public void SuspendLayout() 
{ 
    m_LayoutSuspended = true; 
} 

public void ResumeLayout() 
{ 
    m_LayoutSuspended = false; 
} 

public bool IsLayoutSuspended 
{ 
    get { return m_LayoutSuspended; } 
} 

Por otra parte, como he reconocido recientemente en otros lugares, volátil, no quiere decir exactamente lo que pensé que lo hizo. Sospecho que esto está bien aunque :)

Incluso si te quedas con Interlocked, lo cambiaría a int ... no hay necesidad de hacer que los sistemas de 32 bits tengan dificultades para hacer un atómico de escritura de 64 bits cuando puedan hacerlo fácilmente con 32 bits ...

Answer 2

Si lo que está haciendo es seguro desde el punto de vista de alcanzar el campo m_LayoutSuspended carrera, sin embargo, se requiere un bloqueo por la siguiente razón si el código hace lo siguiente:

if (!o.IsLayoutSuspended) // This is not thread Safe ..... 
{ 
    o.SuspendLayout(); // This is not thread Safe, because there's a difference between the checck and the actual write of the variable a race might occur. 
    ... 
    o.ResumeLayout(); 
} 

Una forma más segura, que usa CompareExchange para asegurarse de que no haya carreras las condiciones han ocurrido:

private long m_LayoutSuspended = 0; 
public bool SuspendLayout() 
{ 
    return Interlocked.CompareExchange(ref m_LayoutSuspended, 1) == 0; 
} 

if (o.SuspendLayout()) 
{ 
    .... 
    o.ResumeLayout(); 
} 

O mejor aún, simplemente use un candado.