Как заставить TextView работать только с одним таймером?

Question

[Bafrektez]

Добрый день. Проблема заключается в следующем: есть таймер, наследованный от CountDownTimer, его значение передается в TextView. Также присутствуют две кнопки - старт и пауза. При повторном нажатии на старт должен происходить перезапуск таймера. Это происходит, но весьма странным образом. TextView получает два и более значения одновременно и получается, что в одну секунду отображаются два и более значения одновременно. Как "убить" предыдущий таймер?

public class MainActivity extends AppCompatActivity implements View.OnClickListener {
 
    TextView mTimer;
    ImageButton startButton, pauseButton;
    int timeToEnd = 25000;
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
 
        mTimer = (TextView) findViewById(R.id.tv);
        startButton = (ImageButton) findViewById(R.id.startButton);
        pauseButton = (ImageButton) findViewById(R.id.pauseButton);
        startButton.setOnClickListener(this);
        pauseButton.setOnClickListener(this);
    }
 
    @Override
    public void onClick(View v) {
        final TimerClass timer = new TimerClass(timeToEnd, 1000);
        switch (v.getId()) {
            case R.id.startButton:
                startButton.setOnClickListener(this);
                timer.start();
                break;
            case R.id.pauseButton:
                pauseButton.setOnClickListener(this);
                timer.cancel();
        }
    }
 
    public class TimerClass extends CountDownTimer {
        public TimerClass(long milliFuture, long interval) {
            super(milliFuture, interval);
        }
 
        @Override
        public void onTick(long millisUntilFinished) {
            long millis = millisUntilFinished;
            String hhmmss = String.format("%02d:%02d:%02d", TimeUnit.MILLISECONDS.toHours(millis),
                    TimeUnit.MILLISECONDS.toMinutes(millis) - TimeUnit.HOURS.toMinutes(TimeUnit.MILLISECONDS.toHours(millis)),
                    TimeUnit.MILLISECONDS.toSeconds(millis) - TimeUnit.MINUTES.toSeconds(TimeUnit.MILLISECONDS.toMinutes(millis)));
            mTimer.setText(hhmmss);
        }
 
        @Override
        public void onFinish() {
            mTimer.setText("Completed");
        }
    }
 
    public void showSetting() {
        Intent intent = new Intent(this, Prefs.class);
        startActivity(intent);
    }
 
    public boolean onCreateOptionsMenu(Menu menu) {
        MenuItem mi = menu.add(0, 0, 0, "Settings");
        mi.setIntent(new Intent(this, Prefs.class));
        return super.onCreateOptionsMenu(menu);
    }
}

P.S. Пробовал старую реализацию, там этой проблемы нет, но она мне не подходит, так как время должно задаваться извне и изменяться по желанию пользователя, а не быть строго фиксированным.

btnStart.setOnClickListener(new OnClickListener() {
            @Override
            public void onClick(View v) {

Answer 1

[Александр Дорофеев]

Я бы на вашем месте так бы не делал, но ладно:

1)Зачем задаёте текст у TextView в таймере?Это нарушает принцип Деметры и абстракцию.Просто делаем переменную currentTime в интерфейсе класса и создаём геттер к нему:

public class TimerClass extends CountDownTimer {
    public String getCurrentTime() {
        return currentTime;
    }

    private String currentTime;

    public TimerClass(long milliFuture, long interval) {
        super(milliFuture, interval);
    }

    @Override
    public void onTick(long millisUntilFinished) {
        long millis = millisUntilFinished;
        String hhmmss = String.format("%02d:%02d:%02d", TimeUnit.MILLISECONDS.toHours(millis),
                TimeUnit.MILLISECONDS.toMinutes(millis) - TimeUnit.HOURS.toMinutes(TimeUnit.MILLISECONDS.toHours(millis)),
                TimeUnit.MILLISECONDS.toSeconds(millis) - TimeUnit.MINUTES.toSeconds(TimeUnit.MILLISECONDS.toMinutes(millis)));
        currentTime = hhmmss;
    }

    @Override
    public void onFinish() {
        currentTime = "Completed";
    }
}

2)Теперь в классе активности создаём метод runTimer() с Handler, который будет
обновлять текст нашего TextView:

public class MainActivity extends AppCompatActivity implements View.OnClickListener {

    TextView mTimer;
    Button startButton, pauseButton;
    int timeToEnd = 25000;
    final TimerClass timer = new TimerClass(timeToEnd, 1000);

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

        mTimer = (TextView) findViewById(R.id.tv);
        startButton = (Button) findViewById(R.id.startButton);
        pauseButton = (Button) findViewById(R.id.pauseButton);
        startButton.setOnClickListener(this);
        pauseButton.setOnClickListener(this);
        runTimer();
    }

    @Override
    public void onClick(View v) {

        switch (v.getId()) {
            case R.id.startButton:
                startButton.setOnClickListener(this);
                timer.start();
                break;
            case R.id.pauseButton:
                pauseButton.setOnClickListener(this);
                timer.cancel();
        }
    }

    private void runTimer() {


        final Handler handler = new Handler();
        handler.post(new Runnable() {
            @Override
            public void run() {

                mTimer.setText(timer.getCurrentTime());
                handler.post(this);
            }
        });
    }

}

