我在我的应用程序中使用Log4J进行日志记录。之前我使用的调试调用如下:
选项1:
logger.debug("some debug text");
但是一些链接建议最好先检查isDebugEnabled()，比如:
选项2:
boolean debugEnabled = logger.isDebugEnabled();
if (debugEnabled) {
logger.debug("some debug text");
}
所以我的问题是“选项2是否能提高性能?”
因为在任何情况下Log4J框架对debugEnabled都有相同的检查。对于选项2，如果我们在一个方法或类中使用多个调试语句可能是有益的，这样框架就不需要多次调用isDebugEnabled()方法(每次调用);在这种情况下，它只调用isDebugEnabled()方法一次，如果Log4J被配置为调试级别，那么实际上它会调用isDebugEnabled()方法两次:
如果将值分配给debugEnabled变量，和
实际上由logger.debug()方法调用。
我不认为如果我们在方法或类中编写多个logger.debug()语句并根据选项1调用debug()方法，那么与选项2相比，Log4J框架的开销是很大的。由于isDebugEnabled()是一个非常小的方法(就代码而言)，因此它可能是内联的良好候选方法。
使用isDebugEnabled()是为当你通过连接字符串来构建日志消息保留的:
Var myVar = new MyVar();
log.debug("My var is " + myVar + ", value:" + myVar.someCall());
然而，在您的示例中，没有速度增益，因为您只是记录一个字符串，而不执行连接等操作。因此，你只是在给你的代码增加膨胀，让它更难阅读。
我个人在String类中使用Java 1.5格式调用，如下所示:
Var myVar = new MyVar();
log.debug(String.format("My var is '%s', value: '%s'", myVar, myVar.someCall()));
我怀疑是否有很多优化，但它更容易阅读。
不过请注意，大多数日志api都提供了这样开箱即用的格式:例如，slf4j提供了以下格式:
logger.debug("My var is {}", myVar);
这样就更容易阅读了。
比如@erickson，这要看情况。如果我没记错的话，isDebugEnabled已经在Log4j的debug()方法中构建了。
在我看来，只要你不在调试语句中做一些昂贵的计算，比如在对象上循环，执行计算和连接字符串，你就可以了。
StringBuilder buffer = new StringBuilder();
for(Object o : myHugeCollection){
buffer.append(o.getName()).append(":");
buffer.append(o.getResultFromExpensiveComputation()).append(",");
}
log.debug(buffer.toString());
会更好
if (log.isDebugEnabled(){
StringBuilder buffer = new StringBuilder();
for(Object o : myHugeCollection){
buffer.append(o.getName()).append(":");
buffer.append(o.getResultFromExpensiveComputation()).append(",");
}
log.debug(buffer.toString());
}
