Java ThreadLocal返回空值问题：高并发下NullPointerException排查

核心问题：静态ThreadLocal引用被多线程竞态覆盖 #

你代码中的threadLocalMyLogger是静态成员变量，所有线程共享同一个引用。当大量线程同时调用init()方法时，会出现竞态条件：

• 线程A执行new ThreadLocal<>()，将新实例赋值给threadLocalMyLogger
• 线程B紧接着执行new ThreadLocal<>()，直接覆盖线程A刚赋值的引用
• 线程A后续调用set()时，实际是往线程B创建的ThreadLocal实例中存储数据；而线程A调用myLogger()时，可能已经拿到被其他线程再次覆盖的ThreadLocal实例，此时get()自然返回null

简言之：静态ThreadLocal的全局引用被多线程频繁替换，导致线程自身存储的MyLogger实例与当前ThreadLocal实例不匹配，最终无法获取到数据。

正确实现方案 #

方案1：静态ThreadLocal仅初始化一次 #

将threadLocalMyLogger的初始化放在静态代码块中，确保全局只创建一个ThreadLocal实例，init()方法仅负责为当前线程设置MyLogger：

public class MyLoggerWrapper
{
    // 静态代码块初始化ThreadLocal，全局仅执行一次
    private static final ThreadLocal<MyLogger> threadLocalMyLogger = new ThreadLocal<>();
    
    public static MyLogger myLogger() 
    {
        return threadLocalMyLogger.get();
    }
    
    static void init(final SomeParameter parameter) 
    {       
        // 直接向已存在的ThreadLocal实例中设置当前线程的MyLogger
        threadLocalMyLogger.set(new MyLogger(parameter));
    }
    
    static void remove() 
    {
        threadLocalMyLogger.remove();
    }
}

这种方式下，所有线程共用同一个ThreadLocal引用，但每个线程内部存储的MyLogger实例完全独立，既满足不同任务使用不同入参的需求，也彻底避免了竞态覆盖问题。

方案2：结合ThreadLocal.withInitial()（可选） #

如果担心线程池复用线程导致残留数据，可以用withInitial()初始化默认值为null，核心逻辑和方案1一致：

public class MyLoggerWrapper
{
    private static final ThreadLocal<MyLogger> threadLocalMyLogger = ThreadLocal.withInitial(() -> null);
    
    public static MyLogger myLogger() 
    {
        return threadLocalMyLogger.get();
    }
    
    static void init(final SomeParameter parameter) 
    {       
        threadLocalMyLogger.set(new MyLogger(parameter));
    }
    
    static void remove() 
    {
        threadLocalMyLogger.remove();
    }
}

额外注意事项 #

• 线程池复用线程时，必须在finally块中调用remove()，避免线程下次执行任务时拿到上一次残留的MyLogger实例（你当前代码已经做到了这一点，是正确的）
• 多线程场景下绝对不要修改静态ThreadLocal的引用，ThreadLocal的设计初衷是让每个线程独立存储数据，而非让线程去替换全局的ThreadLocal实例

内容的提问来源于stack exchange，提问作者nearu04