避免使用if子句

Question

我正在嘗試在我的系統內記錄信息。每當記錄時，我都會檢查一個狀態是否有效，然後才記錄該信息。

//doing this logging all over the place in my code base. 
if(checkIsValid){ 
    Object obj =new Object(string1,Integer2,........String10); 
    log(obj); 
} 

這裏的問題是如何重構我的代碼，以便我不必在各處重複這段代碼以進行日誌記錄。

其中一個解決方案是我可以寫一個這樣的補充方法。

method(String1,Integer2,......String10){ 
    if(checkIsValid){ 
    Object obj =new Object(string1,Integer2,.........String10); 
    log(obj); 
    } 
} 

但不足之處是，我將不得不通過一些字符串作爲參數傳遞給我的方法看起來並不乾淨的。

Answer 1

如果您使用log4j進行日誌記錄，則可以使用您自己的驗證擴展記錄器類並覆蓋日誌，調試，錯誤....等方法。

如果不是像你說的，你可以這樣做：創建下面的新方法：

public void validateAndLog(Object obj){ 
     if(checkIsValid){ 
      log(obj); 
     } 
    } 

OR

public void validateAndLog(String... strs){ 
      if(checkIsValid){ 
       log(Arrays.toString(strs)); 
      } 
     } 

Answer 2

你可以使用一個工廠函數爲對象類似

public Object createObjChecked(String1,......String10){ 
    Object obj = null; 
    if(checkIsValid(String1, ..., String10)){ 
     obj = new Object(string1,.........String10); 
     log(obj); 
    } 
    return obj; 
} 

然後，您可以檢查對象是否在使用位置實際創建。

Answer 3

你的解決方案很好。 你只是缺少傳球可變參數：

void myLogMethod(String format, Object args){ 
    if(checkIsValid){ 
    Object obj = new Object(format, args); 
    log(obj); 
    } 
} 

這是一種由SLF4J採取的做法。請參閱http://www.slf4j.org/apidocs/org/slf4j/Logger.html