私人方法

Question

JUnit測試我讀了這個問題：How do I test a class that has private methods, fields or inner classes?，似乎我可能有一個代碼味道，但我的代碼是非常簡單的實際重構。我創建的設計出了什麼問題。 我創建了一個委託類處理一些行動有三個方法execute(Action);PopulateActionHandlers()和executeActionhandlers(); 我的類象下面這樣：

public class DelegateHandler{ 
    Map<Integer,ActionHandlers> handlerMaps; 

    public execute(Action action){ 
     populateActionHandlers(action); 
     executeActionHandlers(); 

    }//end of execute 

    //This method can create and populate more than one handlers in handlerMap 
    private populateActionHandlers(action){ 
     handlerMap = new LinkedHashMap<ActionHandlers>(); 
     if (action.isMultimode()){ 
      handlerMap.add(1,new handler(action.getabc())); 
      handlerMap.add(2,new handler(action.getabc()-1)); 
     }else{ 
      handlerMap.add(1,new handler(action)); 
     } 

    }//end of populateActionHandlers 

    //This method can execute more than one handlers in handlerMap 
    private executeActionHandlers(){ 
     for(ActionHandler actionHandler : handlerMap.values){ 
      actionHandler.executeAction(); 
     } 

    }//end of executeActionHandlers 
} 

現在我想測試populateActionHandlers()方法使用JUnit，我做私人的有無需在課堂以外暴露。如果我測試​​方法，然後將測試這兩個populateActionHandlers()和executeActionHandlers()方法，這是在同一時間測試兩個單位，我想單獨測試。設計（我認爲）對我來說似乎是好的，並且不允許任何問題，但是我會改變對該方法的訪問權限（並且僅僅爲了測試而沒有證明在我看來是正確的？）或者使用反思（這是一個好主意，它以某種方式感覺不對，人們通常使用反射進行junit測試？）。所以唯一不能排除的就是代碼味道。但可能是我的代碼竇是不是真的幫助我所以我想了解，如果我可以改善這個代碼。

Answer 1

建議不將測試私有方法不應阻止一個留下了私有方法做一個奇怪的設計，而應執行測試只能在具有明確的語義的方法。

私人方法通常是技術幫手。如果底層的數據結構發生變化，它們的語義會發生變化，如果調用的公共方法使用另一種算法來實現相同的目標，它們甚至可以被優化。

我會改寫PROGRAMM以下列方式：

... 
public execute(Action action){ 
    Map<Integer,ActionHandlers> handlerMap = populateActionHandlers(action); 
    executeActionHandlers(handlerMap); 
} 
... 

存儲一個函數的結果轉換成私有字段只從這一領域中的另一個功能是不是線程安全的檢索和難以維護。

然而，這個重構會打破所有（但不是寫）測試你的例子的私有方法，因爲接口被改變。如果你只測試了公共方法，那麼在重構之後所有的測試都是有效的。

我知道少數情況下，測試私有方法就可以了。儘管測試私有方法通常是可以避免的，但我認爲私有狀態的檢查有時是一種更好的選擇，而不僅僅是檢查對象的公共狀態。這種檢查可能不如以前那麼健全（但原因如此），但公共狀態往往不完整，很難斷言。在這兩種情況下，我都使用框架picklock，這使得用戶可以方便地訪問私有方法和字段。