phpで関数内の関数から、エラーをキャッチできませんか？

Question

長くなってしまって申し訳ございませんが、PHPにお詳しい方のお目に留まりましたら幸いです。

以下３つの関数があり、➂の中で➀➁のエラー（getCodeとgetMessage）を取得することはできませんか？

###発生中の問題
➂のgetPreviousで以下メッセージがでて実行できないです。

PHP Fatal error:  Uncaught Error: Call to a member function getPrevious() on null in /workspace/Main.php:13
Stack trace:

###コード

<?php

var_dump( myGet( 'one', ['name'=>'x'] ) );

// ➂右記を実行する「➀引数をバリデーションして、➁SQLから値を取得する」
function myGet( $process, $args ){
    $result = null;
    try {
        $e = null;
        
        // ➀のエラーをチェック
        $name = myValid( $process, $args['name'] );
        $prev = $e->getPrevious();
        if ( $prev ) $e = e('myValid()でエラー', 0, $e);
        
        // ➁のエラーをチェック
        $row = myRow( $name );
        $prev = $e->myRow();
        if ( $prev ) $e = e('myRow()でエラー', 0, $e);
        
        if ( $e ) {
            throw $e;
        }else{
            $result = $row;
        }
    } catch (Exception $e) {
        $result = exception_to_array( __FUNCTION__, $e );
    }
    
    return $result;
}

// ➀SQLから値を取得
function myRow( $name ){
    $result = null;
    try {
        $e = null;
   
        // 本来のコード
        // $row = $dbh->query( "SELECT * FROM t_user WHERE naem=$name" );       
        // 本来のコードによって以下$rowが取得されたと仮定
        $row = ['ID'=>1, 'name'=>$name, 'age'=>20];
        $result = $row; 
        
        // この関数でエラーチェック
        if ( empty($row) ) $e = e('レコードがない', 4, $e);

        if ($e) {
            throw $e;
        }
    } catch (Exception $e) {
        $result = exception_to_array( __FUNCTION__, $e );
    }

    return $result;    
}

// ➁引数をバリデーション
function myValid( $process, $name ){
    $result = null;
    
    $ok = [
        'one'=>['a','b'],
        'two'=>['x','y','z']
    ];

    try {
        $e = null;
        
        // この関数でエラーチェック
        if ( ! isset($process) ) $e = e('processが空', 1, $e);
        if ( ! isset($name) ) $e = e('nameが空', 2, $e);
        if ( !$e ) {
            if ( ! in_array( $name, $ok[$process], true ) ) {
                $e = e('$nameが不正', 3, $e);
            }else{
                $result = $name;
            }
        }
        
        if ($e) {
            throw $e;
        }
    } catch (Exception $e) {
        $result = exception_to_array( __FUNCTION__, $e );
    }

    return $result;
}

// エラーチェック用
function e( $message, $code, $previous = null) {
    return new Exception($message, $code, $previous);
}
function exception_to_array($func_name, Exception $e) {
    do {
        $errors[$func_name][$e->getCode()] = $e->getMessage();
    } while ($e = $e->getPrevious());
    return array_reverse($errors);
}

###試したこと
上記全ての関数で、戻り値の$resultを$result['result']として、➂の中では$result['result'] === 'errror'というチェックをしましたが、これでも➀と➁の何が悪いか？（getCodeとgetMessage）を取得することができませんでした。

// ※変更1から// 変更8が上記との変更箇所です。

<?php

var_dump( myGet( 'one', ['name'=>'x'] ) );

// ➂右記を実行する「➀引数をバリデーションして、➁SQLから値を取得する」
function myGet( $process, $args ){
    $result = ['result'=>null]; // ※変更1
    try {
        $e = null;
        
        // バリデーション
        $name = myValid( $process, $args['name'] );
        if( $name['result'] === 'error' ){ // ※変更2
            $e = e('myValid()でエラー', 0, $e);
        }

        // SQLでレコードを取得
        $row = myRow( $name['result'] );
        if( $row['result'] === 'error' ){ // ※変更3
            $e = e('myRow()でエラー', 0, $e);
        }
        
        if ( $e ) {
            throw $e;
        }else{
            $result = $row;
        }
    } catch (Exception $e) {
        $result = exception_to_array( __FUNCTION__, $e );
    }
    
    return $result;
}

