ThreadLocal存List遇到的坑

之前为了方便一个Service下共用List，将List作为类成员变量，当然有点线程安全意识的兄弟们都知道不能就这么直接加在上面。

public class PermService {
    private List<String> list;
    ....    
}

当然用锁(synchronized)又还不至于，于是这里用了线程本地变量ThreadLocal，如下:

public class PermService {

    private static final ThreadLocal<List<String>> STR_LIST = new ThreadLocal<List<String>>() {
        @Override
        protected List<String> initialValue() {
            return new ArrayList<String>();
        }
    };
    ....  
}

然后我每次请求进来之后都是类似于这么处理的

public void storePermsByRole(String roleId) {
　　List<String> permIds = rolePermMapper.selectPermByRole(roleId);
　　STR_LIST.get().addAll(permIds);
}

后面用postman测的时候问题来了：原本和这个roleId没有关系的permId在返回体里面出现了，查了下库里面也没有这两个关系的记录，debug断点打到上面的第3行permIds里面也没有那一条数据，那么这多出来的一条是哪来的呢？

 

苦思冥想意识到可能和线程池有关系(这里说的线程池不是你配的全局线程池，也不是手动用Executors生成的线程池)，所有线程池一般有个核心线程数，这些线程是不会在一个请求request结束后就销毁的，而线程本地变量又是依附于线程的(具体可以理解下ThreadLocalMap)，所以一个线程上一个请求将数据库中查出来的list加到线程本地变量后，下一个请求进来原来的list还在里面，从而导致了上面的问题。

这里解决方法是在service中加一段清空ThreadLocal的逻辑，在请求结束之前调一下即可。

public void clearThreadLocal() {
　　STR_LIST.remove();
}

 

ThreadLocal用的守则：

1、请求开始时set

2、请求结束前remove

3、加static关键字