SimpleDateFormat 为什么不是线程安全的

怼烎@ 2022-07-15 05:04 236阅读 0赞

Why?

这是个坑，新手经常掉进去，比如我后面的妹子，如果不做压测或多线程实验根本不会看出来。

跟一下 format 方法。

private StringBuffer format(Date date, StringBuffer toAppendTo,
                                FieldDelegate delegate) {
        // Convert input date to time field list
        calendar.setTime(date);
    
        boolean useDateFormatSymbols = useDateFormatSymbols();
    
        for (int i = 0; i < compiledPattern.length; ) {
            int tag = compiledPattern[i] >>> 8;
            int count = compiledPattern[i++] & 0xff;
            if (count == 255) {
                count = compiledPattern[i++] << 16;
                count |= compiledPattern[i++];
            }
    
            switch (tag) {
            case TAG_QUOTE_ASCII_CHAR:
                toAppendTo.append((char)count);
                break;
    
            case TAG_QUOTE_CHARS:
                toAppendTo.append(compiledPattern, i, count);
                i += count;
                break;
    
            default:
                subFormat(tag, count, delegate, toAppendTo, useDateFormatSymbols);
                break;
            }
        }
        return toAppendTo;
    }

能跟到这里，calendar.setTime(date) 根据 date 得到 calendar，然后在 switch 对 toAppendTo 进行拼装最后返回格式化字符串。 其中，subFormat(…) 中 calendar 得到每一个格式化后的字符拼给 toAppendTo。  
线程A 和线程 B 几乎同时执行了 calendar.setTime(date)，此时还没有执行到 switch。后执行的会将先执行的效果抹去，两个线程使用的是同一个 calender，所以格式化的日期自然是相同的了。