Laravelのリポジトリパターン

Question

Laravelのリポジトリパターンについて勉強中です。
わからないことがあるのでご教授下さい。

以下のようなスキーマがあるとします。

User - id, user_name, address, created_at, updated_at

例えば、UserEloquent.getUsername()で全レコードを取得した後、user_nameカラムだけを配列に格納してコントローラに渡したい場合、user_nameだけを変数に格納するループ処理をUserEloquent.getUsername()で書いてしまってよいのでしょうか？

別途UserServiceクラスを作り、その中でforeachで回す･･･という処理はデザインパターンとしておかしいでしょうか？

Answer 1

Laravel自体には、今のところリポジトリーパターンはありません。Laravelで応用する方法が書籍やWeb上の投稿で紹介されています。基本的には「直接DBクラスやORMクラスを使っちゃうと、テストの時にダミーをかませられないし、DBエンジンの変更も面倒なので、一枚かませましょう。」という考えです。

Laravelを使うにせよ、別のものにせよ、置き換え可能にし、必要なメソッドを明らかにするため、インターフェイスクラスを定義し、実装は別のクラスにするのが常套手段です。

その実装を見せないようにするのが目的なため、内部をどう実現するかは、自由です。

もうひとつのポイントは、localdiskさんが書いているように、リターンする構造体は一般的なもの、大抵の場合は配列になるでしょうが、を使うことです。もし、Eloquentのall()などが返すEloquentモデルのコレクションを返してしまえば、Eloquentに依存してしまいます。そんなことをすれば、他のDBエンジンに切り替えたり、テスト用のダミー（スタブ）を作成する場合、Eloquentに変換しなくてはなりません。一般的なオブジェクト構造で返しましょう。

こちらからは、以上です。

Answer 2

やりたいことが users テーブルからすべての user_name を取得して配列化するというだけなら、

User::all(['user_name'])->toArray();

で OK ですよ。UserService や UserRepository などのクラスを作る必要はありません。