P.s
Очень костлявая реализация.Я бы сделал по-другому.

Comments

[Bafrektez]

Спасибо, все работает) Не подскажите, как тогда это лучше реализовать? Просто я новичок в разработке на андроиде и был бы рад правильным решениям)

[Bafrektez]

Однако не все так гладко. Я не могу изменить timeToEnd после нажатия. То есть могу, но timer запомнил его и не хочет менять. А это важно, потому что я пишу pomodoro-таймер и после выполнения одного таймера должен запускаться другой с меньшим или большим временем, а здесь я не могу его поменять.

[Александр Дорофеев]

Вообще - способов дофига.Но на начальном этапе, я бы выделил самый простой:
Вместо Timer лучше использовать Handler, как я и сделал, то есть всю работу
делаем в Runnable и запихиваем в Handler(детали по обратному отсчёту реализуете сами):
Статья: www.mopri.de/2010/timertask-bad-do-it-the-android-...
Более подробное рассмотрение:
stackoverflow.com/questions/4597690/android-timer-how
Статья о подобных задачах(на русском):
https://habrahabr.ru/post/136942/
Можете полистать 4 главу из этой книги:
shop.oreilly.com/product/0636920029045.do

[Bafrektez]

Александр Дорофеев: спасибо) А можете, пожалуйста, еще помочь с timeToEnd. Проблему обозначил выше)

[Александр Дорофеев]

Вот код(сделал рефакторинг, теперь только Handler):

package android.letscoder.xyz.timersolution;

import android.os.Bundle;
import android.os.Handler;
import android.support.v7.app.AppCompatActivity;
import android.view.View;
import android.widget.Button;
import android.widget.TextView;

public class MainActivity extends AppCompatActivity implements View.OnClickListener {

private TextView mTimer;
private Button startButton, pauseButton;
private final int TIME_WORK_SESSION = 1500;//время работы(в секундах)
private final int TIME_REST_SESSION = 300;//время отдыха(в секундах)
private int seconds = 0;
private boolean isRunning;
private boolean isWork;

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

mTimer = (TextView) findViewById(R.id.tv);
startButton = (Button) findViewById(R.id.startButton);
pauseButton = (Button) findViewById(R.id.pauseButton);
startButton.setOnClickListener(this);
pauseButton.setOnClickListener(this);

seconds = TIME_WORK_SESSION;
isWork = true;

runTimer();
}

@Override
public void onClick(View v) {

switch (v.getId()) {
case R.id.startButton:
isRunning = true;
break;
case R.id.pauseButton:
isRunning = false;
}
}

private void runTimer() {

final Handler handler = new Handler();
handler.post(new Runnable() {
@Override
public void run() {

int hours = seconds / 3600;
int minutes = (seconds % 3600) / 60;
int secs = seconds % 60;
String time = String.format("%d:%02d:%02d", hours, minutes, secs);
mTimer.setText(time);
if (isRunning && isWork && seconds == 0) {
isWork = false;
seconds = TIME_REST_SESSION;
} else if (isRunning && !isWork && seconds == 0) {
isWork = true;
seconds = TIME_WORK_SESSION;
} else if (isRunning) {
seconds--;
}

handler.postDelayed(this, 1000);//handler.post(this);
}
});
}

}

Реализация ,как видно отвратительная и говнокодерская , но работает:
в реальном проекте я бы использовал паттерн Состояние для переключения
между режимами.
Вот между прочим ссылочка на проект таймера:
https://github.com/dogriffiths/HeadFirstAndroid/tr...

[Bafrektez]

Жаль, что нельзя плюсовать еще и комментарии. Огромное спасибо еще раз)

P.S. Я эту проблему решил, добавив в onFinish запрос на новое создание taimer, но это тоже попахивает говнокодом + в такой реализации почему то не меняется TextView в onFinish

public void onFinish() {
if (countOfSmallBreak == 1) {
timer = new TimerClass(timeToPause, 1000);
countOfSmallBreak =-1;
countOfBigBreak =-1;
}
else {
timer = new TimerClass(timeToEnd, 1000);
countOfSmallBreak =+1;
}
if (countOfBigBreak == 0) {
timer = new TimerClass(bigCount, 1000);
}
currentTime = "Completed";
}

[Bafrektez]

Александр Дорофеев: а не подскажите, как в данной реализации сделать так, что бы пользователь нажимал на кнопку. а только потом каждый таймер запускался, а не автоматически?

[Bafrektez]

Уже решено)