必须使用参数化信息的方式:
logger.debug("Processing trade with id:[{}] and symbol : [{}] ", id, symbol);对于debug日志,必须判断是否为debug级别后,才进行使用:
if (logger.isDebugEnabled()) { logger.debug("Processing trade with id: " +id + " symbol: " + symbol); }不要进行字符串拼接,那样会产生很多String对象,占用空间,影响性能。 反例(不要这么做):
logger.debug("Processing trade with id: " + id + " symbol: " + symbol);如有参数变量,应该写成如下写法:
logger.debug("Processing trade with id:[{}] and symbol : [{}] ", id, symbol);这样的格式写法,可读性更好,对于排查问题更有帮助。
影响到程序正常运行、当前请求正常运行的异常情况:
打开配置文件失败所有第三方对接的异常(包括第三方返回错误码)所有影响功能使用的异常,包括:SQLException和除了业务异常之外的所有异常(RuntimeException和Exception)不应该出现的情况:
比如要使用Azure传图片,但是Azure未响应如果有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); }不应该出现但是不影响程序、当前请求正常运行的异常情况:
有容错机制的时候出现的错误情况找不到配置文件,但是系统能自动创建配置文件即将接近临界值的时候,例如:
缓存池占用达到警告线业务异常的记录,比如:
当接口抛出业务异常时,应该记录此异常系统运行信息
Service方法中对于系统/业务状态的变更主要逻辑中的分步骤外部接口部分
客户端请求参数(REST/WS)调用第三方时的调用参数和调用结果并不是所有的service都进行出入口打点记录,单一、简单service是没有意义的(job除外,job需要记录开始和结束,)。 反例(不要这么做):
public List listByBaseType(Integer baseTypeId) {
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);}
对于复杂的业务逻辑,需要进行日志打点,以及埋点记录,比如电商系统中的下订单逻辑,以及OrderAction操作(业务状态变更)。
对于整个系统的提供出的接口(REST/WS),使用info记录入参
如果所有的service为SOA架构,那么可以看成是一个外部接口提供方,那么必须记录入参。
调用其他第三方服务时,所有的出参和入参是必须要记录的(因为你很难追溯第三方模块发生的问题)
如果代码中出现以下代码,可以进行优化:
//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级别替代)
