SimpleDateFormat线程安全问题

一、背景

项目上线前QA进行压测，出现SimpleDateFormat线程安全问题，异常如下

Java代码  

1. Exception in thread "Thread-1" java.lang.NumberFormatException: multiple points  
2.     at sun.misc.FloatingDecimal.readJavaFormatString(FloatingDecimal.java:1082)  
3.     at java.lang.Double.parseDouble(Double.java:510)  
4.     at java.text.DigitList.getDouble(DigitList.java:151)  
5.     at java.text.DecimalFormat.parse(DecimalFormat.java:1302)  
6.     at java.text.SimpleDateFormat.subParse(SimpleDateFormat.java:1589)  
7.     at java.text.SimpleDateFormat.parse(SimpleDateFormat.java:1311)  
8.     at java.text.DateFormat.parse(DateFormat.java:335)  
9.     at com.peidasoft.orm.dateformat.DateNoStaticUtil.parse(DateNoStaticUtil.java:17)  
10.     at com.peidasoft.orm.dateformat.DateUtilTest$TestSimpleDateFormatThreadSafe.run(DateUtilTest.java:20)  
11. Exception in thread "Thread-0" java.lang.NumberFormatException: multiple points  
12.     at sun.misc.FloatingDecimal.readJavaFormatString(FloatingDecimal.java:1082)  
13.     at java.lang.Double.parseDouble(Double.java:510)  
14.     at java.text.DigitList.getDouble(DigitList.java:151)  
15.     at java.text.DecimalFormat.parse(DecimalFormat.java:1302)  
16.     at java.text.SimpleDateFormat.subParse(SimpleDateFormat.java:1589)  
17.     at java.text.SimpleDateFormat.parse(SimpleDateFormat.java:1311)  
18.     at java.text.DateFormat.parse(DateFormat.java:335)  
19.     at com.peidasoft.orm.dateformat.DateNoStaticUtil.parse(DateNoStaticUtil.java:17)  
20.     at com.peidasoft.orm.dateformat.DateUtilTest$TestSimpleDateFormatThreadSafe.run(DateUtilTest.java:20)  

二、问题分析

SimpleDateFormat继承自DateFormat，而在最常用的parse()和format()方法的内部，会调用父类DateFormat内部的calendar，

如parse()方法内部有一段这样的代码

Java代码  

1. start = subParse(text, start, tag, count, obeyCount,  
2.                 ambiguousYear, pos,  
3.                 useFollowingMinusSignAsDelimiter, calb);  

其内部实现比较复杂，但很明显它调用了父类DateFormat内部的calendar对象，那就大概理解到问题了，当我们的SimpleDateFormat对象声明为static的时候，在多线程并发的时候就会共享这个format对象，也就共用了calendar这个对象。

Java代码  

1. public static final SimpleDateFormat format = new SimpleDateFormat("yyyy-MM-dd");  

再继续下面代码中调用了CalendarBuilder类的establish的方法

Java代码  

1. parsedDate = calb.establish(calendar).getTime();  
2.            // If the year value is ambiguous,  
3.            // then the two-digit year == the default start year  
4.            if (ambiguousYear[0]) {  
5.                if (parsedDate.before(defaultCenturyStart)) {  
6.                    parsedDate = calb.addYear(100).establish(calendar).getTime();  
7.                }  
8.            }  

可以看到其内部调用了坑爹的clear()重置了一下

Java代码  

1. Calendar establish(Calendar cal) {  
2.        boolean weekDate = isSet(WEEK_YEAR)  
3.                            && field[WEEK_YEAR] > field[YEAR];  
4.        if (weekDate && !cal.isWeekDateSupported()) {  
5.            // Use YEAR instead  
6.            if (!isSet(YEAR)) {  
7.                set(YEAR, field[MAX_FIELD + WEEK_YEAR]);  
8.            }  
9.            weekDate = false;  
10.        }  
11.   
12.        cal.clear();  
13.        // Set the fields from the min stamp to the max stamp so that  
14.        // the field resolution works in the Calendar.  
15.        for (int stamp = MINIMUM_USER_STAMP; stamp < nextStamp; stamp++) {  
16.            for (int index = 0; index <= maxFieldIndex; index++) {  
17.                if (field[index] == stamp) {  
18.                    cal.set(index, field[MAX_FIELD + index]);  
19.                    break;  
20.                }  
21.            }  
22.        }  

于是问题出现了，有的线程clear()，有的线程getTime()，这不是搞事情嘛！

三、解决方案

其实也算不上什么解决方案了，既然问题定位到了，那解决办法自然简单，每个线程使用的对象隔离开就搞定了。

1、每个线程创建一个新的SimpleDateFormat对象，也是最简单的，但这样每次都要new对象，让人感觉很不爽

 

Java代码  

1. /** 
2.  * 字符串转化为Date 
3.  * @param date 
4.  * @return 
5.  * @throws ParseException 
6.  */  
7. public static Date parse(String date) throws ParseException {  
8.     SimpleDateFormat format = new SimpleDateFormat("yyyy-MM-dd");  
9.     return format.parse(date);  
10. }  

2、使用ThreadLocal为每个线程创建一个副本使其隔离，简单的来个饿汉方式搞定吧(毕竟我不是处女座)

Java代码  

1. /** 
2.  * API的格式 
3.  */  
4. private static ThreadLocal<SimpleDateFormat> yyyymmdd = new ThreadLocal<SimpleDateFormat>(){  
5.     @Override  
6.     protected SimpleDateFormat initialValue() {  
7.         return new SimpleDateFormat("yyyy-MM-dd");  
8.     }  
9. };