PDOExceptionのMVCモデルでの記述する場所がわからない。

Question

前提・実現したいこと

データベースの処理をするモデルのメソッドをコンローラで呼び出す場合、trycatch文はコントローラーで呼び出したモデルのメソッドを囲むべきか、あるいはモデルのメソッド内でtrycatchを記述すべきなのかわかりません。

該当のソースコード

class Register extends \MyApp\Controller
{
    public function controller()
  {
// コントローラー内で囲むやり方
    try {
      $hogeModel = new Hoge();
      $hogeModel->hoge();
    } catch (\PDOException $e) {
      echo $e->getMessage();
      exit;
    }
  }

class Register extends \MyApp\Model
{
    public function Hoge()
  {
// モデル内で囲むやり方
    try {
      $stmt = $this->db->prepare("SELECT * FROM users WHERE email = :email");
      $stmt->bindValue(':email', $email, \PDO::PARAM_STR);
      $stmt->execute();
      return $stmt->rowCount();
    } catch (\PDOException $e) {
      echo $e->getMessage();
      exit;
    }
  }

Answer 1

エラーが発生した場合Hoge()内で終了してしまうので
controller()側でtry-catchすることもないかと

Comments

[suika11]

> controller()側でtry-catchすることもないかと

ではコントローラー側でtry~catchするのが無難と考えてよろしいですか？

[yambejp]

極端な話、executeするところさえ囲めばよいでしょう
例えばbindするところまで処理したhogeが$stmtを返し、
controller内でexecuteするならcontrollerにtry-catchが必要です

[suika11]

> 極端な話、executeするところさえ囲めばよいでしょう

これはなぜですか？例外が発生するのはexecuteする時だけで、prepareやbindする時は発生しないということですか？

[yambejp]

PDOExceptionでつかめるエラーはsqlの文法や型違いなどのエラーです。
それ以外のエラーを例外処理したい場合は別の方法でエラーを拾います

try{
throw new Exception('テスト');
$stmt->execute([]);
}catch(PDOException $e){
die($e->getMessage());
}catch(Exception $e){
die($e->getMessage());
}

[suika11]

ありがとうございます。

> 極端な話、executeするところさえ囲めばよいでしょう
例えばbindするところまで処理したhogeが$stmtを返し、
controller内でexecuteするならcontrollerにtry-catchが必要です

これはコントローラー側でexeuteした方がいいということですか？

[yambejp]

> コントローラー側でexeuteした方がいい

いやそうではなくて、executeする場所に必ずtry-catchしてくださいということです
hogeでexecuteするならcontrollerにtry-catchはいらないですし
hogeが$stmtの設定までしかしないならcontroller側でtry-catchしてくださいということ。

executeをどちらでするかは設計思想次第です

[suika11]

わかりました、ありがとうございます。

Answer 2

正直なところ、自分で作ってるなら自分で決めるしかないですね。
参考にしたいのなら既存のフレームワークを研究してください。

Comments

[suika11]

ありがとうございます。

どちらにしようか悩んでいますが、自分のアプリの場合だと
モデル側でバインドした後にコントローラー側でexecuteした方が都合がいいことに気がつきました。
試行錯誤してみます。

[m.ts10806]

それは現在提示されている情報だけでは他者には判断できませんね。
結局自分で作ってるわけですし、オブジェクト指向含めてきちんとおさえられているなら質問するまでもない内容です（質問しなければならない＝基礎や考えが足りない　ということです）

[suika11]

わかりました。オブジェクト指向やそのほかの基礎を一度洗い直してみます。

[m.ts10806]

煽るわけでも見下すわけでもないんですが、「直す」ということは一度以上はちゃんと通したことがあるんですよね？
質問幾つか見た感じ、PHPマニュアルを参照している雰囲気が一切ないので、となると一度も通したことがなさそうに感じました。
書いたとおりにしか動かないので、「なんとなく」「雰囲気で」やっても何も出来上がりませんよ。

[suika11]

通す、というのはどういうことですか？

[m.ts10806]

「基本部分を一度でもきちんと」通す
です。
PHPマニュアル活用、デバッグ、エラーメッセージ読む

1つでもできてないなら「基礎ができた」とは言えません。

[suika11]

どれもできていませんでした。
マニュアルをきちんと読みます。

[m.ts10806]

「活用」です。
そして考えることまでがセット。

[suika11]

ありがとうございます。