在自定义BusinessThreadSession类中，每次调用ThreadLocal的set前需执行remove吗？

不需要在调用setCurrentRedisClusterCode方法执行set操作前调用remove，原因如下：

• ThreadLocal的set方法本身会直接覆盖当前线程绑定的旧值，无论之前是否存在值，新值都会替换掉旧值，所以前置remove属于冗余操作。
• 需要注意线程池复用场景的潜在问题：如果线程被线程池回收复用，之前绑定在ThreadLocal上的值如果没有被清理，会被后续复用该线程的任务读取到，导致业务逻辑出错；同时还可能引发内存泄漏（ThreadLocalMap的Entry中key是弱引用，value是强引用，若不主动remove，value会一直被线程持有无法回收）。

正确的清理时机不是set之前，而是当前线程的业务逻辑执行完毕后，主动调用remove清理ThreadLocal绑定的值。针对你的代码，可以新增一个清理方法：

public final class BusinessThreadSession {
    private static ThreadLocal<String> currentRedisClusterCode = new ThreadLocal<>();

    private BusinessThreadSession() {}

    public static void setCurrentRedisClusterCode(String redisClusterCode) {
        currentRedisClusterCode.set(redisClusterCode);
    }

    public static String getCurrentRedisClusterCode() {
        return currentRedisClusterCode.get();
    }

    // 建议在业务逻辑执行完成后调用此方法清理
    public static void clearCurrentRedisClusterCode() {
        currentRedisClusterCode.remove();
    }
}

比如在业务代码中，可以通过try-finally块确保清理逻辑执行：

try {
    BusinessThreadSession.setCurrentRedisClusterCode("cluster1");
    // 执行核心业务逻辑
} finally {
    BusinessThreadSession.clearCurrentRedisClusterCode();
}

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