一个线上问题让我发现了Calendar类中的秘密-周一真的是每周的第一天吗?

一、问题描述

假设我们当前有一张业务表,表上有两个字段start_date和end_date分别记录了该业务的开始和结束时间Ps:按自然周的逻辑记录,还有个字段achieved根据1和0标记该业务是否完成。

现在有个需求是:查出当前系统时间内未完成的业务。这个需求很简单:获取本周的周一并与end_date进行比较,再把achieved设置为0即可查出符合需求的业务。因此,开发完成后,我简单地自测下就提测了。

问题就出在清明节假收假上班后的第一天,那是一个下着小雨的周日 。这种天气配合周天以及三天假期的快乐后遗症让我慢悠悠地到了公司,把没做完的需求盘点下做完然后就可以早点下班了。就在这时,组长找上了我,告诉我节前的这个需求有问题:今天明明才周日,但是未完成的业务已经查询出来了,应该到了周一才能找出来才对。

本来我是十分自信的,毕竟这么简单的需求,我还不能把它拿捏了吗?但我看到结果后傻眼了,确实未完成的业务提前查出来了,于是开始了修复BUG之路。

二、问题排查

要排查问题当然要先从看代码开始:

java 复制代码
//1.获取当前周的周一  
//1-1.创建Calendar实例  
Calendar cal = Calendar.getInstance();  
//1-2.将日期设置为周一  
cal.set(Calendar.DAY_OF_WEEK, Calendar.MONDAY);  
//1-3.使用Calendar的时间戳创建一个java.sql.Date实例  
java.sql.Date currentWeekMonday = new java.sql.Date(cal.getTimeInMillis());  
//2.设置查询条件
//2-1.声明查询条件构造对象
BusinessQueryCriteria businessQueryCriteria = new BusinessQueryCriteria();  
//2-2.设置查询参数
//2-2-1.设置状态
businessQueryCriteriasetStatus(1);
//2-2-2.设置用户id
businessQueryCriteria.setUserId(userId);
//2-2-3.设置未完成
businessQueryCriteria.setAchieved(0);
//2-2-4.设置结束时间小于等于当前周的周一
businessQueryCriteria.setEndDateLessThan(currentWeekMonday);

乍一看没什么问题,这里也没什么复杂逻辑,就是很基础的查询,那么自然就要开始debug了,于是在这个方法的一些关键位置打上断点。果然,debug发现问题:今天是2024-04-07 ,本周的周一应该是2024-04-01 ,但是通过debug发现1-3处的currentWeekMonday构建的日期却是2024-04-08,它本该生成的本周一却变成了下周一。

这就让人感觉很奇怪了,看起来获取日期那里也没什么问题,毕竟是很简单的逻辑。那么只能往更深层次看,于是点进 Calendar相关的源码发现了其中的秘密:

Calendar 类中,周日被默认为每周的第一天,周一为每周的第二天。那么回到我们上面的代码,问题的原因就水落石出了:今天是周日,在周日这天获取的本周一自然就是明天,在我们看来就是获取到了下周一,这里程序逻辑上没问题,但与我们实际场景使用逻辑相悖,因此针对周日需要特殊处理下。

三、问题修复

解决方案大致想到了以下三种,都可以解决这个问题,当然如果如果各位读者有别的方法,也欢迎在评论区多多交流!

1.加入判断机制

第一种方案思路自然是最简单的,既然只是周日这天会出现异常情况,那么我们就对周日多做一层判断,处理下就好了:

java 复制代码
//1.获取当前周的周一  
//1-1.创建Calendar实例  
Calendar cal = Calendar.getInstance();  
//1-2.判断今天是否是周日  
if (cal.get(Calendar.DAY_OF_WEEK) == Calendar.SUNDAY) {  
//1-2-1.如果是周日,将日期回退7天以上获取上周的周一  
cal.add(Calendar.DATE, -7);  
}  
//1-3.将日期设置为周一  
cal.set(Calendar.DAY_OF_WEEK, Calendar.MONDAY);

2.修改第一天

第二种方案就是利用Calendar提供的API重新将周一设置为第一天即可:

java 复制代码
//1.获取当前周的周一  
//1-1.创建Calendar实例  
Calendar cal = Calendar.getInstance();  
//1-2.设置周一为一周的第一天  
cal.setFirstDayOfWeek(Calendar.MONDAY);  
//1-3.将日期设置为周一  
cal.set(Calendar.DAY_OF_WEEK, Calendar.MONDAY);  
//1-4.使用Calendar的时间戳创建一个java.sql.Date实例  
java.sql.Date currentWeekMonday = new java.sql.Date(cal.getTimeInMillis());

3.使用别的类库

相比于JDK自带的类库,现在越来越多优质的开源类库不断涌现,可以让我们轻松实现日期时间相关的操作,比如Hutool(Ps:不是广告)的星期操作默认就是以周一为第一天:

java 复制代码
  //1.获取当前周的周一
  //1-1.获取当前日期
  Date date = DateUtil.date();
  //1-2.获取本周的周一
  Date monday = DateUtil.beginOfWeek(date);

四、小结

也算是一次比较有意思的排错,但是也提醒我们日常开发中无论是使用JDK自带的类库还是开源的工具类,对于其实现还是要有所了解的,这样在遇到问题的时候才能从容应对。

相关推荐
葫芦和十三4 小时前
图解 MongoDB 04|索引模型:每建一个索引,就是在 B+-tree 森林里多栽一棵
后端·mongodb·agent
用户47949283569155 小时前
claude Fable用不了?把Gpt 5.5pro接到你的claude code里
前端·后端
GetcharZp7 小时前
告别 Nginx 复杂配置!这款带 Web 面板的万能代理神器,让端口转发变得如此简单
后端
IT_陈寒9 小时前
React的useState居然还有这种坑?我差点删库跑路
前端·人工智能·后端
nanxun88610 小时前
记一次诡异的 Docker 容器"串包"故障排查
java
Pedantic10 小时前
SwiftUI 手势笔记
前端·后端
金銀銅鐵10 小时前
[Python] 从《千字文》中随机挑选汉字
后端·python
用户15630681035113 小时前
Day01 | Java 基础(Java SE)
java
飘尘13 小时前
前端转型全栈(Java后端)的快速上手指引
前端·后端·全栈