if文での例外処理をtry catchで書くとどうなるのか。CakePHP

Question

皆さんお疲れさまです。

rikさん昨日の投稿に回答ありがとうございました。
Cakephpでの例外処理(try catch)の書き方を教えてください

それで、昨日回答して頂いて

         //条件分岐による例外処理
         if ($this->Report->save($data) === false) {
             //エラーメッセージやその後の処理
             throw new BadRequestException('コメントの更新に失敗しました');
         }

こういう風に書くとよい、というのは分かったのですが、

try catchで書く場合にはこれはどう書いたらよいのでしょうか？

try{
    $this->Report->save($data) === false;
}catch(Exception $e){
    throw new BadRequestException('コメントの更新に失敗しました')
}

こうでしょうか？
tryのカッコの中にもっと何か足す必要があるのでしょうか？
PHP:例外
これを見てもよくわかりませんでした。

どなたか教えてください！お願い致します！

Answer 1

catchした中でthrowするあたりが微妙というか何がしたいのかわかりません。
通常のExceptionが飛んできた場合BadRequestExceptionとして処理したいという事でしょうか？
そうなるとかなり乱暴な処理だと思うので構造を考え直した方が良いと思います。

    if ( $this->Report->save($data) === false ) throw new BadRequestException('コメントの更新に失敗しました');

こうする事によってBadRequestExceptionを投げてキャッチしてくれます。
throwの前に何か処理を書きたいなら

if ( $this->Report->save($data) === false )
{
    //ここに何か処理
    throw new BadRequestException('コメントの更新に失敗しました');
}

このようにした方が良いと思います。

そもそもステータスコード400番のBadRequestExceptionをここでthrowする意味も全くわかりません。

Comments

[amakusa]

回答ありがとうございます。
本当によくわからずに投稿していました、すみません。
上記のように書いた場合はもうtry catch文を書く必要はないのでしょうか？
その文だけでexceptionをthrowしてcatchもしてくれるという認識であっていますか？

[nanndemoiikara]

CakePHPがBadRequestExceptionをキャッチしてくれるはずです。
CakePHPに明るくはないので、Exceptionはキャッチしてくれるかはわかりません。
たぶんキャッチしないんじゃないかな？
BadRequestExceptionはExceptionをextendして作成された例外です。

Answer 2

reportMain()を呼び出してデータ保存を行うとします。

try catch を使用する場合はこの様になります。

private function reportSave($data) {
	if ($this->Report->save($data) === false) {
		throw new Exception('コメントの更新に失敗しました');
	}
}

private function reportMain() {
	//ここまでに$data用意

	//保存処理
	try {
		$this->reportSave($data);
	} catch (Exception $e) {
		//$e->getMessage() で 'コメントの更新に失敗しました' が得られます。
		//エラー処理
		return;
	} finary {
		//なにかあれば処理。この例ではfinaryは不要
	}
	//正常な場合のその後の処理
}

try catch を使用しない場合はこの様になります。

private function reportMain() {
	//ここまでに$data用意

	//保存処理
	if ($this->Report->save($data) === false) {
		//'コメントの更新に失敗しました'の処理などエラー処理
		return;
	}
	//正常な場合のその後の処理
}

この様に、try catch を使用するメリットがありません。
処理が分断されて分かりにくくなるだけです。
もしreportSave()で色々な前準備や後処理を含めて行わせるのであれば意味が出てきますが、今のところそのような必要性を感じたことが有りません。

Comments

[amakusa]

わざわざありがとうございます！
これだけコード量に差が出るのですね。
確かにtry catchのメリットをあまり感じませんね。

//'コメントの更新に失敗しました'の処理などエラー処理
のところは
throw new Exception('コメントの更新に失敗しました');
とか入れておけばいいでしょうか。

[rik]

それは違います。

CakePHPの場合、

$this->Session->setFlash('コメントの更新に失敗しました');
$this->redirect(array('controller'=>'Homes','action'=>'index'));
return;

などです。
エラーメッセージをセットして、
ホーム画面や目的の画面に遷移させます。

View内では
$this->Session->flash();
でブラウザに表示されます。

このあたりは自由に記述できます。try catch や throw に関する記述はここではしません。
CakePHP参考書通りの記述になります。

[rik]

if ($this->Report->save($data) === false) {
　　//エラー時の記述
} else {
　　//正常時の記述
}
これでも大丈夫です。

[amakusa]

ありがとうございます！！！

Answer 3

phpの例外処理については公式サイトに詳しく書かれています。
http://php.net/manual/ja/language.exceptions.php

Comments

[amakusa]

再度回答ありがとうございます。
じっくり読みます。