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

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

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

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

Code1

lang
1class Checkout
2{
3    public function main()
4    {
5        try {
6            $this->saveOrder();
7            $this->saveOrderDetail();
8            // ...
9        } catch (Exception $e) {
10            // 例外処理
11        }
12    }
13
14    public function saveOrder()
15    {
16        // ...
17        if (Order::save()) {
18            throw new \Exception('注文の保存に失敗');
19        }
20    }
21
22    // ...
23}

Code2

lang
1class Checkout
2{
3    public function main()
4    {
5        try {
6            if ($this->saveOrder()) {
7                throw new \Exception('注文の保存に失敗');
8            }
9
10            if ($this->saveOrderDetail()) {
11                throw new \Exception('注文詳細の保存に失敗');
12            }
13            // ...
14        } catch (Exception $e) {
15            // 例外処理
16        }
17    }
18
19    public function saveOrder()
20    {
21       // ...
22       return Order::save();
23    }
24
25    // ...
26}

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

回答2件

ベストアンサー

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

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

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

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

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

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

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

#####アプリケーション開発者が意識すべき例外

フレームワークを利用している場合

ビジネスルール違反の例外

フレームワークを利用していない場合

上記に加え、一般的なフレームワークがハンドリングしている予期せぬ例外（不正リクエスト時の例外等）

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

PHP
1// 注文に関する例外を定義
2class OrderException extends Exception {}
3
4// 以降は細かくハンドリングしないのであれば不要
5// 在庫が不足した際の例外
6class StockNotEnoughException extends OrderException {}
7// 注文金額の制限を超過した際の例外
8class OrderAmountExceededException extends OrderException {}
9// ・・・

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

PHP
1class Checkout
2{
3    public function saveOrder($order)
4    {
5        if (在庫 < 注文個数) {
6            throw new StockNotEnoughException(メッセージ)
7        }
8        if (最大注文金額 < 注文金額) {
9            throw new OrderAmountExceededException(メッセージ)
10        }
11
12        try {
13            注文をデータベース等に保存
14        } catch (Exception $e) {
15            throw $e
16        }
17    }
18}

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

PHP
1class OrderController
2{
3    public function main()
4    {
5        try {
6            Checkout->saveOrder()
7        } catch (OrderException $e) {
8            // エラーメッセージをもとの画面に表示
9        } catch (Exception $e) {
10            // システムエラー画面を表示
11        }
12    }
13}