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

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

ただいまの
回答率

90.34%

  • PHP

    21323questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

エラーハンドリングのベストプラクティス

解決済

回答 2

投稿

  • 評価
  • クリップ 1
  • VIEW 1,293

hdmi

score 1

PHPで例外処理を書いているのですが、
どのように書くのが良いか悩んでいます。
例えば、以下のCode1とCode2ではどちらのほうが良いでしょうか?

Code1では、メインの関数とは別の関数内でthrowして、
それをメインの関数でcatchしています。
Code2では、メインの関数で別の関数からの結果を受け取って、
メインの関数内でthrowしています。

エラーハンドリングのコードを書き慣れてないので、
その他、このように書くのが良いなどあればお教えいただければ助かります。

 Code1
class Checkout
{
    public function main()
    {
        try {
            $this->saveOrder();
            $this->saveOrderDetail();
            // ...
        } catch (Exception $e) {
            // 例外処理
        }
    }

    public function saveOrder()
    {
        // ...
        if (Order::save()) {
            throw new \Exception('注文の保存に失敗');
        }
    }

    // ...
}
 Code2
class Checkout
{
    public function main()
    {
        try {
            if ($this->saveOrder()) {
                throw new \Exception('注文の保存に失敗');
            }

            if ($this->saveOrderDetail()) {
                throw new \Exception('注文詳細の保存に失敗');
            }
            // ...
        } catch (Exception $e) {
            // 例外処理
        }
    }

    public function saveOrder()
    {
       // ...
       return Order::save();
    }

    // ...
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+2

PHPで例外処理を書いているのですが、 どのように書くのが良いか悩んでいます。

ご質問内容はPHP固有の内容ではありませんが、PHPを使った例で回答させていただきます。

例えば、以下のCode1とCode2ではどちらのほうが良いでしょうか?

どちらか選べと言われればCode1です。

その他、このように書くのが良いなどあればお教えいただければ助かります。

まず例外については、設計が必要な上、奥が深い話なのでteratailでパーフェクトな回答は出てこない(出せない)と思ってください。きれいなハンドリングをしたい場合は、例外処理の手法より前に、例外の設計から勉強してみてください。

ここでは、基本的なポイントに絞って回答させていただきますのでご参考にしてみてください。
また、MVCというデザインパターンをご存知の前提で記載させていただきますのでご容赦ください。(ご存知でない場合はお調べください)

アプリケーション開発者が意識すべき例外
  • フレームワークを利用している場合
    ビジネスルール違反の例外
  • フレームワークを利用していない場合
    上記に加え、一般的なフレームワークがハンドリングしている予期せぬ例外(不正リクエスト時の例外等)
例外のハンドリングについて

ここではビジネスルール違反の例外にフォーカスして記載します。
まず、ビジネス例外の際にはExceptionクラスをそのままthrowしないでください。
以下例のようにExceptionクラスを継承して、ビジネスルールの違反単位に例外を作成されることをおすすめします。

// 注文に関する例外を定義
class OrderException extends Exception {}

// 以降は細かくハンドリングしないのであれば不要
// 在庫が不足した際の例外
class StockNotEnoughException extends OrderException {}
// 注文金額の制限を超過した際の例外
class OrderAmountExceededException extends OrderException {}
// ・・・

次にMVCのModelと言われる部分にビジネスロジックを記載していきます。(質問者様のコードを流用しています)
※変数やメソッド名等は日本語にしています

class Checkout
{
    public function saveOrder($order)
    {
        if (在庫 < 注文個数) {
            throw new StockNotEnoughException(メッセージ)
        }
        if (最大注文金額 < 注文金額) {
            throw new OrderAmountExceededException(メッセージ)
        }

        try {
            注文をデータベース等に保存
        } catch (Exception $e) {
            throw $e
        }
    }
}

そしてリクエストをビジネスロジックにマッピングするコントローラでは以下のようにします。

class OrderController
{
    public function main()
    {
        try {
            Checkout->saveOrder()
        } catch (OrderException $e) {
            // エラーメッセージをもとの画面に表示
        } catch (Exception $e) {
            // システムエラー画面を表示
        }
    }
}

以上、上記が100%正解というわけではないですが、なにかヒントを掴んでいただければ幸甚です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

+1

基本的にはCode1の方がいいと思います。
エラーによって、普通ではない状態が発生したわけなので、その時点で処理を中断して、呼び出し元に例外を返して上げるべきです。
また、例外のスタックトレースに、最初にエラーが発生したコードが記録できる点でもCode1の方がよいでしょう。

※ただし、Order::save())が失敗するケースが、例外的ではなく、正常系でも発生するのであれば、Code2の方がいいかもしれません。(とはいえ、呼び出し元でthrowするのであれば、やっぱり例外的なケースだと思うので、Code1が正しいのかも)

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

  • PHP

    21323questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。