Java SImpleDateFormat线程安全问题

Question

public class Test {    private static SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd");    private static class Task implements Runnable {        public void run() {            try {                System.out.println(sdf.parse("2016-03-21 12:00:00").getTime());            } catch (ParseException e) {                e.printStackTrace();            }        }    }    public static void main(String[] args) {        Task task = new Task();        Thread t1 = new Thread(task);        t1.start();        Thread t2 = new Thread(task);        t2.start();        Thread t3 = new Thread(task);        t3.start();        Thread t4 = new Thread(task);        t4.start();        Thread t5 = new Thread(task);        t5.start();    }}最近才知道，如果像上面代码那样通过定义静态共享的实例来使用SimpleDateFormat是会存在线程安全问题的。例如线程被挂死：或者转换的时间不对：然后自己尝试读JDK源码来试图分析在SimpleDateFormat的代码中，究竟是哪里的函数或者代码会引起线程不安全，但是由于水平有限，最后还是找不出。哪位大牛可以帮忙分析一下。PS:我的jdk版本：1.8.0_73

Answer 1

我看了一下parse()源代码的注释，有一段是这位的

This parsing operation uses the {@link DateFormat#calendar

• calendar} to produce a {@code Date}. All of the {@code

• calendar}'s date-time fields are {@linkplain Calendar#clear()

• cleared} before parsing, and the {@code calendar}'s default

• values of the date-time fields are used for any missing

• date-time information.

大概意思就是parse()方法使用calendar来生成返回的Date实例，问题是这里它用到的calendar不是每次方法执行时都新生成一个，它是定义了一个"protected Calendar calendar"属性，按注释的意思，每次parse之前，都会把calendar里的相关属性都清除，然后用新的处理结果填充此对象，这样的话在多线程里边，parse()执行过程中，就会出现一个线程刚把calendar设置好之后，另一个线程把它给清空了，因为都是一个sdf实例，其他问题原因应该都是这个

Answer 2

private StringBuffer format(Date date, StringBuffer toAppendTo,

                                FieldDelegate delegate) {

        // Convert input date to time field list

        calendar.setTime(date);

        boolean useDateFormatSymbols = useDateFormatSymbols();

        

        ...

 }  

这里使用的calendar是类成员，多线程去setTime会有线程安全问题

Answer 3

这跟SimpleDateFormat的实现无关，与你多线程使用同一个对象有关，没有做好线程同步