静态日历变量上可能存在错误

Question

我声明了以下变量：

private static Calendar calendar = Calendar.getInstance();

我在静态方法中使用变量'日历'，如下所示：

myStaticMethod(String reqDate){
    DateFormat df = new SimpleDateFormat("yyyy-MM-dd");
    Date some_date;
    long seconds = 0;
    int value = 10;
    try {
        some_date = df.parse(reqDate);
        calendar.setTime(some_date);
        calendar.add(Calendar.DATE, value);
        Date effValueDate = calendar.getTime();
        seconds = (effValueDate .getTime() - System.currentTimeMillis()) / 1000;
    } catch (ParseException e) {
        //---Do---something----
    }
}

我得到以下声纳错误：

Make "calendar" an instance variable.
Not all classes in the standard Java library were written to be thread-safe. 
Using them in a multi-threaded manner is highly likely to cause data problems or exceptions at runtime.    
This rule raises an issue when an instance of Calendar, DateFormat, 
javax.xml.xpath.XPath, or javax.xml.validation.SchemaFactory is marked static.

这是我的解决方案，但我不确定解决方案的质量是否足够好。

myStaticMethod(String reqDate) {
    Calendar calendar = Calendar.getInstance();
    // the do the next processing
}

在这种情况下，我可以使用本地日历变量来代替类变量吗？或者有其他聪明的方法来解决这个问题。

Answer 1

由于您似乎已经接受了Gautam M的建议，使用java.time，现代Java日期和时间API，我认为作为补充，我将使用java.time.重写您的方法，我可能不需要补充说，我完全同意这项建议。

public static void myStaticMethod(String reqDate) {
    int value = 10;
    try {
        LocalDate dateThen = LocalDate.parse(reqDate).plusDays(value);
        ZonedDateTime dateTimeThen = dateThen.atStartOfDay(ZoneId.systemDefault());
        long seconds = ChronoUnit.SECONDS.between(Instant.now(), dateTimeThen);
        System.out.println(seconds);
    } catch (DateTimeParseException dtpe) {
        //---Do---something----
        System.out.println(dtpe);
    }
}

对我来说，它看起来不仅更简单，而且比使用日历的原始代码和Java1.0和1.1中其他设计不佳的类更自然。

我正在利用这样一个事实，即您的字符串格式是日期的ISO 8601格式，LocalDate解析为默认格式，也就是说，没有任何显式格式化程序。

我在我的时区（欧洲/哥本哈根）试过这个电话：

    myStaticMethod("2021-06-28");

产出为：

587995

• Oracle教程：解释如何使用java的日期时间。时间

Answer 2

sonar警告是由于可以从多个线程更新日历对象而引起的。或者，正如您所说，您可以将日历声明为该方法的本地。

或者1）在代码中创建一个同步块。（使用本地日历更好）

Date effValueDate ;
synchronized(calendar) {
    calendar.setTime(some_date);
    calendar.add(Calendar.DATE, value);
    effValueDate = calendar.getTime();
}

// or use DateTimeFormatter.ISO_LOCAL_DATE
DateTimeFormatter pattern = DateTimeFormatter.ofPattern("yyyy-MM-dd");

LocalDateTime date = LocalDate.parse(reqDate, pattern).plusDays(value).atStartOfDay();
Duration duration= Duration.between(date, LocalDateTime.now());
long seconds = duration.getSeconds();           

了解更多有关新API的信息