¿Está bien? Sincronizada (rosca), luego pase = null en el bloque sincronía

Question

veo esto:

// thread is a member of this class 

synchronized(this.thread) 
{ 
    this.thread.running = false; 
    this.thread.notifyAll(); // Wake up anything that was .waiting() on 
    // the thread 
    this.thread = null; // kill this thread reference. 
    // can you do that in a synchronized block? 
} 

¿Está bien para establecer el thread=null si bien mantienen un bloqueo en ella?

Encontré este nugget en un poco de código BB.

Answer 1

Sí, está bien. La instrucción sincronizada tomará una copia de la referencia en la que se está bloqueando y utilizará la copia para determinar qué desbloquear al final.

Section 14.19 de la especificación del lenguaje Java no es realmente claro acerca de esto, pero lo hace estado que la expresión se evalúa en el inicio - y no menciona la evaluación de nuevo más adelante.

Answer 2

Hay una diferencia:

synchronized(this.thread) 

está sincronizando en el objeto del campo this.thread puntos a

this.thread = null; 

Usted está reasignando el campo. No está haciendo nada con el objeto al que hizo referencia anteriormente, por lo que el bloqueo sigue siendo válido.

Answer 3

Puede hacerlo, pero es casi seguro que el código sea incorrecto para lo que sea que intente lograr. Publique todo el código, y le garantizo que obviamente el programador no entiende la concurrencia.

No reasigne una variable que se utiliza para la sincronización.

Answer 4

Solo tiene un problema si también tiene un bloque que le asigna un nuevo valor al hilo. En ese caso, tiene una condición de carrera ya que los dos bloques no se bloquean en el mismo objeto, pero actualizará el mismo campo y será aleatorio en cuanto a qué bloque asigna el último valor.

Answer 5

La expresión sincronizada se desreferencia en la entrada, por lo que los usuarios posteriores de este bloqueo obtendrán una NullPointerException. Puede solucionarlo colocando un cheque nulo antes del bloque sincronizado, pero luego ha introducido una condición de carrera.