Log4J - Есть ли какая-либо точка в явном указании имени класса в вызове LogManager.getLogger()?

Question

Я работаю над некоторым унаследованным кодом с большим количеством коды, как

private final Logger logger = LogManager.getLogger(ThisClassName.class); 

Я интересно, если есть какие-либо преимущества печатали «ThisClassName.class», в отличие от

LogManager.getLogger(getClass()); 

Насколько я могу судить, их нет, но мне интересно, есть ли какие-либо негативные последствия для использования getClass(). Благодарю.

Answer 1

Если вы делаете Loggerstatic, вы не можете использовать getClass().

Answer 2

getClass() не работает из статического контекста. ThisClassName.class работает для статических переменных и экземпляров.

Answer 3

Кроме того, getClass() будет скрывать фактический класс, если есть подкласс, который фактически вызывает метод. Например, предположим, что в классе A метод X вы вызываете log.debug(), а класс B расширяет класс A, переписывая метод X, но вызывающий super.X в какой-то момент. Файл журнала будет показывать класс B, а не класс А.

Answer 4

я обычно использовать версию getClass() с экземпляром-переменной Logger (вы не создания другой экземпляр регистратора, просто глядя один вверх).

Причина этого в том, что в иерархиях классов может быть полезно узнать, что представляет собой фактический тип, с которым вы имеете дело, даже если ведение журнала происходит из метода на суперклассе.

Обычно простой текстовый поиск даст вам именно то, что вызывается в журнале, так что я не нашел его запутанным на практике.

Answer 5

Другие плакаты уже прокомментировали, что getClass не будет работать, если вы хотите определить регистратор static - и определение одного для каждого экземпляра неэффективно.

Если вы хотите, чтобы правильный класс выводился во время выполнения, и вы используете хотя бы Java 5, посмотрите на log5j, который обертывает log4j в API Java 5.

Это позволяет писать такие вещи, как:

private static final Logger log = Logger.getLogger(); 

и даже:

log.debug("This thing broke: %s due to bar: %s on this thing: %s", foo, bar, car);