PHPでクラス中のメソッドからreturnで返された場合、そのメソッド内ではreturn以降は処理されないという認識は間違いでしょうか？

【質問】
PHPでクラス中のメソッドからreturnで返された場合、そのメソッド内ではreturn以降は処理されないという認識は間違いでしょうか？

【例】あるクラスhogeを、newした後に、hogeクラス中にhugaメソッドを実行すると仮定します。

php
1(とあるプログラム内)
2$obj = new hoge();
3$obj->huga();
4
5
6(hogeクラス)
7class hoge(){
8    public function huga(){
9        if(条件){
10         return true;   //↓※１ここ以降の処理は行われないと思っていた。
11        }
12    //※２ ここ以降にも処理が実装されている
13    }
14}

hogeのクラスから抜ける場合、上記※１の様にreturnで抜けられると思っておりました。
ところがチームメンバーにソースレビューで
「return の直後にexit入れた方がいいのでは？」
という指摘を受けました。

return を実装するだけで、上記※１と※２のように「return以降の処理は一切されない」という認識は間違いでしょうか？

行動規範の内容に同意します

回答4件

return の後に exitが必要ないと言うのはその通りなのですが、なぜそのような誤解が生じたのか考えてみました。一つ思い当たるは、リダイレクト処理の後には exit を書くべきということです。
次のようなコードを考えてみます。

PHP
1
2if (! isloggedin() ) { // ログインしていない場合は
3  header('Location: https://example.jp/loginform.php'); // ログイン画面にリダイレクト
4}
5// 以下、ログインしているユーザーだけが見ることのできる情報の表示

このようなコードを書くと、一見うまく動いているように見えますが、リダイレクト時のHTTPレスポンスには秘密情報が含まれているので、ツール等を使うとその情報を閲覧できます。ログインしていないユーザーが秘密情報を閲覧できてしまうことになります。

このため、リダイレクト処理の header関数呼び出しの後は、仮に後続の処理がなくても、exitを明示するとこの種の事故を避けることができます。

return とリダイレクトでは全然違いますが、レビュアーの方は、この種のケースと勘違いされたのではないでしょうか。

投稿2016/12/29 14:19

ockeghem

総合スコア11705

退会済みユーザー

2017/01/04 01:22

＞ockeghem様返答に遅れましたことをお詫び申し上げます。あの後レビュアーに、まさに同じことを誤解されて「exitを入れるべきだ」という指摘を行ったとのことでした。セキュリティの実例を交えたご説明、誠にありがとうございました。

行動規範の内容に同意します

exitはそのプログラムを終わらせるものです。一方、returnは呼び出し元へ返り値を返し呼び出し元で処理を終了するものです。
returnは呼び出し元へ戻ってから終了するのに対し、exitは即座に終了します。
たとえば、以下の例1のreturnを用いたコードならば、textMatch()以降を続けていますが、例2のexitを用いたコードでは関数でexitでプログラムを終了させたため、textMatch()以降の処理が行われません。
例1

PHP
1<?php
2$text1 = "text";
3$text2 = "text";
4$text3 = "text2";
5function textMatch($text1, $text2)
6{
7    if ($text1 == $text2) {
8        echo "text1";
9        return true;
10    }
11    return true;
12}
13
14textMatch($text1, $text2);
15
16if ($text1 != $text3) {
17    echo "text2";
18}

例2

PHP
1<?php
2$text1 = "text";
3$text2 = "text";
4$text3 = "text2";
5function textMatch($text1, $text2)
6{
7    if ($text1 == $text2) {
8        echo "text1";
9        exit();
10    }
11    return true;
12}
13
14textMatch($text1, $text2);
15
16if ($text1 != $text3) {
17    echo "text2";
18}