laravel5.2 依存注入について

Question

FatなControllerを避けるためサービスクラスを作っています。
Appの直下にServicesというフォルダを作り、そこにPhotoPublish.phpファイルを作成しました。
※UserがDB(publish_lists)へ任意で登録した画像（Path）を吐き出す処理を書いております。
お気に入りの写真を他のUserに一般公開するイメージを持って頂ければと思います。

<?php

namespace App\Services;

class PhotoPublish
{
    public function publish(PhotoPublish $request)
    {

		$published_photos = DB::table('publish_lists')
            ->join('users','users.id', '=', 'publish_lists.user_id')
            	・
            	・
            	・
            ->get();

    	return json_encode( $published_photos, JSON_UNESCAPED_SLASHES );

    }


}

質問は、2つあります。
1.現在サービスクラスにある$published_photosをcontrollerへ注入する方法についてです。
※最終的に下記のような値をViewへ送りたいです。

return view('sampl.sampleview')->with('published_photos',$published_photos);

2.そもそもserviceクラスの書き方（概念、作り方）に問題はないか。

よろしくお願いいたします。

Answer 1

<?php

namespace App\Http\Controller;

use App\Services\PhotoPublish;


class HogeController extends Controller {

    public function __construct(PhotoPublish $photo_publish)
    {
        $this->photo_publish = $photo_publish;
    }

    public function getHoge()
    {
        // PhotoPublishクラスのpublishメソッドの呼び出し
        $published_photos = $photo_publish->publish();
        return view('sampl.sampleview')->with('published_photos', $published_photos);
    }
}

こんな感じかと思いますー

__constructメソッドに関して

Comments

[twin_bird]

2に関してですが、今回のようなデータを取得するロジックはモデル側に実装した方がよいのでは・・？という印象です。
サービスクラスをどういう定義でやるかは人それぞれかと思いますが、一つのコントローラでしか使わないような処理をサービスクラスに分けるのではなく、複数のコントローラーで使うような処理をサービスクラスに任せるのがより効率的な気がします。
例えば、画像を外部ストレージに保存する処理だったり、通知を行う処理だったり（Laraveだと通知は他に便利な方法がありますね）・・

これについては、他の経験豊富な方の意見も参考にしたいところです。