android thread management onPause

Question

Tengo una clase que amplía la clase Thread y tiene su método de ejecución implementado como tal.

public void run(){ 
    while(!terminate){ 
     if(paused){ 
      Thread.yield(); 
     }else{ 
      accummulator++; 
     } 
    } 
} 

Este tema se generó a partir del método onCreate.

Cuando mi UI está oculta (cuando se presiona la tecla de inicio) mi método onPause establecerá la marca pausada en verdadero y cederá la banda de rodadura. Sin embargo, en el DDMS todavía veo el uTime del hilo acumulado y su estado como "en ejecución".

Así que mi pregunta es. ¿Cuál es la forma correcta de detener el hilo para que no agote el tiempo de CPU?

Answer 1

En realidad, es una mala práctica mantener un hilo ejecutándose después de onPause. La razón es que, después de OnPause, es posible que su aplicación se pierda de memoria en cualquier momento sin que usted lo sepa, por lo tanto, no podrá realizar la limpieza después de usted.

La forma correcta de hacerlo es detener el hilo en Pausa y recrearlo en Reanudar. Si necesita estado, puede usar los métodos o configuraciones saveState integradas de Android o cualquiera que lo conserve.

Answer 2

Su variable paused probablemente se esté almacenando en caché localmente. Esto se debe a que solo se lee y no se modifica en el ciclo. Entonces, lo que está sucediendo es que el compilador/intérprete/jitter optimiza leyendo solo la variable una vez y luego solo ejecutando la rama else. Debe marcar ese campo como volatile para que la variable pause se lea cada iteración a través del ciclo. Consulte la documentation de la palabra clave volatile. Aquí está some info about threading y some info about synchronization and concurrency.

Answer 3

Aunque estés invocando thread.yield(), estás dentro de un ciclo while() que probablemente esté repitiendo miles de veces por segundo, cada vez que se llame a .yield() pero el hecho de que está saliendo en bucle de control significa que está consumiendo recursos. Si pones un mensaje de Log.d allí, verás lo que quiero decir.

Recomiendo usar un Thread.sleep() en lugar de Thread.yield(). La razón es que, mientras un hilo está durmiendo, se produce. Además, con el modo de reposo obtienes la ventaja adicional de ralentizar el while() y no agotar los recursos. Un intervalo de espera de 500ms debería ser suficiente =)