Annuler la boîte de dialogue après 3 secondes - continue de bloquer mon application après plusieurs utilisations

Question

J'ai une classe de dialogue étendue que je veux afficher pendant 3 secondes puis disparaître. Cela fonctionne très bien les 2 premières fois que ça s'appelle, mais ensuite il plante mon application après ça. Certes, je ne suis pas le meilleur avec des discussions et je pense que c'est là où mon problème pourrait être. Comme vous pouvez le voir dans le code ci-dessous (section commentée), j'ai essayé d'utiliser un événement cancel pour supprimer le thread généré, mais cela le fait tomber en panne la première fois qu'il est lancé. J'ai également essayé de faire tout cela sur le thread d'interface utilisateur de la classe parent qui donne le même résultat que celui-ci (se bloque après 3 fois l'affichage de la boîte de dialogue).

import java.util.Timer; 
import java.util.TimerTask; 

import android.app.Dialog; 
import android.content.Context; 
import android.content.DialogInterface; 
import android.os.Handler; 


public class HandResults extends Dialog implements DialogInterface { 

    HandResults hr; 
    Timer myTimer; 
    Handler hand = new Handler(); 
    Thread t; 

    public HandResults(Context context) { 
     super(context); 
     setContentView(R.layout.handresults); 
     hr = this; 
     /* 
     this.setOnCancelListener(new OnCancelListener(){ 
      public void onCancel(DialogInterface dialog) { 
       t.destroy(); 

      } 
     }); 
     */ 
    } 

    public void showHands(){ 
     this.show(); 
     myTimer = null; 
     myTimer = new Timer(); 
     myTimer.schedule(new TimerTask() { 
      @Override 
      public void run() { 
       TimerMethod(); 
      } 

     }, 3000); 
    } 

    private void TimerMethod() 
    { 
     t = new Thread(){ 
      public void run(){ 
        hand.post(Timer_Tick);  
      } 
     }; 
     t.start(); 
    } 

    private Runnable Timer_Tick = new Runnable() { 
     public void run() { 
      hr.cancel(); 
     } 
    }; 
} 

Answer 1

1. Lorsque l'événement est onCancel que vous avez reçu et vous appelez t.destroy()destroy est une méthode désapprouvée.
2. Vous n'avez pas besoin de démarrer un autre thread si vous créez un minuteur, qui s'exécute déjà de manière asynchrone.

Donc cela pourrait fonctionner mieux:

public class HandResults extends Dialog implements DialogInterface { 

    HandResults hr; 
    Timer myTimer; 
    Handler hand = new Handler(); 

    public HandResults(Context context) { 
     super(context); 
     setContentView(R.layout.handresults); 
     hr = this; 
    } 

    public void showHands(){ 
     this.show(); 
     myTimer = null; 
     myTimer = new Timer(); 
     myTimer.schedule(new TimerTask() { 
      @Override 
      public void run() { 
       hr.cancel(); // call the cancel method directly 
      } 

     }, 3000); 
    } 
} 

Il n'y a pas besoin de créer votre propre fil, de sorte que le code ci-dessus devrait faire à peu près ce que vous essayez de faire.

Answer 2

Vous n'avez pas besoin du traitement ici, vous pouvez exécuter hand.post() dans le TimerTask. Et même le TimerTask/Timer n'est pas nécessaire, vous pouvez utiliser handler.postDelayed(). Je ne sais pas d'où vient le comportement que vous observez. En outre, vous devez considérer que votre boîte de dialogue est fermée tôt (par exemple, lorsque l'utilisation fait pivoter l'écran). Vous pouvez appeler handler.removeCallbacks() dans ce cas.