В моем приложении я запускаю свой код через PMD, и он показывает мне это сообщение:
- Избегайте printStackTrace (); используйте вместо этого вызов регистратора.
Что это значит?
java
logging
pmd
printstacktrace
user1305398
источник
источник
Ответы:
Это означает, что вы должны использовать структуру ведения журналов, например вход в систему или log4j и вместо прямой печати исключений:
вы должны регистрировать их, используя API этой платформы:
log.error("Ops!", e);
Фреймворки ведения журнала предоставляют вам большую гибкость, например, вы можете выбрать, хотите ли вы вести журнал в консоли или в файл - или, возможно, пропустить некоторые сообщения, если вы обнаружите, что они больше не актуальны в какой-либо среде.
источник
Если вы вызываете
printStackTrace()
исключение, трассировка записывается,System.err
и ее трудно направить в другое место (или отфильтровать). Вместо этого вам рекомендуется использовать структуру ведения журнала (или оболочку для нескольких платформ ведения журнала, например, Apache Commons Logging) и регистрировать исключение, используя эту структуру (напримерlogger.error("some exception message", e)
).Это позволит вам:
источник
Программа обеспечения качества продукции должна использовать одну из многих альтернатив журналирования (например, log4j, logback, java.util.logging) для сообщения об ошибках и других диагностических данных. Это дает ряд преимуществ:
Напротив, если вы просто используете printStackTrace, разработчик / конечный пользователь практически не имеет контроля, а сообщения журнала могут быть потеряны или показаны конечному пользователю в неподходящих обстоятельствах. (И ничто не пугает робкого пользователя больше, чем случайная трассировка стека.)
источник
В Simple e.printStackTrace () не является хорошей практикой, потому что он просто распечатывает трассировку стека до стандартной ошибки. Из-за этого вы не можете контролировать, куда идет этот вывод.
источник
Практически каждая среда ведения журнала предоставляет метод, с помощью которого мы можем передать бросаемый объект вместе с сообщением. Подобно:
public trace(Marker marker, String msg, Throwable t);
Они печатают трассировку стека бросаемого объекта.
источник
Давайте поговорим из концепции компании. Журнал предоставляет гибкие уровни (см. Разницу между logger.info и logger.debug ). Разные люди хотят видеть разные уровни, например, тестировщики, разработчики, бизнесмены. Но e.printStackTrace () все распечатает. Также, как если бы этот метод был вызван restful, эта же ошибка может выводиться несколько раз. Тогда сотрудники Devops или Tech-Ops в вашей компании могут быть сумасшедшими, потому что они будут получать одинаковые напоминания об ошибках. Я думаю, что может быть лучше замена.
log.error("errors happend in XXX", e)
Это также распечатает всю информацию, которую легче читать, чем e.printStackTrace ()источник
Основная причина в том, что Proguard удалит вызовы журнала из производства. Потому что, зарегистрировав или распечатав StackTrace, их можно увидеть (информацию внутри трассировки стека или журнал) внутри телефона Android, например, с помощью приложения Logcat Reader. Так что это плохая практика для безопасности. Кроме того, мы не получаем к ним доступ во время производства, их лучше снять с производства. Поскольку ProGuard удаляет все вызовы журнала, а не stackTrace, поэтому лучше использовать блоки перехвата входа в систему и позволить им удалить их из производства с помощью Proguard.
источник