PHPでdieさせるタイミングとテクニック

Question

初めまして。
よろしければPHPでのdieの使い方について教えてください。
要領を得ない部分があるかもしれませんが、どうぞ宜しくお願い致します。

###前提
下記のような「関数➀（check_data）」があります。
「$input」が「$lists」に一致するかどうかをチェックする関数です。

そしてこの「関数➀（check_data）」は「関数➁（echo_data）」の中で使うのですが、そこで質問があります。
###質問
もし「$input」が「$lists」に一致しないときは➁を「die」させたいのですけど、それを書くタイミングがわかりません。

どこでどのように「die」させるのが正解でしょうか？

現状では下記の場所に書いていますが、これでは（たぶんforeachがlistの一件目で終わってしまうせいでしょうか）出来ませんでした。

<?php
/*
関数➀
値をチェックする
*/
function check_data($input){
    
    $lists = [
    '^a-.{1}$',
    '^b-.{1}$', 
    ];
    
    $matched = FALSE; 
    
    foreach ( $lists as $list ){
        if( @preg_match('|'.$list.'|', $input) ) {
            $parts = explode( '-', $input );
            if( $parts[0] == 'a' ){
                $type = 'A'; 
            } 
            elseif( $parts[0] == 'b' ){
                $type = 'B'; 
            }
            else{
                die('不正な値です。'); // この場所は違うが、どこに書けばいいのか？
            } 
            return array($input, $type);
        }
    } 

}

/*
関数➁
値をチェックしつつ色々出力する
*/
function echo_data(){
    echo '<h4「$input」が「$lists」に一致しなければ、この関数の実行をdieさせたい</h4>';
    
    $input = 'b-1x'; // xがあるので一致せず、よって一切echoさせずにdieしたいけどできない
    
    if( !empty($input) ){
        list( $input, $type ) = check_data($input); 
        echo '<p>$inputは$listsと一致しました。</p>';
        echo '<p>$inputの値はこれ→'.$input.'</p>';
        echo '<p>$typeの値はこれ→'.$type.'</p>';
    }

}

echo_data();
?>

###バージョン
PHP 7.3.2

Answer 1

どこでどのように「die」させるのが正解でしょうか？

特殊な場合を除いて、HTML出力の途中でdie()させないのが正解です。途中でdie()してしまうと、全部の処理がそこで打ち切られるため、HTMLもぶった切れた状態となってしまいます。

例外などでエラーの状態もきちんと引き回して、表示側でそれを見て適切なHTMLを生成するようにしましょう。

Comments

[omorikomori]

echo_dataは必要なＨＴＭＬが全て出力した後に実行されるので、仰る「特殊な場合」に該当するかと思います。

[maisumakun]

</html>などはどこで出力していますか？

[maisumakun]

「「$input」が「$lists」に一致しなければ」とありますが、その条件が成立するのはどのような状況でしょうか。

[omorikomori]

</html>はページの最後です。あ、echo_dataはAjaxの先にあるPHPに書かれているものですよ。条件の成立判定はAjaxでされるので、不一致ならdieして、HTML（Ajaxで出力されるHTML）もぶった切ってしまっても良さそうかと考えているのですが、やはりおかしいでしょうか？

[tabuu]

check_datahはAjaxで呼び出されるということが前提に書いてないので、書いておいたほうが良い回答もらえますよ。

Answer 2

想定外のデータを検知した場合、return falseやreturn null
して呼び出し元で不正を判断できるようにすればいいでしょう。

きちんとやるには例外をthrowしたりアサーションで処理したりする
必要があります。

Comments

[omorikomori]

ありがとうございます。遅くなってしまい申し訳ございません。

echo_dataはAjaxの先にあるPHPに書かれているものでして、条件の成立判定はAjaxでされるので、不一致ならdieしてようかと考えているのですが、質問のコードにdieがあること自体がおかしいのでしょうか？

「die」と「return falseやreturn null」との違いやメリットなどはどのような事になりますでしょうか？

Answer 3

どこでどのように「die」させるのが正解でしょうか？

全出力の前に header(http ステータスコード発行のため) と共に使用するのが正解だと思います。
例外吐いて、補足させる感じが一般的になるかと。