java.util.Calendar混淆

时间:2012-12-15 19:10:02

标签: java internationalization calendar locale cloudbees

我在使用java.util.Calendar和commons-lang DateUtil时遇到问题 问题是我的测试在本地计算机上正常运行并且在CloudBees上失败。似乎区域设置存在问题,但我不确定。

以下是代码:

import org.testng.Assert;
import org.testng.annotations.BeforeClass;
import org.testng.annotations.Test;

//bla-bla-bla

public static final String TEST_DATE_AS_STRING = "13 10 2012 20:50:44";
public static final int MILLIS_IN_HOUR = 3600000;
private static final String LEAP_WEEK_DATE_AS_STRING = "31 10 2012 20:50:44";

private final SimpleDateFormat sdf = new SimpleDateFormat("dd MM yyyy HH:mm:ss");

@Test
public void getWeekDatePair() throws ParseException{
    Date date = sdf.parse(TEST_DATE_AS_STRING);
    DatePair dp = Util.DateTime.getWeekDatePair(date);
    Assert.assertEquals(sdf.format(new Date(dp.getStart())), "08 10 2012 00:00:00");
    //java.lang.AssertionError: expected [14 10 2012 00:00:00] but found [07 10 2012 00:00:00]
    Assert.assertEquals(sdf.format(new Date(dp.getEnd())), "14 10 2012 00:00:00");
}

@Test
public void getLeapWeekDatePair() throws ParseException {
    Date leapDate = sdf.parse(LEAP_WEEK_DATE_AS_STRING);
    DatePair dp  = Util.DateTime.getWeekDatePair(leapDate);
    Assert.assertEquals(sdf.format(new Date(dp.getStart())), "29 10 2012 00:00:00");

    //java.lang.AssertionError: expected [04 11 2012 00:00:00] but found [28 10 2012 00:00:00]
    Assert.assertEquals(sdf.format(new Date(dp.getEnd())), "04 11 2012 00:00:00");
}

这是测试输出失败:

 java.lang.AssertionError: expected [04 11 2012 00:00:00] but found [28 10 2012 00:00:00]
    at org.testng.Assert.fail(Assert.java:94)
    at org.testng.Assert.failNotEquals(Assert.java:494)
    at org.testng.Assert.assertEquals(Assert.java:123)
    at org.testng.Assert.assertEquals(Assert.java:176)
    at org.testng.Assert.assertEquals(Assert.java:186)
    at ru.rating.utils.UtilDateTimeTest.getLeapWeekDatePair(UtilDateTimeTest.java:77)

expected [14 10 2012 00:00:00] but found [07 10 2012 00:00:00]
Stacktrace

java.lang.AssertionError: expected [14 10 2012 00:00:00] but found [07 10 2012 00:00:00]
at org.testng.Assert.fail(Assert.java:94)
at org.testng.Assert.failNotEquals(Assert.java:494)
at org.testng.Assert.assertEquals(Assert.java:123)
at org.testng.Assert.assertEquals(Assert.java:176)
at org.testng.Assert.assertEquals(Assert.java:186)
at ru.rating.utils.UtilDateTimeTest.getWeekDatePair(UtilDateTimeTest.java:69)

这是实施:

    public static DatePair getWeekDatePair(){
        return getWeekDatePair(new Date());
    }


/**
 * This is test method
 * */
        static DatePair getWeekDatePair( Date date){
            Date truncDay = truncate(date.getTime(), Calendar.DAY_OF_MONTH);
            Calendar calStart = getCalendarInstance(date, Calendar.DAY_OF_MONTH);
            calStart.setTime(truncDay);
            calStart.set(Calendar.DAY_OF_WEEK, Calendar.MONDAY);        

            Calendar calEnd = Calendar.getInstance();
            calEnd.setTime(calStart.getTime());
            calEnd.set(Calendar.DAY_OF_WEEK, Calendar.SUNDAY);

            return new DatePair(calStart.getTime(), calEnd.getTime());
        }
    public static Date truncate(long date, int calField) {
        Calendar cal = getCalendarInstance(new Date(date), calField);
    cal = DateUtils.truncate(cal, calField);
        return cal.getTime();
    }

