"Subproceso ya iniciado" al reanudar la actividad

Question

Aquí está mi situación: Estoy creando un juego para Android y la actividad de mi juego se compone de un SurfaceView personalizado que tiene un hilo para la lógica del juego y la representación. La arquitectura es similar a la demo LunarLander del sitio web de Google.

Cuando se inicia la actividad que crea la SurfaceView y llama a este método:

@Override 
    public void surfaceCreated(SurfaceHolder holder) 
    { 
     renderThread.start(); 
    } 

Al pulsar el botón de inicio para salir del juego, el método onPause() se llama, que llama surfaceDestroyed(). En surfaceDestroyed dejo el juego Thread llamando al:

@Override 
    public void surfaceDestroyed(SurfaceHolder holder) 
    { 
     synchronized(holder) 
     { 
      renderThread.stop(); 
     } 
    }  

La aplicación va bien al fondo. Luego, cuando reinicio la aplicación presionando el ícono, aparece el mensaje "Subproceso ya iniciado" en el registro junto con una ventana emergente "forzar cierre" en la pantalla. Este mensaje ocurre cuando la actividad ingresa al método "surfaceCreated" cuando llama a start() en el hilo de renderizado.

Ahora lo he visto durante horas y no puedo entender por qué es esto. Creo que mi hilo se detiene cuando cierro la aplicación, así que no entiendo por qué dice que ya ha comenzado.

Answer 1

Esos métodos no hacen lo que crees que hacen. Desde el API doc:

Nunca es legal para iniciar un hilo más de una vez.

Y:

public final void stop() - desuso. Este método es inherentemente inseguro.

Si desea hacer una pausa en un hilo, se tienen que utilizar Object.wait() y Objecft.notifyAll() desde el interior de la rosca.

Answer 2

una mala solución, pero funciona ..

public void surfaceCreated(SurfaceHolder holder) { 
     try{ 
     _thread.setRunning(true); 
     _thread.start(); 
     }catch(Exception ex){ 
      _thread = new TutorialThread(getHolder(), this); 
      _thread.start(); 
     } 

    } 

correcciones son bienvenidos.

Answer 3

En mi opinión, no debe empaquetar el código en una subclase de subproceso si tiene la intención de iniciar y detener el subproceso con frecuencia (los ejemplos lo hacen porque hace que el código sea más corto). Use un Runnable en su lugar. De esta forma, puede detener y descartar el hilo viejo cuando lo desee y crear un nuevo objeto hilo para ejecutar el ejecutable cuando necesite volver a iniciarlo.

private TutorialRunnable tutorialRunnable; 

... 

// Synchronization and error checking omitted for brevity. 
public void surfaceCreated(SurfaceHolder holder) { 
    thread = new Thread(tutorialRunnable); 
    thread.start(); 
} 

public void surfaceDestroyed(SurfaceHolder holder) { 
    tutorialRunnable.setRunning(false); 
    while (thread != null) { 
     try { 
      thread.join(); 
      thread = null; 
     } catch (InterruptedException e) { 
     } 
    } 
} 

Además, confiar en las excepciones es mala forma. Eso debería usarse como último recurso cuando su propio código se comporta de forma inesperada.