Mettre à jour l'interface utilisateur sans geler

Question

J'ai un GridLayout avec plusieurs boutons.

J'ai besoin de changer la couleur de chaque bouton dans une séquence, disons, toutes les 3 secondes.

J'ai un problème ici ... Jetez un oeil à mon code:

runnableScan = new Runnable() { 
    @Override 
    public void run() { 
     GridLayout view = Grid; 
     for (int i = 0; i < view.getChildCount(); i++) { 
      Button btn = (Button)view.getChildAt(i); 
      btn.setBackgroundColor(Color.RED); 
      handlerScan.postDelayed(this, 3000);           
     } 
    } 
};  
handlerScan.postDelayed(runnableScan, 3000);  

Eh bien .. il ne change la couleur de tous les boutons au rouge, mais il les change à la fois !!! Je voulais qu'il change les couleurs dans une séquence, un bouton toutes les 3 secondes.

Si je mets un Toast à l'intérieur, le Toast sera déclenché toutes les trois secondes avec l'index de chaque bouton.

Qu'est-ce que je fais mal? Pourquoi tous les boutons sont-ils changés en rouge à la fois?

Answer 1

méthode run() du runnable est encore bouclez tous boutons et changer la couleur sur tous d'entre eux au cours d'un appel à run().

Cette ligne:

handlerScan.postDelayed(this, 3000);           

ne pas ajouter un délai entre les itérations de la boucle for. Il ne planifie que run() d'être appelé à nouveau, après 3 secondes. Essayez ceci (avec non pour la boucle).

runnableScan = new Runnable() { 
    @Override 
    public void run() { 

     GridLayout view = Grid; 
     Button btn = (Button)view.getChildAt(buttonIndex++); 
     if (btn != null) { 
      btn.setBackgroundColor(Color.RED); 
      handlerScan.postDelayed(this, 3000); 
     } 
    } 
    }; 
    handlerScan.postDelayed(runnableScan, 3000); 

Vous aurez besoin de garder une trace de buttonIndex comme variable membre dans votre classe:

private int buttonIndex; 

puis l'initialiser à l'index de votre premier bouton. Par exemple, si vous définissez votre point de vue du contenu en onCreate(), utilisez:

protected void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 
     setContentView(R.layout.some_layout_name); 

     GridLayout view = Grid; 
     for (buttonIndex = 0; buttonIndex < view.getChildCount(); buttonIndex++) { 
     View v = view.getChildAt(buttonIndex); 
     if (v instanceof Button) { 
      // we found the first button, so stop looping 
      break; 
     } 
     } 

     handlerScan = new Handler(); 

La façon dont le code que je vous ai montré fonctionne est que cela va changer le bouton 1 de couleur rouge, puis 3 secondes plus tard, le bouton de changement 2 rouge, puis 3 secondes plus tard, changez le bouton 3 en rouge. Si vous voulez recommencer le processus, il suffit de réinitialiser buttonIndex à l'index du premier bouton, après avoir terminé le changement du bouton 3.