Коллеги - я правильно помню что fillInStackTrace весьма дорогая операция

Question

Коллеги - я правильно помню что fillInStackTrace весьма дорогая операция

?
Нашел такой код в коммонс библиотеке проекта и смотрю на это в печали
public static Logger getLogger() {
Throwable t = new Throwable();
t.fillInStackTrace();
return getLogger(t.getStackTrace()[1].getClassName());
}

#backend #java #programming #russian

0

14.10.2021

7 ответов

17 просмотров

Artjom Kalita Автор вопроса

Oleh Oliinyk
вообще к коду есть вопросы: 1) внутри конструктор...

Ага у меня схожие вопросы возникли =)

0

14.10.2021

Aleksey Shipilev

Непонятно, зачем отдельный вызов fillInStackTrace(), когда конструктор и так его позовёт. Если нужны только верхние N фреймов, то StackWalker будет подешевле, скорее всего.

0

14.10.2021

Artjom Kalita Автор вопроса

Aleksey Shipilev
Непонятно, зачем отдельный вызов fillInStackTrace(...

Мопед не мой я обьявление только подал =) Просто бежал по коду и видел что такой подход создания логгинга реиспользуется во многих местах приложения и это меня напрягло

0

14.10.2021

Aleksey Shipilev

Artjom Kalita
Мопед не мой я обьявление только подал =) Просто б...

Ну вот fillStackTrace точно стоит снести. Может, они хотели защититься от -XX:-OmitStackTraceInThrowable, но вряд ли.

0

14.10.2021

Alexandr ∨∧‾ Emelyanov

камон, логгер объявляется статической переменной на класс во время инициализации класса....

0

14.10.2021

Aleksey Shipilev

Aleksey Shipilev
Ну вот fillStackTrace точно стоит снести. Может, о...

А впрочем, судя по коду Throwable, второй вызов ни на что сильно не повлияет, если стектрейс уже создался, а от ` -XX:-OmitStackTraceInThrowable` защитит.

0

14.10.2021

Oleh Oliinyk · Accepted Answer

вообще к коду есть вопросы: 1) внутри конструктора Throwable и так вызывается fillInStackTrace(), то есть дорогой вызов 2 раза подряд происходит 2) зачем? если это для инициализации private static Logger logger в поле класса, то еще понятно, но тогда вопрос почему вы беспокоитесь о производительности если это для того, чтобы каждый раз резолвить логгер в месте вызова, то тут как ни крути операция дорогая будет все равно

170 похожих чатов

Коллеги - я правильно помню что fillInStackTrace весьма дорогая операция

7 ответов

Похожие вопросы