サービスクラスから直接画面にreturnしたい

Question

LaravelでAPIを作っています。

Api呼び出し→ルーティング→コントローラー→サービスクラス→モデル、IF等→サービスクラス→コントローラー→API呼び出し元に値を返却できればと思いますが、
サービスクラス内でエラーが起きた場合、下記のパターン
Api呼び出し→ルーティング→コントローラー→サービスクラス→モデル、IF等→サービスクラス→API呼び出し元

サービスクラスのJsonレスポンスをAPI呼び出し元に返却可能でしょうか。

以上、宜しくお願い致します。

ソースがないとわからないとのことなのでサンプルです。

ルーティング

Route::post('/service', 'AbcController@select');

コントローラー

class AbcController extends Controller
{

    protected Abc $Abc ;

    public function __construct(Abc $abc)
    {
        $this->Abc = $Abc ;
    }

    public function select(Request $request)
    {

        $json = Service::sample($this->Abc, $request);
　　　　
　　　　return response()->json(json , 500, [], JSON_UNESCAPED_UNICODE);


    }

}

サービスクラス

class Service
{
    public static function sample(Abc $abc, $request)
    {
        //IDをキーに管理テーブルからレコード取得
        $def = $abc->findByIdXXXX($request->input('xxxx_id'));

        //例外キャッチされたか判定
        //コレクションならfalse、stringならtrue
        if(is_string($def) === false)
        {

            return $def;

        }
        Log::critical('', ['request' => $request], ['message' => $def]);

        //エラー
        //ここからAPI呼び出し元にJsonレスポンス返却したい
　　　　//通常リターンならばコントローラーに返るから
        return response()->json(null, 500, [], JSON_UNESCAPED_UNICODE);

    }
}

Comments

[m.ts10806]

実際のコードがないとなんとも言えないのでは。

Answer 1

レスポンスを返すのがコントローラーの役割なので
サービスクラスからはこれだけ。

return $abc->findByIdXXXX($request->input('xxxx_id'));

判定はコントローラーで。

$def = Service::sample();

if(is_string($def) === false) {
    return response()->;
} else {
    return response()->;
}

少し別の話。
そもそもエラーの判定がこれでいいのかとかサービスクラスに分けてる意味がないとかあるけど
一番まずいのはstaticメソッドで作ってる所。
とてつもなく迷惑かけることになるのでこれだけはやめてほしい。

public static function sample(Abc $abc, $request)

普通に作った上で

public function sample(Abc $abc, $request)

コントローラーではこう使うのがLaravelでの普通。

public function select(Request $request, Service $service)
{
    $json = $service->sample($this->Abc, $request);
}

Comments

[larachan]

staticだと迷惑をかけるのですね。。。ありがとうございました。
Serviceクラスにはビジネスロジックを書くとのことだったので、判定等も書くのかと思っていました。
ソースまでご丁寧にありがとうございました。