[FuelPHP]例外をcatchした後で配列を作ると __toString() must not throw an exception

Question

お世話になっております。
FuelPHPでAPIをRequest_Curlで実行するアプリを作っているのですが、例外発生時のハンドリングで困っております。

PHP 5.6.30
FuelPHP 1.7

try{
  // ～省略 ～
  $curl = Request::forge('http://no-exist-domain/info', 'curl');
  $result = $curl->execute();  // この実行を失敗させる

} catch(Exception $e) {
  $result = array(
    'code' => $e->getCode()     // エラーコードを返したい
  )
}

return $result;  //成功時はCurl実行結果、失敗時はエラーコードを返したいとする

端折りましたが、おおむね上記のような実装としました。
これをexecuteが失敗するように、例えば存在しないURIへリクエストを送ったりすると、
$e->getCode()
で404エラーは取れるのですが、catchの中で配列にすることができず、下記のエラーとなります。

ErrorException [ Fatal Error ]:
Method Fuel\Core\Response::__toString() must not throw an exception

DOCROOT/index.php @ line 0

エラーの原因と解決策、またはエラーコードを返せるような方法があればご教示いただきたく存じます。

基本的な例外処理の話かもしれませんが、よろしくお願いいたします。

Answer 1

検証していないので予想ですが、$resultをreturnした先でどのように扱っているのかが問題なんだと思います。

成功時には$result はRequest_Curlクラスのオブジェクト
失敗時には$result は配列
でreturnされます。同じ種類のデータじゃないので注意です。

失敗した場合でもreturn先でResponseを取り出そうとしてるコードを書いているような気がします。

Comments

[yoshiky]

早速のご回答、ありがとうございます！
まさにご指摘の通りでした。

現在、仮実装なので返り値をブラウザに返すようにするため、呼び出し元のcontrollerにて
return するところまで$resultを渡しています。配列をreturnでブラウザに返そうとしたのが悪かったようです。
当該部分のreturn を試しにjson_encodeなど変換したら、エラー無く動きました！

大変助かりました。勉強になりました。
ありがとうございました。