Мне нужно вызвать 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(); 
    } 
}; 

---

Для моего 3-го подхода, есть ли необходимость позвонить ThreadLocal.remove?

Answer 1

Если ваше единственное намерение состоит в том, чтобы сделать это threadsafe, тогда действительно не нужно это делать. Но когда потоки поддерживаются threadpool, и ваше намерение состоит в том, чтобы дать каждому свежевыпущенному потоку из threadpool свое собственное начальное значение, тогда вы должны это сделать.

Answer 2

Как @BalusC говорит, это зависит от того, что вас беспокоит.

У меня есть подозрение, что ваша утилизация объектов Calendar с использованием локальной локальной сети может стоить больше, чем вы экономите, не звоните Calendar.getInstance(). У этого есть запах преждевременной микро-оптимизации.