在以下情况下是否需要调用 ThreadLocal.remove

Question

而不是写下面的非线程安全方法。

private static final Calendar calendar = Calendar.getInstance();
public void fun() {
    // Going to call mutable methods in calendar.
}

---

我将其更改为线程安全版本。

public void fun() {
    final Calendar calendar = Calendar.getInstance();
    // Going to call mutable methods in calendar.
}

---

我没有每次都为同一个线程创建一个新实例，而是通过

public void fun() {
    final Calendar calendar = getCalendar();
    // Going to call mutable methods in calendar.
}

/**
 * Returns thread safe calendar.
 * @return thread safe calendar
 */
public Calendar getCalendar() {
    return calendar.get();
}

private static final ThreadLocal <Calendar> calendar = new ThreadLocal <Calendar>() {
    @Override protected Calendar initialValue() {
        return Calendar.getInstance();
     }
 };

---

对于我的第三种方法，是否需要致电ThreadLocal.remove？

Answer 1

如果您的唯一目的是使其线程安全，那么确实没有必要这样做。但是当线程由线程池维护并且您的意图更多是为线程池中每个新释放的线程赋予其自己的初始值时，那么您应该这样做。

Answer 2

正如@BalusC 所说，这取决于您关心的问题。

我怀疑您使用本地线程回收Calendar 对象实际上可能比不调用Calendar.getInstance() 节省的成本更多。这有点过早的微优化的味道。