必須使用參數化信息的方式: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()); } }