foreachで二つのテーブルから一つのテーブルを作りたい。

こんにちは！

只今、簡単なシフト機能を作っています。
いつ、誰が、どのシフトに入るかのshiftテーブルをつくる必要があります。

人数が増えたとき
二重のforeachが必要だと思っています。

・１人が全部のシフトに這入る場合の手書き

//ユーザー一人が全部のシフトに入る
$dates = Date::all();
$user = User::first();
$user_id = $user->id;
$date_array = [];

foreach($dates as $date){
    $shift = new Shift;
    $shift->date_id = $date->id;
    $shift->user_id = $user_id;
    $shift->save();
}

//ユーザー二人が全部のシフトに入る

$dates = Date::all();
foreach($dates as $date){
    $shift = new Shift;
    $user  = User::find(1);
    $shift->date_id = $date->id;
    $shift->user_id = $user->id;
    $shift->save();
    $shift = new Shift;
    $user  = User::find(2);
    $shift->date_id = $date->id;
    $shift->user_id = $user->id;
    $shift->save();
}

のような形で書けます。

この場合人数が増えるとコードがいっぱいになるのでforeachで書きたいと思っています。
foreachの二重に挑戦してみました

        $dates = Date::all();
        $users = User::all();
        foreach($dates as $date){
            $shift = new Shift;
            $shift->date_id = $date->id;
            foreach($users as $user) {
                $user_id = $user->id;
                $user  = User::find($user_id);
                $shift->user_id = $user->id;
            }
           $shift->save();
        }

user_idに２しか入ってこず、しっかりforeachが回っていないみたいです。
何日間も悩んでいますが解決しなくて質問しました！
よろしくお願いいたします。

m.ts10806

2020/02/19 23:43

foreachではなくjoinで一発でとれるのでは？SQLは組んでみましたか？

sugawa

2020/02/19 23:46

SQLは組んでいません。コントローラー側の処理の一部なのでコントローラー内で処理したいです

m.ts10806

2020/02/19 23:56

いえ、どうみてもモデル側で済む処理ではないでしょうか。DB側から取得時に出力したい形でデータ取得すればロジックもスッキリしてパフォーマンスにも良い影響があります。どうしてもというのでしたらご自由に、ですけど。

sugawa

2020/02/19 23:59

すみません知識不足でした。モデル側の処理なんですね。モデルでSQL書くのですかね？

m.ts10806

2020/02/20 00:55

そもそもjoinで通じてますか？それ次第かと。結局のところshiftテーブルにある各idから名称なり明細を取得したいという風に見えるのですが。

sugawa

2020/02/20 01:07

joinはさっきまで知らなかったので調べないといけない状況です。やりたい事はそれです！

行動規範の内容に同意します

回答1件

ベストアンサー

まず「とりたい結果が得られるSQL」を作りましょう。
余程の事情がない限り、画面に表示したい形でデータがとれればアプリケーションサイドで色々こねくり回す必要がなくなるのでそれが最も良いはずです。

今回だとたぶんこう(未検証)

SQL
1SELECT
2   s.id as shift_id
3  ,s.user_id
4  ,u.name as user_name
5  ,s.date_id 
6  ,d.year
7  ,d.mdw
8FROM
9  shifts s
10INNER JOIN
11  users u
12  ON u.id=s.user_id
13INNER JOIN
14  dates d
15  ON d.id=s.date_id