static Calendar getCalendarInstance(Date date, int calendarField){
    //Calendar cal = Calendar.getInstance();
    Calendar cal = new GregorianCalendar(Locale.ENGLISH);

    cal.setTime(date);
    if(calendarField!=Calendar.HOUR){
        cal.set(Calendar.HOUR_OF_DAY, 0);
    }
    cal.set(Calendar.MINUTE, 0);
    cal.set(Calendar.SECOND, 0);
    cal.set(Calendar.MILLISECOND, 0);
    return cal;
}

1 个答案:

答案 0 :(得分:4)

虽然我们在这里缺少关键信息(Util.DateTime.getWeekDatePair(java.util.Date)如何实现),但我怀疑你在那里做的是使用默认java.util.Calendar实例化Locale然后搜索第一天本周。

我怀疑你没有将Locale实例传递给getWeekDatePair()方法。

现在,这里有什么问题?那么,一周的第一天取决于Locale。因此,当您像Calendar那样实例化Calendar.getInstance()时,您实际上做的是:Calendar.getInstance(Locale.getDefault(Locale.Category.FORMAT)。当然,一周的第一天可能在两台不同的机器上有所不同,因为Locales 可能不同。
例如,一周的第一天是美国的星期日,但波兰的星期一(我相信它就像在俄罗斯一样,不是吗?)因此,如果你在两台不同的机器上进行这项测试,其中第一个就是Locale set对于en-US和第二个ru-RU,你可能期望不同的结果。

如果只是测试的问题,你也可以设置默认的Locale,一切都应该正常工作。但是,请记住,如果您正在测试Web应用程序,使用默认Locale是一件坏事,因为它将返回服务器Locale而不是来自Web浏览器(或某些用户配置文件,如果您有)。如果此区域设置不同,您可能会使用令最终用户感到困惑的内容。


修改

很明显为什么它会从实现中发生,我之前给了你提示。请考虑以下代码:

Calendar cal = Calendar.getInstance(Locale.ENGLISH);
System.out.println(sdf.format(cal.getTime()));
// Cloning so that Calendar could be re-used
Calendar calEnd = (Calendar) cal.clone();
// Setting start of the week date
cal.set(Calendar.DAY_OF_WEEK, Calendar.MONDAY);
System.out.println(sdf.format(cal.getTime()));
calEnd.set(Calendar.DAY_OF_WEEK, Calendar.SUNDAY);
System.out.println(sdf.format(calEnd.getTime()));

这打印(正确):

  

13 10 2012 00:00:00
  08 10 2012 00:00:00
  07 10 2012 00:00:00

现在,让我们将Calendar实例化更改为:

Calendar cal = Calendar.getInstance(Locale.forLanguageTag("ru-RU"));

Voila,现在你会看到:

  

13 10 2012 00:00:00
  08 10 2012 00:00:00
  14 10 2012 00:00:00

要了解为什么这是正确的行为,让我们测试一下这段代码:

System.out.println(cal.getFirstDayOfWeek());

对于英语区域设置,它将返回1(星期日),而不是2(星期一),这是俄语语言环境的结果。此代码行为正确,因为它从给定周返回周一和周日。唯一的“问题”是这个事实,那个星期意味着世界各地的其他东西。

正如您所看到的,它与DateUtils无关,它只与Calendar的行为有关。而且由于这段代码:Calendar cal = new GregorianCalendar(Locale.ENGLISH);行为实际上应该是一致的,所以无论你在哪台机器上测试代码,都应该总是出错。它不是,我真的不明白为什么。

根据您要实现的目标,将Locale作为参数添加到您的方法(并相应地实例化Calendar)可能是有意义的,然后编写一些涵盖少数Locales的测试(一些阿拉伯语Locale也可能很有趣,因为没有人说一周的第一天必须是周日或周一)仅修改测试日期,以便它们与正确的日历行为相匹配。