必须使用参数化信息的方式:java
logger.debug("Processing trade with id:[{}] and symbol : [{}] ", id, symbol);
对于debug日志,必须判断是否为debug级别后,才进行使用:git
if (logger.isDebugEnabled()) { logger.debug("Processing trade with id: " +id + " symbol: " + symbol); }
不要进行字符串拼接,那样会产生不少String对象,占用空间,影响性能。 反例(不要这么作):github
logger.debug("Processing trade with id: " + id + " symbol: " + symbol);
若有参数变量,应该写成以下写法:缓存
logger.debug("Processing trade with id:[{}] and symbol : [{}] ", id, symbol);
这样的格式写法,可读性更好,对于排查问题更有帮助。架构
影响到程序正常运行、当前请求正常运行的异常状况:框架
不该该出现的状况:ide
若是有Throwable信息,须要记录完成的堆栈信息:性能
log.error("获取用户[{}]的用户信息时出错",userName,e);
反例(不要这么作):优化
try{ .... }catch(Exception ex){ String errorMessage=String.format("Error while reading information of user [%s]",userName); logger.error(errorMessage,ex); throw new UserServiceException(errorMessage,ex); }
不该该出现可是不影响程序、当前请求正常运行的异常状况:debug
即将接近临界值的时候,例如:
业务异常的记录,好比:
系统运行信息
外部接口部分
log.info("开始查询基地"); BaseExample ex=new BaseExample(); BaseExample.Criteria ctr = ex.createCriteria(); ctr.andIsDeleteEqualTo(IsDelete.USE.getValue()); Optionals.doIfPresent(baseTypeId, ctr::andBaseTypeIdEqualTo); log.info("查询基地结束"); return baseRepository.selectByExample(ex);
} 2. 对于复杂的业务逻辑,须要进行日志打点,以及埋点记录,好比电商系统中的下订单逻辑,以及OrderAction操做(业务状态变动)。 3. 对于整个系统的提供出的接口(REST/WS),使用info记录入参 4. 若是全部的service为SOA架构,那么能够当作是一个外部接口提供方,那么必须记录入参。 5. 调用其余第三方服务时,全部的出参和入参是必需要记录的(由于你很难追溯第三方模块发生的问题)
若是代码中出现如下代码,能够进行优化:
//1. 获取用户基本薪资 //2. 获取用户休假状况 //3. 计算用户应得薪资
优化后的代码:
logger.debug("开始获取员工[{}] [{}]年基本薪资",employee,year); logger.debug("获取员工[{}] [{}]年的基本薪资为[{}]",employee,year,basicSalary); logger.debug("开始获取员工[{}] [{}]年[{}]月休假状况",employee,year,month); logger.debug("员工[{}][{}]年[{}]月年假/病假/事假为[{}]/[{}]/[{}]",employee,year,month,annualLeaveDays,sickLeaveDays,noPayLeaveDays); logger.debug("开始计算员工[{}][{}]年[{}]月应得薪资",employee,year,month); logger.debug("员工[{}] [{}]年[{}]月应得薪资为[{}]",employee,year,month,actualSalary);
特别详细的系统运行完成信息,业务代码中,不要使用.(除非有特殊用意,不然请使用DEBUG级别替代)
@Override @Transactional public void createUserAndBindMobile(@NotBlank String mobile, @NotNull User user) throws CreateConflictException{ boolean debug = log.isDebugEnabled(); if(debug){ log.debug("开始建立用户并绑定手机号. args[mobile=[{}],user=[{}]]", mobile, LogObjects.toString(user)); } try { user.setCreateTime(new Date()); user.setUpdateTime(new Date()); userRepository.insertSelective(user); if(debug){ log.debug("建立用户信息成功. insertedUser=[{}]",LogObjects.toString(user)); } UserMobileRelationship relationship = new UserMobileRelationship(); relationship.setMobile(mobile); relationship.setOpenId(user.getOpenId()); relationship.setCreateTime(new Date()); relationship.setUpdateTime(new Date()); userMobileRelationshipRepository.insertOnDuplicateKey(relationship); if(debug){ log.debug("绑定手机成功. relationship=[{}]",LogObjects.toString(relationship)); } log.info("建立用户并绑定手机号. userId=[{}],openId=[{}],mobile=[{}]",user.getId(),user.getOpenId(),mobile); }catch(DuplicateKeyException e){ log.info("建立用户并绑定手机号失败,已存在相同的用户. openId=[{}],mobile=[{}]",user.getOpenId(),mobile); throw new CreateConflictException("建立用户发生冲突, openid=[%s]",user.getOpenId()); } }