laravel DBクエリについて

Question

フローとして、

1.ordersテーブルから全件取得。
2.全件取得したordersのuser_idを配列で取得
3.配列で取得したuser_idをforeachで絞り込み、再度ordersテーブルから取得。
4.3.のデータを使用して処理を実行

ordersテーブル
 id   //primaryKey
 user_id
 created_at
 updated_at

//1
$orders = DB::table('orders')->get();
//2
$userIds = $orders->map(function ($item) {
            return $item->user_id;
        })->unique()->toArray();
//3
foreach( $userIds as $userId) {
	$ordersByUserId = DB::table('orders')->where('user_id', '=', $userId)->get();
        //4  ~省略~
          .
          .
          .
}

1.と3.の処理が無駄にDBへアクセスしている状態になってしまっています。
1.で全件取得したデータから同じ形のデータを作成するような処理に変更できないでしょうか。
最終的には3.のデータ($ordersByUserId)を使用したいです。

情報不足等ありましたらご指摘をよろしくお願いします。

Answer 1

もちろん今の処理は全部間違ってるので
LaravelならまずEloquentとリレーションを調べる。
ordersからではなくusersから始めて
「このUserの持つOrder」という形でデータを取って来れる。

//一つ以上のorderを持つユーザー
$users = User::has('orders')->get();

foreach($users as $user){
    $orders = $user->orders();
}

Comments

[yuuuui]

ありがとうございます。リレーションは行なっていたので、
上記で取ってこれることを確認取れました。
ベストアンサーとさせていただきます。

Answer 2

実際に確認したわけではなく申し訳ないのですが...
Eloquentを利用して、以下のような感じでいけないでしょうか。

$orders = Order::whereHas('user', function ($query) use ($userId) {
    $query->where('id', $userId);
})->get();

ご参考: 存在するリレーションのクエリ

Comments

[yuuuui]

ご回答いただきありがとうございます。
whereHasというのもあるのですね、勉強になりました。
今回はwhereHasでは期待する動作をしなかったのですが、別の場所で使用できそうでしたので
活用させていただきます。
ありがとうございました。

[Lulucom]

要件を勘違いしていたようです、失礼しました。

[yuuuui]

いえ、私の説明不足でしたのでこちらこそすみませんでした。
ありがとうございました。

Answer 3

これは結局「同じuser_idは1レコードとみなす」ようにしたいということでしょうか？groupBy()とかでできそうに思います。

$orders = DB::table('orders')->groupBy('user_id')->get();

Comments

[yuuuui]

私の説明不足ですみません。groupBy()では期待する動作をしませんでした。
でも今まで使ったことがなかったので、勉強になりました。
ありがとうございました。

[m.ts10806]

なるほど了解です。
質問する際はコードより要件部分を充実させた方がいいかもしれませんね。
冒頭にあるのはあくまで「自分がやってみた手順」なので、「本来の大目的部分、前提」辺りがきっちりあれば適切な回答がつきやすくなります。

[yuuuui]

そうですね、おっしゃる通りです。
きちんと前提も含めて質問しようと思います。