TextSwitcher配合SwipeRefresh刷新后切换加速问题排查与解决

我将在XML中定义好的TextSwitcher放入如下方法（该TextSwitcher默认每2秒自动切换一次文本），但每次使用swipeRefresh刷新页面时，它的切换速度都会越来越快。请问问题成因是什么？如何修复？

TextSwitcher ts = findViewById(R.id.textswitcher);

swipeRefresh.setOnRefreshListener(() -> {
        showSwitcher(new String[]{"NO WATER", "NO ENERGY"});
        swipeRefresh.setRefreshing(false);
                    });
...
private void showSwitcher(String[] mArray) {
     final int[] idx = {0};

            Runnable runnable = new Runnable() {
                @Override
                public void run() {
                    idx[0] = ((idx[0] + 1) < 2) ? (idx[0] + 1) : 0;
                    ts.setText(mArray[idx[0]]);
                    ts.postDelayed(this, 2000);
                }
            };

            ts.postDelayed(runnable, 2000);
}

效果说明 #

• 单次刷新效果：TextSwitcher每2秒正常切换一次文本
• 多次刷新效果：TextSwitcher切换速度持续加快，间隔远小于2秒

每次调用showSwitcher方法时，都会新建一个Runnable并通过ts.postDelayed添加到消息队列，但之前刷新时创建的旧Runnable并没有被取消，仍然在定时执行切换操作。多次刷新后，消息队列中会存在多个独立的Runnable同时触发文本切换，最终导致切换速度叠加变快。

方案1：复用全局Runnable，刷新前移除旧任务 #

定义全局Runnable变量，每次调用showSwitcher前，先移除TextSwitcher上未执行的旧任务，再添加新任务：

TextSwitcher ts = findViewById(R.id.textswitcher);
// 定义全局Runnable，用于后续取消任务
private Runnable switcherRunnable;

swipeRefresh.setOnRefreshListener(() -> {
        showSwitcher(new String[]{"NO WATER", "NO ENERGY"});
        swipeRefresh.setRefreshing(false);
                    });
...
private void showSwitcher(String[] mArray) {
     final int[] idx = {0};
     // 先移除之前未执行的任务
     if (switcherRunnable != null) {
         ts.removeCallbacks(switcherRunnable);
     }

     switcherRunnable = new Runnable() {
         @Override
         public void run() {
             idx[0] = ((idx[0] + 1) < 2) ? (idx[0] + 1) : 0;
             ts.setText(mArray[idx[0]]);
             ts.postDelayed(this, 2000);
         }
     };

     ts.postDelayed(switcherRunnable, 2000);
}

方案2：用Handler统一管理任务 #

如果需要更灵活的任务控制，可使用Handler持有任务，每次刷新时清空所有未执行的回调：

private Handler switcherHandler = new Handler(Looper.getMainLooper());
private Runnable switcherRunnable;

TextSwitcher ts = findViewById(R.id.textswitcher);

swipeRefresh.setOnRefreshListener(() -> {
        showSwitcher(new String[]{"NO WATER", "NO ENERGY"});
        swipeRefresh.setRefreshing(false);
                    });
...
private void showSwitcher(String[] mArray) {
     final int[] idx = {0};
     // 清空Handler中所有未执行的回调和消息
     switcherHandler.removeCallbacksAndMessages(null);

     switcherRunnable = new Runnable() {
         @Override
         public void run() {
             idx[0] = ((idx[0] + 1) < 2) ? (idx[0] + 1) : 0;
             ts.setText(mArray[idx[0]]);
             switcherHandler.postDelayed(this, 2000);
         }
     };

     switcherHandler.postDelayed(switcherRunnable, 2000);
}

额外优化：页面销毁时清理任务 #

为避免内存泄漏，在页面销毁时记得移除所有未执行的任务：

@Override
protected void onDestroy() {
    super.onDestroy();
    // 方案1的清理方式
    if (switcherRunnable != null) {
        ts.removeCallbacks(switcherRunnable);
    }
    // 方案2的清理方式
    switcherHandler.removeCallbacksAndMessages(null);
}

内容的提问来源于stack exchange，提问作者user22635266