Android闹钟设置早于当前时间/午夜后立即触发问题求助

嘿，这个问题我一眼就看出来了——你处理闹钟日期的逻辑顺序搞反了，导致设置过去的时间时，AlarmManager会直接触发（因为它认为这个闹钟应该已经响了）。

问题根源 #

你现在的代码里有这一行：

if (calendar.before(Calendar.getInstance())) calendar.add(Calendar.DATE, 1);

但这行代码是在设置用户选择的小时分钟之前执行的！这时候calendar刚创建，就是当前时间，所以before判断永远是false，根本不会触发加一天的操作。等你设置完用户选的时间（比如16:00，当前是17:00），calendar就变成了过去的时间，AlarmManager拿到这个过去的时间戳，自然会立即触发闹钟。

修复方案 #

调整逻辑顺序：先设置用户选择的时间，再判断这个时间是否在当前时间之前，如果是，就给Calendar加一天。

修改你的saveButton.setOnClickListener里的Calendar处理代码：

saveButton.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View v) {
        // 获取当前时间的Calendar实例
        Calendar calendar = Calendar.getInstance();
        long currentTimeMillis = System.currentTimeMillis();

        // 先设置用户选择的小时、分钟
        if (Build.VERSION.SDK_INT >= 23) {
            calendar.set(Calendar.HOUR_OF_DAY, timePicker.getHour());
            calendar.set(Calendar.MINUTE, timePicker.getMinute());
            hour = timePicker.getHour();
            minute = timePicker.getMinute();
        } else {
            calendar.set(Calendar.HOUR_OF_DAY, timePicker.getCurrentHour());
            calendar.set(Calendar.MINUTE, timePicker.getCurrentMinute());
            hour = timePicker.getCurrentHour();
            minute = timePicker.getCurrentMinute();
        }
        // 统一设置秒和毫秒为0，确保闹钟触发时间精确
        calendar.set(Calendar.SECOND, 0);
        calendar.set(Calendar.MILLISECOND, 0);

        // 关键判断：如果设置的时间在当前时间之前，说明是第二天的闹钟
        if (calendar.getTimeInMillis() < currentTimeMillis) {
            calendar.add(Calendar.DATE, 1);
        }

        // 格式化时间字符串
        hourString = hour < 10 ? "0" + hour : String.valueOf(hour);
        minuteString = minute < 10 ? "0" + minute : String.valueOf(minute);
        textView.setText("Next alarm: " + hourString + ":" + minuteString);

        intent.putExtra("extra", "on");
        setAlarm(calendar.getTimeInMillis());
        // notification();
    }
});

额外优化建议 #

1. 使用精确的闹钟API：从Android 4.4（API 19）开始，AlarmManager.set()会变成不精确调度，建议改用setExactAndAllowWhileIdle()（针对Doze模式也能触发）或者setExact()，确保闹钟准时触发：

   private void setAlarm(long timeInMillis) {
       int flags = PendingIntent.FLAG_UPDATE_CURRENT;
       if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
           flags |= PendingIntent.FLAG_IMMUTABLE;
       }
       pendingIntent = PendingIntent.getBroadcast(this, 0, intent, flags);
       
       if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
           alarmManager.setExactAndAllowWhileIdle(AlarmManager.RTC_WAKEUP, timeInMillis, pendingIntent);
       } else if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.KITKAT) {
           alarmManager.setExact(AlarmManager.RTC_WAKEUP, timeInMillis, pendingIntent);
       } else {
           alarmManager.set(AlarmManager.RTC_WAKEUP, timeInMillis, pendingIntent);
       }
   }
   

2. 修复取消按钮的潜在空指针：如果用户还没设置过闹钟就点击取消，pendingIntent会是null，导致崩溃。建议在取消时增加判断：

   cancelButton.setOnClickListener(new View.OnClickListener() {
       @Override
       public void onClick(View v) {
           textView.setText("Alarm off!");
           if (pendingIntent != null) {
               alarmManager.cancel(pendingIntent);
           }
           intent.putExtra("extra", "off");
           sendBroadcast(intent);
           // notificationManager.cancel(0);
       }
   });
   

这样修改后，不管用户设置的是当天晚于当前的时间，还是早于当前（第二天）的时间，或者午夜后的时间，都会正确计算触发时间，不会立即触发了。

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