使用Log4j时,为什么要写if (log.isDebugEnabled()) 或 if (log.isInfoEnabled())

在项目中我们经常可以看到这样的代码:

if (logger.isDebugEnabled()) {  
    logger.debug(message);  
}  

简单来说,就是用isDebugEnabled方法判断下是能提升性能的!!

有些人不明白为什么要这样写。有些人认为这样是为了能够控制日志的输出,对于下面这行代码:

logger.debug(message);  

他们的看法是:如果这样写的话,就算你把日志级别调整为info, 这里也会输出日志。其实,在debug()方法里面,就已经叛断了日志的级别。以下是isDebugEnabled()的源码:

public boolean isDebugEnabled() {  
    if(repository.isDisabled( Level.DEBUG_INT))  
        return false;  
    return Level.DEBUG.isGreaterOrEqual(this.getEffectiveLevel());  
}  

以下是debug()的源码:

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

我们可以看到,在debug()方法里做了跟isDebugEnabled()方法一样的判断。在这里,我们不禁会想,那我们凡是要输日志的地方,都不判断isDebugEnabled,直接调debug方法,不更省事吗?

官方的说法是:出于效率考虑,看具体情况而定.

我们来看下面一行代码:

logger.debug("The money is " + getTotalMoney());  

假设我们的日志级别设置为info,那这句话不会输出日志,但这个方法还是会调用。要调用这个方法,必须提供参数。getTotalMoney()方法返回的结果就是参数的一部分。假设getTotalMoney()要执行10秒钟,10秒
钟后,进入到debug()方法里,碰到了第一个判断:

if(repository.isDisabled(Level.DEBUG_INT))  
    return;  

在这里就返回了。结果是日志虽然没有输出,却花费了10秒钟来构造参数。很显然这里得不偿失的。尽管实
际应用中几乎不可能有这种花10秒钟来构造这样一个参数的情况,但如果并发数大的话,这样写还是会影响系统的性能的。这个时候,就应该写成:

if(logger.isDebugEnabled()){  
    logger.debug("The money is " + getTotalMoney());  
}  

如果debug的参数很简单的话,也可以直接写 logger.debug(message)的。官方的说法,执行一次logger.isDebugEnabled()这样的判断花费的时间大概是写日志时间的万分之一.虽然这个比例很小,但是,程序中的任何地方放到并发的环境下,我们就得重新考虑了。

所以,我觉得,我们开发的时候, 应该视具体的情况选择不同的写法。

  • 3
    点赞
  • 4
    收藏
    觉得还不错? 一键收藏
  • 2
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 2
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值