質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.60%

PHP: functionの中で検知されたPDOExceptionの内容を、当該function呼び出し元で取り扱いたい

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 777

score 168

functionの中でもDB更新をしていて、PDOのエラーを検知する可能性があります。
呼び出し元でのDB更新部分で検知されるPDOのエラー同様、エラーの取り扱いは呼び出し元で取り扱いたい=呼び出し元で画面構成を行うコーディング部分に活かしたい、と考えています。(呼び出された側で発生したPDOのエラーメッセージを呼び出し元で取得したい)
こういった場合、どういう手続きをとるのが一般的でしょうか?

以下コーディングでは、一般的な例外としてキャッチはできるものの、PDOのエラー詳細まで呼び出し元で取得できないのかな??と思い悩んでいます。
Exceptionクラスの拡張・getPreviousとかいうものを利用してみたかったのですが、よく分からないのでお問い合わせさせて頂きました。

よろしくお願いいたします。
参考を試みたサイト

try {

    『DB更新処理①』、グダグダ直接記載    //errorはPDOEceptionでcatch

    if(★DB更新処理②★) != null) {        //DB更新処理②は使い勝手よくFunctionにしてる
            throw new Exception();        //内部ではやはりPDOEceptionでcatchされる
        }                    //しかし、呼出し元で上記同様エラー制御をしたい。

    GOTO 正常完了処理へ

} catch (PDOException $e) {
        error_log('DB更新処理①に失敗'.$e->getMessage(),0);
} catch (Exception $e) {
        error_log('DB更新処理②に失敗'.$e->getMessage(),0);
}

//DB更新処理①でも②でも、エラー目的の処理をここで行う
//つまりDB更新処理②についても、エラー取扱いは呼出し元で行いたい。


