Невозможно правильно использовать обработчик - PullRequest
Новая
       22

Невозможно правильно использовать обработчик

0 голосов
/ 11 февраля 2019

Я начал изучать Android Development.Я строил простую сложную игру, в которой пользователь должен нажать на кнопку, которая показывает сложение двух чисел.Есть четыре Textview с.Сначала дайте time limit за каждый вопрос.Второй дает question для пользователя.Третий дает current score пользователя, а последний - правильность или неправильность выбранного открытия.Все работает, кроме первой кнопки.Каждый раз, когда нажимается кнопка, отсчет идет очень быстро.

Screenshot

// When First button is pressed

public void onClickButton1(View view) {
        correctIncorrect.setVisibility(View.VISIBLE);
        if (Integer.parseInt(button1.getText().toString())==sum) {
            correctIncorrect.setText("Correct");
            correctUpdater();
        }
        else {
            correctIncorrect.setText("Incorrect");
            inCorrectUpdater();
        }

}

//Similar to all the buttons


//To Update the score

public void correctUpdater() {
        n++;
        yourScore++;
        score.setText(Integer.toString(yourScore) + "/" + Integer.toString(n));
        update();
}

public void inCorrectUpdater() {
    n++;
    score.setText(Integer.toString(yourScore) + "/" + Integer.toString(n));
    update();
}


// To update the timer

//=======================================================================//
    public void resetTimer() {
        timer.setText(Integer.toString(temp)+"s");
        if (temp == 0) {
            inCorrectUpdater();
            update();
        }
        else {
            timeUpdater();
        }

    }

    public void timeUpdater() {
        Handler timeHandler = new Handler();
        Runnable timeRunnable = new Runnable() {
            @Override
            public void run() {
                temp--;
                resetTimer();
            }
        };
        timeHandler.postDelayed(timeRunnable,1000);
    }
    //=================================================================//

// Updater function

public void update() {

        correctIncorrect.setVisibility(View.INVISIBLE);
        Random random = new Random();
        int a = random.nextInt(21);
        int b = random.nextInt(21);
        question.setText(Integer.toString(a) + " + " + Integer.toString(b));
        Log.i("info", "onCreate: a = " + a);
        Log.i("info", "onCreate: b = " + b);
        sum = a+b;
        Log.i("info", "onCreate: sum = " + sum);
        int whichButton = random.nextInt(4);
        Log.i("info", "onCreate: random button is " + whichButton);
        values.clear();
        for (int i = 0; i< 4; i++) {
            if (i == whichButton) {
                values.add(sum);
            }
            else {
                values.add(random.nextInt(50));
            }
            Log.i("info", "onCreate: value[" + i + "] = " + values.get(i));
        }
        button1.setText(Integer.toString(values.get(0)));
        button2.setText(Integer.toString(values.get(1)));
        button3.setText(Integer.toString(values.get(2)));
        button4.setText(Integer.toString(values.get(3)));
        temp = 10;
        resetTimer();
}

Использую ли я Handler неправильно?Что я могу сделать?

1 Ответ

0 голосов
/ 11 февраля 2019

Я использую обработчик неправильно?Что я могу сделать?

Лучший способ сделать это - CountDownTimer , с его помощью вам не придется иметь дело с Handler с собой и с вамитакже есть возможность отменить работающий CountDownTimer.

Все работает, кроме первой кнопки

Не совсем уверен, что может вызывать различные поведения нанажатие кнопок с одинаковым кодом в onClick ().Но вы можете рассмотреть возможность использования одного и того же onClickListener для всех 4 кнопок следующим образом: 

View.OnClickListener myListener = new View.OnClickListener() {
        @Override
        public void onClick(View button) {
            correctIncorrect.setVisibility(View.VISIBLE);
            if (Integer.parseInt(button.getText().toString())==sum) {
                correctIncorrect.setText("Correct");
                correctUpdater();
            }
            else {
                correctIncorrect.setText("Incorrect");
                inCorrectUpdater();
            }
        }
    };

button1.setOnClickListener(myListener);
button2.setOnClickListener(myListener);
button3.setOnClickListener(myListener);
button4.setOnClickListener(myListener);

Это обеспечит одинаковое поведение при нажатии любой из 4 кнопок (в зависимости отна ответ, конечно)

Если вы запутались с методом onTick () в CountDownTimer, вы можете использовать модифицированное CountDownTimer , которое гарантирует, что таймер не пропустит никаких тиков.

...