StringBuilder 为什么线程不安全？

一、引言

面试时被面试官问道这个问题：

面试官：StringBuilder和StringBuffer的区别在哪？
我：StringBuilder不是线程安全的，StringBuffer是线程安全的
面试官：那StringBuilder不安全的点在哪儿？
我：。。。（哑巴了）

二、分析

在分析设个问题之前我们要知道StringBuilder和StringBuffer的内部实现跟String类一样，都是通过一个char数组存储字符串的，不同的是String类里面的char数组是final修饰的，是不可变的，而StringBuilder和StringBuffer的char数组是可变的。

首先通过一段代码去看一下多线程操作StringBuilder对象会出现什么问题

public static void main(String[] args) throws InterruptedException {
       final StringBuilder builder = new StringBuilder();
       for (int i=0;i<10;i++){
           new Thread(new Runnable() {
               @Override
               public void run() {
                   for (int j=0;j<1000;j++){
                       builder.append("a");
                   }
               }
           }).start();
       }
       Thread.sleep(100);
        System.out.println(builder.length());
    }

输出结果：9943

我们能看到这段代码创建了10个线程，每个线程循环1000次往StringBuilder对象里面append字符。正常情况下代码应该输出10000，但是实际运行会输出什么呢？

为什么输出值跟预期值不一样

我们先看一下StringBuilder的两个成员变量（这两个成员变量实际上是定义在AbstractStringBuilder里面的，StringBuilder和StringBuffer都继承了AbstractStringBuilder）

//存储字符串的具体内容
char[] value;
//已经使用的字符数组的数量
int count;

再看StringBuilder的append()方法：

@Override
public StringBuilder append(String str) {
    super.append(str);
    return this;
}

StringBuilder的append()方法调用的父类AbstractStringBuilder的append()方法

1.public AbstractStringBuilder append(String str) {
2.    if (str == null)
3.        return appendNull();
4.    int len = str.length();
5.    ensureCapacityInternal(count + len);
6.    str.getChars(0, len, value, count);
7.    count += len;
8.    return this;
9.}

我们先不管代码的第五行和第六行干了什么，直接看第七行，count += len不是一个原子操作。假设这个时候count值为10，len值为1，两个线程同时执行到了第七行，拿到的count值都是10，执行完加法运算后将结果赋值给count，所以两个线程执行完后count值为11，而不是12。这就是为什么测试代码输出的值要比10000小的原因。