什么情况需要 if (log.isDebugEnabled()) {}

在使用log4j,common-log这样的log框架时，发现很多代码中这样写
if   (log.isDebugEnabled())   {
          log.debug( "xxxx ");
}

我很奇怪，为什么要与log.isDebugEnabled()？既然log.debug()在没有指定输出级别为DEBUG时不会有输出，为什么还要在前面加一个isDebugEnabled()的判断？

为了效率，如果上述那个代码那么简单是没有必要的 

但是如果这样 
if   (log.isDebugEnabled())   { 
          log.debug(buildFullString()); 
} 

如果这个buildFullString效率不太高，那么如果直接写log.debug(buildFullString());的话 

虽然它不会打印语句，但是buildFullString还是被执行了，这样就白费了功夫 

所以加上isDebugEnabled就可以避免执行buildFullString了 

这个方法一般用在认为buildFullString这个函数效率不太高的情况下

 

加不加if，效果相同，但是效率不同 

在你的例子里，debug的参数就是一个string，所以没有太本质的差别 

但是假如log.debug( "map= "   +   map)的话，在调用debug之前，必然需要调用map.toString()，而这个操作可能导致更多的大量字符串操作，最后，才能得到这个辛辛苦苦拼接出来的结果，作为参数传入debug()，结果，这个参数却不需要（一般在log方法入口就判断输出level）。

  

评论

4 楼 梦回秘蓝 2015-04-15  

whichisnotuse 写道

非也， 除非需要很执行一段代码为log准备数据，否则不需要对log级别进行判断。因为在log4j里已经先进行判断了。
源码如下：

  public void debug(Object message)
  {
    if (this.repository.isDisabled(10000))
      return;
    if (Level.DEBUG.isGreaterOrEqual(getEffectiveLevel()))
      forcedLog(FQCN, Level.DEBUG, message, null);
  }

判断日志级别源码log.isDebugEnabled()的源码如下：
  public boolean isDebugEnabled()
  {
    if (this.repository.isDisabled(10000))
      return false;
    return Level.DEBUG.isGreaterOrEqual(getEffectiveLevel());
  }

public void debug(Object message) 就是要节省参数message生成的时间

3 楼 helloint 2010-12-23  

不错,明白了

2 楼 kaowww153 2010-12-07  

非毛也
楼主写的很有道理
log4j判断了也无法阻止map.toString()的执行。

1 楼 whichisnotuse 2010-10-28  

非也， 除非需要很执行一段代码为log准备数据，否则不需要对log级别进行判断。因为在log4j里已经先进行判断了。
源码如下：

  public void debug(Object message)
  {
    if (this.repository.isDisabled(10000))
      return;
    if (Level.DEBUG.isGreaterOrEqual(getEffectiveLevel()))
      forcedLog(FQCN, Level.DEBUG, message, null);
  }

判断日志级别源码log.isDebugEnabled()的源码如下：
  public boolean isDebugEnabled()
  {
    if (this.repository.isDisabled(10000))
      return false;
    return Level.DEBUG.isGreaterOrEqual(getEffectiveLevel());
  }