// ➁SQLから値を取得
function myRow( $name ){
    $result = ['result'=>null]; // ※変更4
    try {
        $e = null;
   
        // 本来のコード
        // $row = $dbh->query( "SELECT * FROM t_user WHERE naem=$name" ); 
        // 本来のコードによって以下$rowが取得されたと仮定
        $row = ['ID'=>1, 'name'=>$name, 'age'=>20];
        $result['result'] = $row; // ※変更5 
        
        // この関数でエラーチェック
        if ( empty($row) ) $e = e('レコードがない', 4, $e);

        if ($e) {
            throw $e;
        }
    } catch (Exception $e) {
        $result = exception_to_array( __FUNCTION__, $e );
    }

    return $result;    
}

// ➁引数をバリデーション
function myValid( $process, $name ){
    $result = ['result'=>null]; // ※変更6
    
    $ok = [
        'one'=>['a','b'],
        'two'=>['x','y','z']
    ];

    try {
        $e = null;
        
        // この関数でエラーチェック
        if ( ! isset($process) ) $e = e('processが空', 1, $e);
        if ( ! isset($name) ) $e = e('nameが空', 2, $e);
        if ( !$e ) {
            if ( ! in_array( $name, $ok[$process], true ) ) {
                $e = e('$nameが不正', 3, $e);
            }else{
                $result['result'] = $name; // ※変更7
            }
        }
        
        if ($e) {
            throw $e;
        }
    } catch (Exception $e) {
        $result = exception_to_array( __FUNCTION__, $e );
    }

    return $result;
}

// エラーチェック用
function e( $message, $code, $previous = null) {
    return new Exception($message, $code, $previous);
}
function exception_to_array($func_name, Exception $e) {
    do {
        $errors['result'] = 'error'; // ※変更8
        $errors[$func_name][$e->getCode()] = $e->getMessage();
    } while ($e = $e->getPrevious());
    return array_reverse($errors);
}

Comments

[退会済みユーザー]

「エラーをキャッチ」したいのですか？例外ではなくて？

[myako]

すみませんエラーと例外を混同していました。「例外をキャッチ」したいです。

Answer 1

前置き

PHP Fatal error: Uncaught Errorを正確にハンドリングするのは結構大変ですし、そもそも発生した時点でどうにもリカバリーできない状態になるので、エラー内容を把握して修正することをお勧めします。
参考お前は PHP 7 における Fatal Error / Catchable Fatal Error / Error / ErrorException / Exception の違いを言えるか？

回答

今回のケースだと例外をスタックに積むためにthrowせずに戻り値として返しているのに、
その戻り値を使用していないことが問題です。
throwと戻り値の違いについて把握する必要があります。

Call to a member function getPrevious() on nullなので、

        //ここで$eがnullになってるので
        $e = null;

        // ➀のエラーをチェック
        $name = myValid( $process, $args['name'] );
        //null->getPrevious() をしていることになる
        //throwしているわけでは無いので、myValidの中でエラーがあっても$nameに格納される
        $prev = $e->getPrevious();

今の構造のままだと、こういう感じにしないといけないでしょうね。

        //ここで$eがnullになってるので
        $e = null;
        
          // ➀のエラーをチェック
          $name = myValid( $process, $args['name'] );
          //null->getPrevious() をしていることになる
          if(isset($name['error']) === true){
            $prev = $name->getPrevious();
            if ( $prev ) $e = e('myValid()でエラー', 0, $prev);
          }

ただ、私には最終的にどうしたいのかがよくわかりません。
myValid()で例外が発生している=myRow()に進んではいけないはずなので、その場でthrowし直すなりして処理を止めないといけないのでは？

---

おそらく、
[PHP] まとめて例外をスローする小技
を参考にされたのかなと思いますが、まずは前述のthrowと戻り値の違いについて把握されること、参考URL中の前提となっている

というように Exception を使った処理を諦めている人が多いのではないだろうか？

という処理で実装出来るようにすること、
処理をどこで止めなければならないかを整理すること
スタックしない単一の例外で例外処理を実装してみること
をお勧めします。