调用removeCallbacksAndMessages（null）后，处理程序不会结束

Question

我有两个处理程序。处理程序中的处理程序。它们都处于for循环中。

概述是这样的，

for{
    handler.postDelayed(runnableA{
        for{
            handler2.postDelayed(runnableB{
                function();
            }, 3000);
        }
    }, 1000);
}

当用户点击back button时，我想随时结束处理程序的工作。所以，我创建了两个Runnable类，以便我可以使用像runnableA.removellbacksAndMessages(null)这样的东西。

Handler messageHandler;
Handler countDownHandler;

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

    Toast.makeText(this, "Start Play in 5 seconds", Toast.LENGTH_SHORT).show();

    countDownHandler = new Handler();

    for (int i = 7; i >= 0; --i) {
        final int idx = i;
        Runnable countRunnable = new CountRunnable(idx, countDownView);
        countDownHandler.postDelayed(countRunnable, 1000 * (7 - i));
    }
}

这是Runnable Classes。

public class CountRunnable implements Runnable {
    int idx;
    TextView countDownView;

    public CountRunnable(int idx, TextView countDownView) {
        this.idx = idx;
        this.countDownView = countDownView;
    }

    @Override
    public void run() {
        int messageSize = messageItems.size();
        for (int j = 0; j < messageSize; j++) {
            final int jdx = j;
            messageHandler = new Handler();
            Runnable messageRunnable = new MessageRunnable(jdx);
            messageHandler.postDelayed(messageRunnable, 3000 * jdx);
        }
    }
}

class MessageRunnable implements Runnable {
    int jdx;

    public MessageRunnable(int jdx) {
        this.jdx = jdx;
    }

    @Override
    public void run() {
        addMessageView(messageItems.get(jdx));
    }
}

这是onBackPressed()：

@Override
public void onBackPressed() {
    super.onBackPressed();
    Toast.makeText(getApplicationContext(), "All Work Ended.", Toast.LENGTH_SHORT).show();
    scrollFlag = true;

    try {
        messageHandler.removeCallbacksAndMessages(null);
    } catch (Exception e) {
        Log.d(TAG, "messageHandler never used");
        e.printStackTrace();
    }
    try {
        countDownHandler.removeCallbacksAndMessages(null);
    } catch (Exception e) {
        e.printStackTrace();
    }
}

public void addMessageView(String message){
    try{
        mTextView.setText(message);
    }catch(Exception e){
        Toast.makeText(getApplicationContext(), "Abnormal End", Toast.LENGTH_SHORT).show();
    }
}

但是，我不断收到错误，因为活动已经结束，但处理程序无法找到活动。因此，Abnormal End Toast消息显示内部for循环的大小。

如果我不使用Toast消息，我可以忽略它，但我害怕内存泄漏或Bad形成的程序或类似的东西。

我该如何解决这个问题？

Answer 1

主要问题是你正在创建n数量的CountRunnables和m数MessageRunnables。尽管创建了多个处理程序，但您只能删除最新创建的Hanlder的回调。

这是你应该做的：

保留所有Handlers和Runnables的参考，并在所有这些上调用messageHandler.removeCallbacksAndMessages(null);和countDownHandler.removeCallbacksAndMessages(null);。