try,catch文はクラスファイル、もしくは呼び出し側、どちらに書くべきか

try,catch文をどちらに書いた方が良いか、といった点でお聞きしたいことがございます
以下のコードは、クラスファイルのtest_class.phpになります。

php
1
2<?php
3class test
4{
5
6  private $dbh;
7
8  public function  __construct($dsn,$user,$password){
9    $dbh = new PDO($dsn,$user,$password);
10    $dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
11    $dbh->setAttribute(PDO::ATTR_EMULATE_PREPARES,false);
12    $this->dbh = $dbh;
13  }
14
15  public function test($name,$password){
16    try{
17      $stmt = $dbh->prepare("INSERT INTO data(name,password)VALUES(:name,:password");
18      $stmt->bindValue(':name',$name,PDO::PARAM_STR);
19      $stmt->bindeValue(':passwrod',$password,PDO::PARAM_STR);
20      $stmt->execute();
21    }catch(PDOException $e){
22      echo $e->getMessage();
23    }
24    
25  }
26}

以下は、test.phpになります。

php
1<?php
2$test = new test('mysql:host=localhost;dbname=test','root','');
3$name = $_POST['name'];
4$password = $_POST['password'];
5try{
6  $shop->test($nama,$password);
7}catch(PDOException $e){
8  echo $e->getMessage();
9}

上記２つのファイルにおきまして、クラスファイルに、try,catch文を書いた方が良いか、それとも呼び出し側に書くべきか、迷っております。
どちらに書いた方が良いか、ご助言頂ければ幸いです

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

回答1件

ベストアンサー

エラーが発生した場合をキャッチしたいところにつけてください

投稿2019/04/25 01:47

yambejp

総合スコア114843

toll_tree

2019/04/25 02:06

ご回答ありがとうございます。ご質問に掲示しました、クラスファイル、メソッド呼び出し側のファイルどちらに、try,catch文を記載したとしても、PDOExceptionエラーが発生した場合の挙動は、同じになるのでしょうか？

yambejp

2019/04/25 02:12

エラーになった箇所がtryで囲まれていればエラーをthrowしてもらえますしそうでないなら無視されるだけです

toll_tree

2019/04/25 02:18

つまり、呼び出し側、クラス側、どちらに記載してもエラーが発生すれば、$e->getMessage()の内容が出力されるということでしょうか？

退会済みユーザー

2019/04/25 02:39

程度問題だと思うけど、try～catchの中に大量にデータベースアクセス処理を突っ込んで、いざ例外が発生したときに問題箇所が特定できれば広く捉えてもいいし、ラクに捉えるには狭く囲ってなんどもtry～catchブロックを置いて例外発生箇所を捉えやすくするのもよし。王道なし。

toll_tree

2019/04/25 02:47

>m6uさん補足ありがとうございます。そうでしたか。自分は、王道のような方法があるのかと、思っていたのですが、そうではないということなんですね。

yambejp

2019/04/25 02:51 編集

逆に運用面ではエラーなんてユーザーは見えないほうがよいのでエラーが発生したら「ごめんなさい、管理者に連絡してね」ページに飛ぶなどを記載することになります。エラーでメッセージを出しているうちはデバッグ環境もしくは個人用（自社用）のレベルです。デバッグ中はなんでもかんでもtry-catchして構わないので怪しい箇所は一通り囲むのが王道じゃないですかね？最近のphpはアサーションが強化されていますので運用前試験はもっと違うアプローチもありでしょう

toll_tree

2019/04/25 03:54

そうなんですね。となりますと、実際に運用する際には、try,catch文などは、コードから取っ払ってしまうような感じでしょうか？

yambejp

2019/04/25 04:15

ちゃんと読んでください別ページに飛ばすしてエラー表示させないようにするということです

toll_tree

2019/04/25 05:02 編集

なるほど...エラーが起こった時専用のページを用意するということなんですね...

yambejp

2019/04/25 05:11

質問者さんが意識していないだけで403,404,500などエラーを返すページはサーバー側に用意されています。またこれも前述しましたエラーを返さないという選択肢もあります