Button sofort neu zeichnen

webratteAnfaenger · 28. Dezember 2016

Hallo erst mal und schöne Feiertage.

Ich bin noch ein wirklich blutiger Anfänger. So dieses oder jenes habe ich schon mit Hilfe meiner Lieblingssuchmaschine oder einfach durch ausprobieren herausgefunden.
Aber jetzt komme ich gerade nicht mehr weiter.

Folgendes habe ich vor.
Ich fülle ein Array mit Zufallszahlen. Dieses Array lese ich dann aus und will vier Schalter je nach ausgelesenem Wert umfärben. Das klappt soweit auch.

Dazu initialisiere ich die Button in der MainActivity.Von dort rufe ich dann auch (mit einem Klick auf einen Button) folgende Methode (ist das der richtige Ausdruck in diesem Fall?) auf.
Nun wird die For Schleife durchlaufen, Liest den ersten Wert aus dem Array aus und färbt den entsprechenden Button ein.
Allerdings wird die Änderung erst sichtbar wenn die Methode komplett abgearbeitet ist.
Ich möchte aber, dass die Schalter nach jedem Schleifendurchlauf bereits die neue Farbe haben.
Dann warte ich 1000ms und die Schalter sollen alle wieder grau sein.

Hat jemand eine Idee wie ich das Problem in den Griff bekommen kann?
Danke schon mal

Code

private void arrayAuslesen() {
            int inhaltArray;                       
            for (int schleifeZufallszahl =0;  schleifeZufallszahl &lt;= 10; schleifeZufallszahl++){
                inhaltArray = ZufallsZahlenArray[schleifeZufallszahl];
                        if (inhaltArray == 1) {
                            button_1.setBackgroundColor(Color.BLUE);
                        }
                        if (inhaltArray == 2) {
                            button_2.setBackgroundColor(Color.RED);
                        }
                        if (inhaltArray == 3) {
                            button_3.setBackgroundColor(Color.GREEN);
                        }
                        if (inhaltArray == 4) {
                            button_4.setBackgroundColor(Color.YELLOW);
                        }
                        try {
                            Thread.sleep(1000);
                        } catch (InterruptedException e) {
                            e.printStackTrace();
                        }
                        button_1.setBackgroundColor(Color.GRAY);
                        button_2.setBackgroundColor(Color.GRAY);
                        button_3.setBackgroundColor(Color.GRAY);
                        button_4.setBackgroundColor(Color.GRAY);


                    }


            }

Alles anzeigen

Ben · 29. Dezember 2016

hi

auf den ersten blick kann ich kein problem im code erkennen. Kannst du uns vielleicht noch zeigen wie deine methode aufgerufen wird und was davor und danach passiert?
Wenn du das thread sleep in deinem MainThread aufrufst, schläft der GUI thread solang wie du angibst, das könnte zumindest mit in dein problem hineinspielen.

toad · 29. Dezember 2016

das würde aber nicht unbedingt das Phänomen klären, das die Ganze Methode durchlaufen wird und dann erst der Button gefärbt wird, oder irre ich mich ?

webratteAnfaenger · 30. Dezember 2016

Danke schon mal das ihr euch meines Problems annehmt. Hier habe ich mal noch einen Codeschnipsel. Ich habe auch meine Kommentare mal drin gelassen, die Importe habe ich mir gespart. Ich kann mir jetzt nicht vorstellen das es mit dem "Schlafen" zusammen hängt. Wenn ich mir ein paar Breakpoints setze und debugge, dann sehe ich das die Schleife durchlaufen wird, an der entsprechenden Stelle pausiert und dann wieder durchläuft bis sie den Endwert erreicht hat. Dann springt das Programm erwartungsgemäß an die Stelle nach dem Aufruf.
Vielleicht sieht ein geschultes Auge ja was ich falsch mache. Denn der Fehler liegt sicher bei mir

Java

public class MainActivity extends AppCompatActivity  {


    Button button_start;                            
    Button button_1;                                
    Button button_2;                                
    Button button_3;                                
    Button button_4;                               


    TextView InfoFeldZufallsZahl;                   //Nur zum Tesen. Kann später entfernt werden                   
    int[] ZufallsZahlenArray;                      
    int TestVariable;                               //Nur zum Tesen. Kann später entfernt werden
    boolean ButtonAktiv;                            //Sollen die farbigen Button aktiv oder inaktiv sein
    boolean PlaySequense;                           //True wenn die Sequence gespielt werden soll


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);


        // View Elemente finden
        InfoFeldZufallsZahl = (TextView) findViewById(R.id.InfoFeldZufallsZahl);


        button_start = (Button) findViewById(R.id.button_start);
        button_1 = (Button) findViewById(R.id.button_1);
        button_2 = (Button) findViewById(R.id.button_2);
        button_3 = (Button) findViewById(R.id.button_3);
        button_4 = (Button) findViewById(R.id.button_4);


        //Disable Button zum Start
        ButtonAktiv = false;
        buttonAktivierenDeaktivieren();


        // Klick auf Button "Let's go"
        button_start.setOnClickListener(new OnClickListener() {
            @Override
            public void onClick(View v) {
                //ausführen bei click auf "let's go"
                TestVariable = 0;
                zufallsZahlErzeugen();
                //Disable startbutton nachdem er geklickt wurde
                button_start.setEnabled(false);
                //Sequence spielen
                sequenceSpielen();
            }


        });


        //Klick auf Button 1
        button_1.setOnClickListener(new OnClickListener() {
            @Override
            public void onClick(View v) {
                //ausführen bei click auf "1"
                TestVariable =100;
                zufallsZahlErzeugen();
                arrayAuslesen();
            }
        });
        ..........

Alles anzeigen

webratteAnfaenger · 31. Dezember 2016

Ups, ich habe da einen wichtigen Part des Quelltextes vergessen zu posten.
Das hole ich später nach.
Sorry und danke für eure Hilfe.

webratteAnfaenger · 31. Dezember 2016

So, jetzt gibt es noch eine relevanten Teil.
Und noch eine kurze Zusammenfassung was ich tue:
Beim Klick auf den Button_start rufe ich die Methode "sequenceSpielen" (zuvor werden die Zufallszahlen erzeugt und ins Array geschrieben) auf und deaktiviere den Button.
aus dieser Methode rufe ich wiederum die Methode "arrayAuslesen" auf. Hier sollen die Button umgefärbt werden (mein erster Post).
Nun habe ich vermutlich alle verwirrt.
Und ja, ich weiß das ich meinen Code noch deutlich übersichtlicher schreiben muss

Noch etwas: Bei meinem Post habe ich versehentlich eine Zeile gelöscht. Im original Quellcode ist in Zeile 4/5 noch

Java

if (PlaySequense == true)

Wenn ich sie auskommentiere ändert sich das Verhalten in der Situation allerdings nicht. Sie ist nur wichtig wenn ich das Array auslesen will ohne die Button umzufärben.

Java

private void sequenceSpielen() {
            PlaySequense = true;
            ButtonAktiv = true;
            buttonAktivierenDeaktivieren();
            arrayAuslesen();
        }

Button sofort neu zeichnen

Jetzt mitmachen!

Teilen

Tags