メソッドの作り方

Question

引数のないメソッドの作成の仕方について

下記のプログラムはServletの一部分です。
if文の中に共通部分（request.setAttribute("params"……）があるので、これを
まとめてメソッドをつくりたいのですが、どのようにすればよいでしょうか。

 private void touroku(HttpServletRequest request,

HttpServletResponse response) throws ServletException, IOException {

     Answer<Meibo> analyzed = analysis.analyze(request);
     if (analyzed.hasError()) {
         analyzed.set(request);

         request.setAttribute("params", analysis.params(request));
         dispatch(request, response, "/WEB-INF/jsp/meibo.jsp");

     }

     Answer<Integer> result = supply.touroku(analyzed.getValue());
     if (answer.hasError()) {
         answer.set(request);

         request.setAttribute("params", analysis.params(request));
         dispatch(request, response, "/WEB-INF/jsp/meibo.jsp");

     } else {
         answer.addReport("SUBMIT");
         answer.set(request);
         list(request, response, answer);
     }
 }

よろしくお願いします。

Answer 1

analyzed に対して、nullチェック無しでanalyzed.hasErrorを書いているので、おそらくnullがないのだと判断しています。
また、analyzed.getValue()がerrorがあっても問題なく取得できると仮定しています。

    private void touroku(HttpServletRequest request, 
HttpServletResponse response) throws ServletException, IOException { 

         Answer<Meibo> analyzed = analysis.analyze(request);
         Answer<Integer> result = supply.touroku(analyzed.getValue()); 
         
         if (analyzed.hasError() || answer.hasError()) { 
             analyzed.set(request); 

             request.setAttribute("params", analysis.params(request)); 
             dispatch(request, response, "/WEB-INF/jsp/meibo.jsp"); 
         } else {
             answer.addReport("SUBMIT"); 
             answer.set(request); 
             list(request, response, answer); 
         }
     } 

もし、Errorがある時にresultが取得できない内容だと申し訳がないですが、
メソッドを作らなくてもこれで良いのではないかと思います。

Comments

[eripong]

これだと、answer.hasError()がtrueの場合に、
answer.set(request)が実行されないですが、良いのでしょうか？

それと、質問のコードでもそうですが、resultは、戻り値を代入したまま、
その後使用されないですが良いのでしょうか？

[anonymouskawa]

そもそもanswerはどこから出てきたのでしょうか…。

hasError: {
if (analyzed.hasError() {
analyzed.set(request);
} else {
if (answer.hasError()) {
answer.set(request);
} else {
System.out.println("answer.addReport(\"SUBMIT\");");
answer.addReport("SUBMIT");
answer.set(request);
list(request, response, answer);
break hasError;
}
}
request.setAttribute("params", analysis.params(request));
dispatch(request, response, "/WEB-INF/jsp/meibo.jsp");
}
という手法もあります。

Answer 2

おそらく、下記の2行を引数のないメソッドにしたいとうことだと想像しますが、

             request.setAttribute("params", analysis.params(request)); 
             dispatch(request, response, "/WEB-INF/jsp/meibo.jsp"); 

この場合、呼び出しメソッドの引数(request, response)を参照しているので、その引数を別途作成するメソッドに引数で引き継ぐ必要があります。もしくは、呼び出し元メソッドでrequest、responsの参照をクラスのprivate変数にいったん格納して、その変数を別途作成する引数なしのメソッドで参照するようにします。