"Thread bereits gestartet" beim Fortsetzen der Aktivität

Question

Hier ist meine Situation: Ich baue ein Spiel für Android und meine Spielaktivität besteht aus einem benutzerdefinierten SurfaceView, die einen Thread für die Spiellogik und Rendering hat. Die Architektur ähnelt der LunarLander-Demo von Googles Website.

Wenn die Aktivität startet er den Surface erstellt und ruft diese Methode:

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

Wenn ich die Home-Taste drücken um das Spiel zu beenden, die onPause() -Methode aufgerufen, die surfaceDestroyed() aufruft. In surfaceDestroyed stoppen ich das Spiel Thema durch den Aufruf:

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

Die App geht in den Hintergrund gut. Dann, wenn ich die App neu starte, indem ich auf das Symbol drücke, bekomme ich eine Meldung "Thread bereits gestartet" im Log zusammen mit einem "Force Close" Popup auf dem Bildschirm. Diese Meldung wird angezeigt, wenn die Aktivität beim Aufruf von start() für den Render-Thread die Methode "sourceCreated" aufruft.

Jetzt habe ich es stundenlang untersucht und kann nicht herausfinden, warum das so ist. Ich glaube, dass mein Thread gestoppt wird, wenn ich die App schließe, also verstehe ich nicht, warum es sagt, dass es bereits begonnen hat.

Answer 1

Diese Methoden tun nicht, was Sie denken, dass sie tun. Vom API doc:

ist es nie einen Thread mehr als einmal zu starten legal.

Und:

public final void stop() - Veraltete. Diese Methode ist von Natur aus unsicher.

Wenn Sie einen Thread zu unterbrechen wollen, müssen Sie Object.wait() und Objecft.notifyAll() aus dem Thread verwenden.

Answer 2

Eine schlechte Lösung, aber es funktioniert ..

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

    } 

Korrekturen sind herzlich willkommen.

Answer 3

Meiner Meinung nach sollten Sie Ihren Code nicht in eine Unterklasse von Thread packen, wenn Sie den Thread häufig starten und stoppen wollen (die Beispiele machen das, weil er den Code kürzer macht). Verwenden Sie stattdessen eine ausführbare Datei. Auf diese Weise können Sie den alten Thread anhalten und verwerfen, wenn Sie möchten, und ein neues Thread-Objekt erstellen, um das Runnable auszuführen, wenn Sie es erneut starten müssen.

Auch auf Ausnahmen ist schlechte Form. Dies sollte als letzte Möglichkeit verwendet werden, wenn sich der eigene Code unerwartet verhält.