5

我遇到了来自 java.util.Calendar 的奇怪行为：

import static org.junit.Assert.*;
import org.junit.Test;

import java.util.Calendar;

public class Tester1 {
    @Test
    public void test_monthOfDate() {
        assertEquals(1, monthOfDate(2013, 1, 30)); // OK
        assertEquals(1, monthOfDate(2013, 1, 31)); // OK

        // Start of February
        assertEquals(2, monthOfDate(2013, 2, 1));  // FAIL
        assertEquals(2, monthOfDate(2013, 2, 28)); // FAIL
        // to the end of it

        // and after that it is okay also
        assertEquals(3, monthOfDate(2013, 3, 1));  // OK
    }

    public int monthOfDate(int year, int month, int day) {
        Calendar cal = Calendar.getInstance();
        cal.set(Calendar.YEAR, year);
        cal.set(Calendar.MONTH, month - 1);

        // just a simple get! but seems it is very important
        cal.get(Calendar.MONTH);
        //

        cal.set(Calendar.DAY_OF_MONTH, day);

        return cal.get(Calendar.MONTH) + 1;
    }
}

我想知道为什么会发生这种情况？

4

1 回答 1

18

问题是您从 2013 年 1 月 30 日的日历开始。

然后，您将年份设置为 2013 年 - 这不是问题。

然后，您将月份设置为 1（即二月）。你期望在这里发生什么？实际发生的是它会记住它需要将月份设置为 1，但不会重新计算实际时间值。根据文档（强调我的） ，时间值将在您调用时重新计算：get

set(f, value) 将日历字段 f 更改为值。此外，它设置一个内部成员变量来指示日历字段 f 已更改。尽管日历字段 f 立即更改，但在下一次调用 get()、getTime()、getTimeInMillis()、add() 或 roll() 之前，不会重新计算日历的时间值（以毫秒为单位）。因此，多次调用 set() 不会触发多次不必要的计算。作为使用 set() 更改日历字段的结果，其他日历字段也可能更改，具体取决于日历字段、日历字段值和日历系统。此外，在重新计算日历字段后，get(f) 不一定会返回调用 set 方法设置的值。具体由具体的日历类决定。

当您尝试将“1 月 30 日”更改为“2 月 30 日”并强制进行计算时，实际发生的情况是您在 3 月 2 日结束了我的盒子 - 但它可能在您的实现上有所不同。

最好的修复是：

于 2013-01-30T13:43:26.093 回答