【Java】サーブレットに書かれているreturnについて

Question

サーブレットについて勉強しています。

protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
 RequestDispatcher dispatcher = request.getRequestDispatcher("test.jsp");
dispatcher.forward(request, response);
return;
}

上記のコードについて、returnが書かれているコードとないコードがあるのですが、どちらが良いのでしょうか？

以前、https://teratail.com/questions/764で「returnで終了させた場合は例外処理のfinallyブロック内の処理が実行されます」という回答があったのですが、これと関係があるのでしょうか。

宜しくお願いします。

Answer 1

以前、https://teratail.com/questions/764で「returnで終了させた場合は例外処理のfinallyブロック内の処理が実行されます」という回答があったのですが、これと関係があるのでしょうか。

これは例外処理（try-catch-finally)を入れていた場合なので、
今回のメソッドには例外処理がなく、関係ありません。

なので、最後のreturn;は冗長だと思います。

私はこんな風にreturn;を書いててできる上司から「無駄なことすんな」と怒られました。

Comments

[natsuno]

貴重な経験談をありがとうございました。
色々な現場があるのですね。
参考にさせて頂きます。

Answer 2

あっても、なくても動作に変わりは無いので
必要か？と問われると。「必要は無い。」ですね。

総じて、記載したほうが良いと思います。
マストではないがベターという意味です。
理由は2点。

Servlet特有の理由で記載したほうが良いです。
下記のようなコードを書くとExceptionが発生します。

response.Redirect("XXXXXXX");
response.Write("");

Redirect後は、responseの更新は認めていないらしく、Exceptionが発生します。
ずいぶん、古いVersionのJavaでしたし、
私が利用していた、WebアプリケーションServerの問題かもしれません。
かなり有名な商用Serverとだけお伝えします。

結果、私はExceptionが発生しているPG全ての「response.Redirect("XXXXXXX");」の後に
returnをつける作業をすることになりました。

今回のケースでも似たことが起こる可能性もあります。
ただ、毎回つけるつけないの判断をして、付け忘れるよりかは、
全部つけてしまった方が、分かりやすいですしコードも早く書けます。

理由をもう一つ。これは個人差もあると思います。
可読性を上げるためです。

なぜなら、私は読みやすいコードは、読みやすい文章と似ている。と思っているからです。
「return」は文章で言うところの「以上」「敬具」と同じ意味でとらえています。

読み手に、コードはここで処理が終了ですよ。と明示する意味で記載しています。
無くても分かる。のですが、存在したほうが少し、読み手の理解が早くなります。

長くなってしまったコード等で効果を発揮します。
が、私は判断したくないのですべてのコードでreturnをつけます。

以上、長くなってしまいましたが、参考にしてもらえればと思います。

Comments

[natsuno]

色々な考え方があるのですね。
大変勉強になりました。
丁寧な回答をありがとうございました。
参考にさせて頂きます。

Answer 3

このメソッドには System.exit() も try - catch - finally も書かれていませんし, return は最後の行なので, 有っても無くても構わないと思います

Comments

[natsuno]

回答ありがとうございました。
私もそうなのかな・・・と思って投稿した次第です。
参考にさせて頂きます。