グローバル変数を作成してメソッド呼び出しを減らす

Question

私のコードには4つのメソッドがあり、すべてのメソッドが単一メソッドを呼び出しています。重複したメソッド呼び出しを避けるために、私はグローバル変数を作成し、そこにメソッドの結果を格納しました。私はそれが良い考えではないと言われました。私たちにこれのための他の選択肢がありますか？

public void initializePortletData(PortletRequest request, PortletResponse portletResponse, Model model) { 
     model.addAttribute(TABLE, test(request)); 
    } 

    public void getInvitedUsers(PortletRequest request, PortletResponse response, Model model) { 
     model.addAttribute(TABLE, test(request)); 
    } 

    @Override 
    public Map getJSParams(PortletRequest request, ModelAndView modelAndView) { 
     Map<String, Object> model = new HashMap<>(); 

      model.put(TABLE, test(request)); 

     return model; 
    } 

Answer 1

場合によっては、メソッドが返す前にいくつかのクラス変数を設定している可能性があります。次回メソッドが呼び出されたときに、この値を戻します。