我们团队的政策是像这样进行日志记录：
if (LOGGER.isDebugEnabled()) {
LOGGER.debug("model[" + model + "]");
}
不是像这样简单地调用日志记录方法：
LOGGER.debug("model[" + model + "]");
这种做法能够带来一些性能改进，但另一方面它也使代码库更加复杂。 我们的应用程序没有性能问题，而且很可能永远不会出现，引入该策略的论点仅仅是因为它是一个好的实践，所以每次进行日志记录时都应该使用它。
您认为这是一个好的策略吗？
你应该使用SLF4J并将log4j作为其实现。通过SLF4J，您可以通过使用带参数的消息完全消除isDebugEnabled()。
以下两行代码将生成完全相同的输出。但是，在禁用日志记录语句的情况下，第二种形式的性能将至少比第一种形式提高30倍。
logger.debug("The new entry is " + entry + ".");
logger.debug("The new entry is {}.", entry);
对于那些构建日志字符串会显著影响性能的情况，这是一个很好的策略。原因可能有两个:
这应该不是一个笼统的规则，我希望这两种情况都是相对罕见的。
当然，我们真正想要的是能够说："调用debug方法，传入一个参数，该参数不是调试字符串本身，而是知道如何构建调试字符串的东西，仅在必要时才使用。"这在Java中可能比较丑陋，没有简洁的闭包。(即使在支持lambda表达式形式的闭包的语言中，根据语言处理捕获变量的方式，在某些情况下，捕获相关变量可能是重要的。)
getEntryDebugString()。最好多写几行容易出错的代码，而不是因为罕见的问题而感到遗憾。 - xmedeko我同意迈克尔·A·杰克逊的名言：
程序优化的第一法则：不要去做。
程序优化的第二法则——只有专家才需要考虑的：暂时不要优化。
在大多数情况下，我认为如果不能确定性能增益是显著的，让代码库变得更复杂是不值得的。
我知道这会提高性能，但我认为是否值得在特定情况下添加额外的代码行应该由个人程序员负责。在大多数情况下，额外的代码行只会增加复杂性，在性能方面没有任何显著的收益。
我认为在编写日志消息之前调用isDebugEnabled()函数是有意义的。 这样你可以轻松更新你的日志消息。 我们使用以下代码使日志消息更加灵活。
if (LOGGER.isDebugEnabled()) {
String msg = "A Message {0} you can put {1} differet Objects in {2}"; //$NON-NLS-1$
Object[] args = new Object[] {file1.toString(),new Integer(23),anObject};
LOGGER.debug(MessageFormat.format(msg,args));
}
@Test
void loggerTest() {
System.out.println("START " + LocalDateTime.now());
System.out.println("TRACE: " + LocalDateTime.now());
log.trace("{}", sleepAndReturnString(5000L, "TRACE PRINT"));
System.out.println("DEBUG: " + LocalDateTime.now());
log.debug("{}", sleepAndReturnString(5000L, "DEBUG PRINT"));
System.out.println("INFO: " + LocalDateTime.now());
log.info("{}", sleepAndReturnString(5000L, "INFO PRINT"));
System.out.println("WARN: " + LocalDateTime.now());
log.warn("{}", sleepAndReturnString(5000L, "WARN PRINT"));
System.out.println("ERROR: " + LocalDateTime.now());
log.error("{}", sleepAndReturnString(5000L, "ERROR PRINT"));
System.out.println("END " + LocalDateTime.now());
}
public static String sleepAndReturnString(long millis, String message) {
try {
Thread.sleep(millis);
} catch (InterruptedException e) {
log.error("", e);
}
return message;
}
logback.xml 文件：
<configuration>
<property name="MY__PATTERN"
value="%d{yyyy-MM-dd HH:mm:ss.SSS} | %highlight(%-5level) | %message%n%rEx"/>
<appender name="CONSOLE" class="ch.qos.logback.core.ConsoleAppender">
<encoder class="ch.qos.logback.classic.encoder.PatternLayoutEncoder">
<outputPatternAsHeader>true</outputPatternAsHeader>
<pattern>${MY__PATTERN}</pattern>
</encoder>
</appender>
<root level="info">
<appender-ref ref="CONSOLE"/>
</root>
START 2023-01-20T11:17:57.255
TRACE: 2023-01-20T11:17:57.255
DEBUG: 2023-01-20T11:18:02.256
INFO: 2023-01-20T11:18:07.262
2023-01-20 11:18:12.267 | INFO | INFO PRINT
WARN: 2023-01-20T11:18:12.283
2023-01-20 11:18:17.285 | WARN | WARN PRINT
ERROR: 2023-01-20T11:18:17.285
2023-01-20 11:18:22.287 | ERROR | ERROR PRINT
END 2023-01-20T11:18:22.287
正如我们所看到的，每次线程休眠5秒钟，尽管TRACE和DEBUG消息没有打印到控制台。这表明使用isLevelEnabled是合理的，并且可以提高性能。
LOGGER.debug("model[" + proxy.getModel() + "]");
entry是昂贵的时候，字符串格式化无法提供帮助。例如，考虑一些类似于logger.debug("The new entry is {}.", getEntry());的代码，你不知道getEntry()方法需要花费多少成本。 - xmedeko