代码中日志必须先判断级别

错误案例

现象描述：发现某些服务器的pv数不高，但服务器的load却不低，高于平均水平

错误分析

分析过程：

通过内存监控发现，GC的动作比较频繁，但无法找到原因，一次PLA偶然的发现了大量如log.debug(“memberId:” + member.getMemberId())代码

原因分析：

在代码中发现如下代码段很多：

log.debug(“memberId:” + member.getMemberId())

以上代码执行时，分两步：

1. 先执行的是括号中的字符串相”+”的动作，而每次”+”运算都会导致新字符串的生成，这样就产生了很多“中间字符串”，在极大次数被调用时，这种字符串被创建和销毁的数量非常庞大，从而造成了jvm gc频繁执行，进而影响了性能。

2. 再执行log.debug()函数，在生产环境log level一般大于info，所以实际不会打印debug信息综上所述，这些代码在生产环境不会产生日志，但会执行字符串”+”运算，而这些运算是无意义的，所以需要先判断日志的优先级，方式是log.isXXXEnabled() { log.XXX(……); }

Log Level的级别： Fatal->error->warn->info->debug，级别从高到低

一般我们生成环境的log level都是error，所以对于Error以上级别的日志，不用判断；对于error以下级别的都要加上判断。

正确用法

正确用法：

//加上是否enable的判断，屏蔽log.debug的执行
if(log.isDebugEnable()){
    log.debug(“memberId:”+member.getMemberId());
}