并发访问用于计算逾期天数的静态方法

Question

我有一个带有以下签名的静态方法:

private static volatile SimpleDateFormat payDayFormat = new SimpleDateFormat("yyyyMMdd");
public static int overdueDays(String repayDay){
    try {
        Date billDate = payDayFormat.parse(repayDay);
        Calendar startDate = Calendar.getInstance();
        startDate.setTime(billDate);
        Calendar endDate = Calendar.getInstance();
        long end = endDate.getTimeInMillis();
        long start = startDate.getTimeInMillis();
        Long days = TimeUnit.MILLISECONDS.toDays(Math.abs(end - start));
        return days.intValue();
    } catch (ParseException e) {
        logger.error("????????????");
    }
    return 0;
}

当同一个参数同时对上述方法进行多次调用时,结果不同.

56
56
1279716
56
56
56
5
736387
56
-1226645849
56

只有56才是正确答案.但是当我用Joda-Time替换Calendar 时,它会返回正确的答案.如何在不使用Joda-Time或锁定,解锁并获得正确答案的情况下更改此代码.

Answer 1

这是因为分享payDayFormat,这是SimpleDateFormat:

日期格式未同步.建议为每个线程创建单独的格式实例.如果多个线程同时访问格式,则必须在外部进行同步.

它与Jodatime一起工作的原因是它的日期格式化程序没有共享的可变状态,因为它是明智的.

所以,您可以:

• 添加synchronized到方法签名;

• synchronized在您使用的位置附近添加一个块payDayFormat:

  Date billDate;
  synchronized (payDayFormat) {
    billDate = payDayFormat.parse(repayDay);
  }
  

• SimpleDateFormat在方法内创建一个新实例;

• 创建payDayFormat一个ThreadLocal<SimpleDateFormat>,以便每个线程都有自己的副本.