functionDB更新処理②★($conn,~) {
    try {
    $errcd = null;
        『DB更新処理②本質』、グダグダ直接記載    //errorはPDOEceptionでcatch
    return $errcd;                //errorなければnullでかえす

    } catch (PDOException $e) {
        $errcd = 999;
    return $errcd;
    }
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • 退会済みユーザー

    退会済みユーザー

    2019/02/10 13:43

    何をしたいのかいまいち読み取れませんが、DB接続部分に書かれている try 〜 catch ではなく、接続部分の関数を呼び出している関数のところで catch したいということ?

    キャンセル

  • saya24

    2019/02/10 14:08

    Kosuke_Shibuyaさんお時間を頂き誠にありがとございます。
    function呼び出し元で対応することもDB更新(Aテーブル)、functionの中で対応することもDB更新(Bテーブル)。開発の構成上、Bテーブルの更新がfunctionになっているだけのことで、PDOExceptionが別れて定義されている状態です。Function内でPDO系のエラーが生じても、Function呼び出し元でエラー処理を統一しているので、呼び出し元でfunction内部で生じたPDOexceptionについて参照する(getMesseage)方法を教えて欲しい、ということです。

    呼び出し元にfunction内部のPDOExceptionの内容を引き継ぎたい、ということですね。呼び出し元のPDOExceptionと呼び出されるfunction内部のPDOExceptionは、別の次元のイメージがあります。
    で、function内部で生じたPDOExceptionを呼び出し元へ どう引き継ぐの?という質問になっています。

    キャンセル

回答 2

checkベストアンサー

0

何をしたいのかいまいち読み取れませんが、DB接続部分に書かれている try 〜 catch ではなく、接続部分の関数を呼び出している関数でcatchしたいということですかね?

サンプル

<?php

ini_set('display_error', 1);
error_reporting(E_ALL);

/**
 * DB 接続
 */
function connect()
{
    $dsn = 'mysql:host=localhost;dbname=database;charset=utf8mb4';
    $username = 'root';
    $passwd = 'password';
    $pdo = new PDO($dsn, $username, $passwd);
    $pdo->setAttribute(PDO_ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
    return $pdo;
}

/**
 * 更新処理
 * @param type $id
 * @param type $value
 * @return type
 */
function update($id, $value)
{
    $pdo = connect();
    $sql = 'update table set column = ? where id = ?';
    $stmt = $pdo->prepare($sql);
    return $stmt->execute([$value, $id]);
}

try {
    update(1, 'test');
} catch (Exception $e) {
    // connect(), update() で発生した例外はここでキャッチできる
    var_dump($e->getMessage());
}

サンプル2

<?php

ini_set('display_error', 1);
error_reporting(E_ALL);

/**
 * DB 接続
 */
function connect()
{
    try {
        $dsn = 'mysql:host=localhost;dbname=database;charset=utf8mb4';
        $username = 'root';
        $passwd = 'password';
        $pdo = new PDO($dsn, $username, $passwd);
        $pdo->setAttribute(PDO_ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        return $pdo;
    } catch (Exception $e) {
        // (1)例外をそのままthrow
        throw $e;
    }
}

/**
 * 更新処理
 * @param type $id
 * @param type $value
 * @return type
 */
function update($id, $value)
{
    $pdo = connect();
    $sql = 'update table set column = ? where id = ?';
    $stmt = $pdo->prepare($sql);
    return $stmt->execute([$value, $id]);
}

try {
    update(1, 'test');
} catch (Exception $e) {
    // (1)の例外も update() の例外もここでキャッチ可能
    var_dump($e->getMessage());
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/10 14:40

    ご丁寧な説明をありがとうございます。
    ご提示は一つのtryでのcatchですね。当方のケースでは、tryが処理Aと処理Bの中双方に存在していることの違いがありますね。処理Aに処理Bが含まれている場合、かつ双方にtry・catchが存在しているケースで、処理Bで発生したExceptionを処理Aに引き継げるか?という主旨の質問ですが、kousuke_Shibuyaさんの構成にするのが一般的ですかねぇ

    キャンセル

  • 2019/02/10 14:42 編集

    いたるところにtry - catch を書いてしまうと、例外を途中で握りうつぶしてしまい、バグの検出が困難になってしまうので、開発案件では禁じ手とするのが普通です。

    キャンセル

  • 2019/02/11 12:06

    ご親切な説明をありがとうございました。Throw $eでfuncton呼出し元へ引き継ごうと思えば引き継げるのですね。ただ仰られるようにtry~catchを複数の箇所にコーディングすることに、価値を見出せなくってきた自分がいるので方針を改めたいと思います。大変参考になるアドバイスをありがとうございました。

    キャンセル

0

マニュアルの User Contributed Notes を覗いてみてください。
参考になる記述がいくつかあります。

ざっくりというと、catchでさらに例外をthrowしてやればよいです。

try {
    $PDO = new PDO( '...' ); // PDO Driver DSN. Throws A PDOException.
}
catch( PDOException $Exception ) {
    // Note The Typecast To An Integer!
    throw new MyDatabaseException( $Exception->getMessage( ) , (int)$Exception->getCode( ) );
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/09 23:06

    お時間を頂きまして誠にありがとうございます。
    ご想像以上に初心者で申し訳ないのですが、ご見解はfunction側のcatchでthrowをする、というアイデアでしょうか?ご提示のようにして当該function呼び出し元に、PDOExceptionがまるっと戻るのでしょうか??
    function側で、PDOExceptionを継承するクラス(extendsする)を作っておいて、function内の・PDOExceptionのcatch時には その継承クラス丸っとfunctionの戻り値にする、とかでしょうか?functionが戻り値からしか挙動結果をうかがい知れないと思っていて。(getPreviousとかいうクラスがあるのはちょっと気になっているのですが)

    キャンセル

  • 2019/02/09 23:51

    ちょっと何言っているかわかりません。試してみれば?

    キャンセル

  • 2019/02/10 12:52

    throw new MyDatabaseException( $Exception->getMessage( ) , (int)$Exception->getCode( ) );
    をfunction側のcatch内に挿入しましたが、MyDatabaseExceptionが見当たらないみたいなコーディング上のエラーになってしまいます。
    呼び出しもとのcatchに入れるのでしょうか?

    functionの中で
    class MyDatabaseException extends PDOException とか記述するのでしょうか?

    ご想像以上に初心者で申し訳ないです

    キャンセル

  • 2019/02/10 15:26

    > functionの中で
    class MyDatabaseException extends PDOException とか記述するのでしょうか?

    うーん。多分 php の基礎的なところで学習が足りてないです。
    object とか class とか instance とかちゃんと学習しないとちゃんと書けないです。
    例外に関しては以下の資料を見てみると面白いです。
    https://speakerdeck.com/twada/php-conference-2016?slide=99

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.60%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る