java开发:每日bug001(慎用对象的引用)

【Java江湖，bug（不拘）一格】

嘿，各位编程大侠、代码小侠们，欢迎来到我们的神秘小岛——“Java每日虫洞探险”！在这里，我们将一起记录并解决那些奇奇怪怪的bug。

案例分享：

我曾遇到这样一个场景：需要遍历一个对象集合，集合中包含月初和月末的日期对象Calendar。我的任务是分天查询每个id对应一个月的数据，每个id分天查询完成后，再重置日期查询下一个id。

异常现象：

在结果文件中，只存在第一个id的月数据。

问题代码：

java

for (MonthRange monthRange : monthRanges) {
    executorService.submit(() -> {
        // 初始化日期
        Calendar finalStartTimeBeginOfDay = monthRange.getMonthStart();
        Calendar finalEndTimeBeginOfDay = monthRange.getMonthEnd();
 
        List shipIdList = shipIdGetter.getShipIds(finalStartTimeBeginOfDay);
        for (String shipId : shipIdList) {
            // 日期重置
            Calendar startTimeBeginOfDay = finalStartTimeBeginOfDay;
            Calendar endTimeBeginOfDay = finalEndTimeBeginOfDay;
 
            ExportPositionResult tempResult = new ExportPositionResult();
 
            while (CalendarUtil.compare(startTimeBeginOfDay, endTimeBeginOfDay) <= 0) {
                try {
                    // 根据id按天查询月数据
                    // ...（省略具体查询代码）
                } finally {
                    // 开始日期加1天
                    startTimeBeginOfDay.add(Calendar.DATE, 1);
                }
            }
            tempWriter.flush();
        }
    });
}

相信各位肯定已经发现问题了！

--------------------------------------------

经过仔细排查日志，我发现第一次循环后，日期对象已经被修改。后续的startTimeBeginOfDay引用的finalStartTimeBeginOfDay对象也已经改变，导致后续循环中的数据查询出错。

修改后代码：

java

for (MonthRange monthRange : monthRanges) {
    executorService.submit(() -> {
        // 初始化日期
        Calendar finalStartTimeBeginOfDay = monthRange.getMonthStart();
        Calendar finalEndTimeBeginOfDay = monthRange.getMonthEnd();
 
        List shipIdList = shipIdGetter.getShipIds(finalStartTimeBeginOfDay);
        for (String shipId : shipIdList) {
            // 单独克隆一份对象副本使用，避免操作原始对象
            Calendar startTimeBeginOfDay = (Calendar) finalStartTimeBeginOfDay.clone();
            Calendar endTimeBeginOfDay = (Calendar) finalEndTimeBeginOfDay.clone();
 
            ExportPositionResult tempResult = new ExportPositionResult();
 
            while (CalendarUtil.compare(startTimeBeginOfDay, endTimeBeginOfDay) <= 0) {
                try {
                    // 根据id按天查询月数据
                    // ...（省略具体查询代码）
                } finally {
                    // 按天加，此时操作的是克隆后的对象，不影响原始对象
                    startTimeBeginOfDay.add(Calendar.DATE, 1);
                }
            }
            tempWriter.flush();
        }
    });
}

完美搞定！

总结：

在Java中，Calendar是一个抽象类，其具体实现（如GregorianCalendar）是可变的。这意味着当创建一个Calendar对象并对其进行修改时，这个对象的内部状态会随之改变。

在原始代码中，finalStartTimeBeginOfDay和finalEndTimeBeginOfDay是通过方法monthRange.getMonthStart()和monthRange.getMonthEnd()获取的Calendar对象。这些对象代表了某个月的第一天和最后一天的开始时刻。然而，由于Calendar对象是可变的，当我们将这些对象赋值给循环中的startTimeBeginOfDay和endTimeBeginOfDay时，实际上引用的是同一个对象。因此，在循环中对startTimeBeginOfDay调用add(Calendar.DATE, 1)时，也会修改finalStartTimeBeginOfDay引用的对象，导致循环超出预期。

解决办法：

使用克隆方法创建一个新的Calendar实例。这个新实例的初始状态与原始实例相同，但它们是完全独立的对象。因此，对克隆后的对象所做的任何修改都不会影响到原始对象。

在平时的开发过程中，我们应该注意多个代码块、多线程中使用同一个对象引用时是否满足预期结果。特别是在使用可变对象时，要谨慎处理对象的共享和修改。

结束语：

今天的Bug故事就讲到这里。虽然每个Bug都像是一场小小的“灾难”，但正是这些挑战让我们的编程之旅充满了乐趣和成就感。好啦，明天见了各位！愿你每天都能轻松搞定Bug，享受编程带来的无限乐趣！

【今日bug迹，明日传奇身】