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#calendarcalendar} to produce a {@code Date}. All of the {@codecalendar}'s date-time fields are {@linkplain Calendar#clear()cleared} before parsing, and the {@code calendar}'s defaultvalues of the date-time fields are used for any missingdate-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的实现无关，与你多线程使用同一个对象有关，没有做好线程同步