Time Picker异常：定时任务每分钟触发直至设定时间而非仅在该时间执行

我来帮你搞定这个Toast每分钟弹一次的问题！这大概率是因为你的定时逻辑写得有问题——比如用了每分钟重复触发的任务，却没在到达目标时间时停止它，或者每次触发后又重新启动了循环，才会导致Toast一直弹到设定时间。

问题根源分析 #

通常出现这种情况，是因为你可能用了类似Timer.scheduleAtFixedRate或者每分钟调用一次的Handler.postDelayed，每次触发时都检查时间，但没在时间匹配后终止这个定时任务，导致它一直重复执行。

正确的解决方案 #

我们应该只触发一次延迟任务，而不是每分钟重复。具体步骤：

• 获取用户在TimePicker上设置的小时和分钟，转换成毫秒级的时间戳
• 计算当前时间和目标时间的差值：如果差值大于0，就延迟对应毫秒数后显示Toast；如果差值小于等于0（比如设置了过去的时间），直接给用户提示
• 用Handler实现延迟任务，并且在Fragment销毁时移除未执行的回调，避免内存泄漏

修正后的代码示例 #

public class ManualControlsFragment extends Fragment {
    private TimePicker tp;
    private Calendar calendar;
    private Handler toastHandler = new Handler();
    private Runnable toastRunnable;

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        View view = inflater.inflate(R.layout.your_fragment_layout, container, false);
        tp = view.findViewById(R.id.your_timepicker_id);
        calendar = Calendar.getInstance();

        // 监听TimePicker的时间变化
        tp.setOnTimeChangedListener((view1, hourOfDay, minute) -> {
            // 先移除之前可能存在的未执行任务，避免重复触发
            if (toastRunnable != null) {
                toastHandler.removeCallbacks(toastRunnable);
            }

            // 构建目标时间的Calendar对象
            Calendar targetCalendar = Calendar.getInstance();
            targetCalendar.set(Calendar.HOUR_OF_DAY, hourOfDay);
            targetCalendar.set(Calendar.MINUTE, minute);
            targetCalendar.set(Calendar.SECOND, 0);
            targetCalendar.set(Calendar.MILLISECOND, 0);

            long currentTime = System.currentTimeMillis();
            long targetTime = targetCalendar.getTimeInMillis();
            long timeDifference = targetTime - currentTime;

            // 目标时间在未来，延迟对应时间后显示Toast
            if (timeDifference > 0) {
                toastRunnable = () -> {
                    Toast.makeText(getContext(), "时间到啦！", Toast.LENGTH_SHORT).show();
                    // 任务执行完后清空引用
                    toastRunnable = null;
                };
                toastHandler.postDelayed(toastRunnable, timeDifference);
            } else {
                // 目标时间已过期，直接提示用户
                Toast.makeText(getContext(), "设置的时间已过期！", Toast.LENGTH_SHORT).show();
            }
        });

        return view;
    }

    @Override
    public void onDestroyView() {
        super.onDestroyView();
        // Fragment销毁时清理未执行的回调，防止内存泄漏
        if (toastRunnable != null) {
            toastHandler.removeCallbacks(toastRunnable);
        }
    }
}

关键改进点 #

• 每次用户修改时间时，先移除之前的延迟任务，避免多个任务同时存在
• 只设置一次性的延迟任务，而非每分钟重复触发
• 在Fragment销毁时清理Handler的回调，避免内存泄漏
• 处理了目标时间在过去的情况，给用户合理提示

这样修改后，Toast就只会在用户设定的时间点弹一次，不会每分钟重复出现啦！

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