在这种特殊情况下，选择1更好。

当涉及调用各种对象的toString()方法并连接结果时，保护语句(检查isDebugEnabled())是为了防止日志消息潜在的昂贵计算。

在给定的示例中，日志消息是一个常量字符串，因此让记录器丢弃它与检查记录器是否启用一样有效，并且由于分支较少，因此降低了代码的复杂性。

更好的方法是使用最新的日志记录框架，其中日志语句接受一个格式规范和一个参数列表，由日志记录器代替——但是“惰性地”，只有在启用了日志记录器的情况下。这是slf4j采用的方法。

有关更多信息，请参阅我对相关问题的回答，以及使用log4j执行类似操作的示例。

它提高了速度，因为在调试文本中连接字符串是很常见的，这是昂贵的，例如:

boolean debugEnabled = logger.isDebugEnabled();
if (debugEnabled) {
    logger.debug("some debug text" + someState);
}

选择2更好。

它本身并不能提高性能。但它确保性能不会下降。这是如何。

通常我们认为 logger.debug (someString);

但通常，随着应用程序的发展，会有很多人转手，尤其是新手开发人员

logger.debug（str1 + str2 + str3 + str4）;

诸如此类。

即使日志级别设置为ERROR或FATAL，字符串的连接也会发生! 如果应用程序包含大量带有字符串连接的DEBUG级别消息，那么它肯定会受到性能影响，特别是使用jdk 1.4或以下版本时。(我不确定以后版本的jdk内部是否做任何stringbuffer.append())。

这就是为什么选择2是安全的。即使字符串连接也不会发生。

Since in option 1 the message string is a constant, there is absolutely no gain in wrapping the logging statement with a condition, on the contrary, if the log statement is debug enabled, you will be evaluating twice, once in the isDebugEnabled() method and once in debug() method. The cost of invoking isDebugEnabled() is in the order of 5 to 30 nanoseconds which should be negligible for most practical purposes. Thus, option 2 is not desirable because it pollutes your code and provides no other gain.

正如其他人提到的，只有在创建字符串是一个耗时的调用时，使用guard语句才真正有用。具体的例子是当创建字符串时会触发一些延迟加载。

值得注意的是，这个问题可以通过使用Java的简单日志Facade或(SLF4J) - http://www.slf4j.org/manual.html来完成避免。这允许调用如下方法:

logger.debug("Temperature set to {}. Old temperature was {}.", t, oldT);

这只会在启用调试时将传入的参数转换为字符串。SLF4J顾名思义只是一个facade，日志调用可以传递给log4j。

你也可以很容易地“滚动”你自己的版本。

希望这能有所帮助。

在Java 8中，您不必使用isDebugEnabled()来提高性能。

https://logging.apache.org/log4j/2.0/manual/api.html#Java_8_lambda_support_for_lazy_logging

import java.util.logging.Logger;
...
Logger.getLogger("hello").info(() -> "Hello " + name);