Q&A
スペルミスがかなり目に付いてしまうので修正していただければと。
Larvel larabel ではなくLaravel
前提・実現したいこと
LaravelのCollection同士を結合
（イメージとしてはDB left join）
そのコマンドがあるかどうか？
データベース（案）
Collection出力の元となるデーターベースです。
users
| id | name |
|---|---|
| 1 | 名前１ |
| 2 | 名前２ |
| 3 | 名前3 |
titles
| id | name |
|---|---|
| 1 | タイトル1 |
| 2 | タイトル2 |
| 3 | タイトル3 |
items
| id | title_id | name |
|---|---|---|
| 1 | 1 | アイテム1 |
| 2 | 1 | アイテム2 |
| 3 | 1 | アイテム3 |
| 4 | 2 | アイテム4 |
| 5 | 2 | アイテム5 |
| 6 | 2 | アイテム6 |
| 7 | 3 | アイテム7 |
| 8 | 3 | アイテム8 |
| 9 | 3 | アイテム9 |
datas
| id | user_id | title_id | item_id |
|---|---|---|---|
| 1 | 1 | 1 | 2 |
| 2 | 1 | 2 | 5 |
| 3 | 1 | 3 | 8 |
以上のデータベースから
datasより名前１（user_id 1）のデータを以下のように出力し
結果
php
1$collection = collect([ 2 ['id' => 1, 'title' => 'タイトル1', 'value' => 2,'name' => 'アイテム2'], 3 ['id' => 2, 'title' => 'タイトル2', 'value' => 5,'name' => 'アイテム5'], 4 ['id' => 3, 'title' => 'タイトル3', 'value' => 8,'name' => 'アイテム8'] 5]);
以上の構成であればデーターベースでやれば簡単なのですが、
それをCollectionで似たようなことが、できないかというのが今回の質問の意図です。
また理由の１つに、上記のテーブルが一部jsonデータで保存されていた場合にどのようにすればいいかと方法に悩んでしまいました。
Collectionであれば以下の方法でjsonデータをcollect化できますので。。
collect(json_encode(jsonデータ))
質問
２つのCollectionを結合して１つのCollectionデータを作成を考えています。
php
1$collection_a = collect([ 2 ['id' => 1, 'value' => 2], 3 ['id' => 2, 'value' => 5], 4 ['id' => 3, 'value' => 8], 5]); 6 7$collection_b = collect([ 8 ['id' => 1, 'title' => 'タイトル1', 9 'items'=>[ 10 ['id' => 1, 'name' => 'アイテム1'], 11 ['id' => 2, 'name' => 'アイテム2'], 12 ['id' => 3, 'name' => 'アイテム3'] 13 ] 14 ], 15 ['id' => 2, 'title' => 'タイトル2', 16 'items'=>[ 17 ['id' => 4, 'name' => 'アイテム4'], 18 ['id' => 5, 'name' => 'アイテム5'], 19 ['id' => 6, 'name' => 'アイテム6'] 20 ] 21 ], 22 ['id' => 3, 'title' => 'タイトル3', 23 'items'=>[ 24 ['id' => 7, 'name' => 'アイテム7'], 25 ['id' => 8, 'name' => 'アイテム8'], 26 ['id' => 9, 'name' => 'アイテム9'] 27 ] 28 ] 29]);
以上の２つのCollectionを以下のように出力を考えています。
php
1$collection_c = collect([ 2 ['id' => 1, 'title' => 'タイトル1', 'value' => 2,'name' => 'アイテム2'], 3 ['id' => 2, 'title' => 'タイトル2', 'value' => 5,'name' => 'アイテム5'], 4 ['id' => 3, 'title' => 'タイトル3', 'value' => 8,'name' => 'アイテム8'] 5]);
foreachで回す方法しかないのでしょうか？
まわりくどい方法は思いつくのですが
Laravel自体はまだ始めたばっかで適切なコマンド・方法があるかと思った次第です。
リファレンス読んだ感じ、色々駆使すればできそうですけど
https://readouble.com/laravel/5.7/ja/collections.html
まだ試してないので調整してみます。
要件確認です。
collect()されたものをマージということですよね？
提示されている配列は仮のものとは思いますが、大目的が別にあるのでしたらそれもあった方がよいかもしれません。
～～～最後1行余計でした。編集削除しました。
おっしゃる通りです。あとでcollectという面で言いますと元々、collection_a,bはEloquentでDBから出力したイメージになります。なのでDB出力時にcollection_cに調整してやればとも思います。
データベース部分も含めて全体イメージを質問に追加します。
回答2件
0
探すのはcollectionにお任せするとしても、やっぱり大元をは自分で回さざるを得なさそうな・・
php
1$collection_c = $collection_a->map(function($_a,$_)use($collection_b){ 2 $_a['title'] = $_a['name'] = null; 3 4 //コレクションBから、b.id=a.idを探す 5 if($_b = $collection_b->where('id', $_a['id'])->first()){ 6 $_a['title']=$_b['title']; 7 //Bの中のitemsもcollectionにして、そこからb.items.id = a.valueを探す 8 $_items = collect($_b['items']); 9 if($_b_item = $_items->where('id', $_a['value'])->first()){ 10 $_a['name'] = $_b_item['name']; 11 } 12 } 13 return $_a; 14});
自分でforeachの入れ子を沢山書くよりはマシなのかなーという感じでしょうかねぇ^^;
投稿2019/02/13 07:19
総合スコア1910
すみません、解答いただいて自分で調べて解答書いてた間にアンサーいただいてました。
結局foreachの入れ子なのでそもそも構成が甘かったようです。
1人で開発しているので他の方の意見がお聞きできて嬉しいです。
ありがとうございます。
0
ベストアンサー
あまり美しくないやり方かもしれませんが。（結局foreach使ったし・・・）
php
1$collection_c=$collection_a->map(function ($item_a,$key_a) use ($collection_b){ 2 $collection_b->map(function ($item_b,$key_b) use (&$item_a){ 3 if($item_b['id'] === $item_a['id']){ 4 foreach($item_b['items'] as $b_collect){ 5 if($b_collect['id'] === $item_a['value']){ 6 $item_a['title']=$item_b['title']; 7 $item_a['name']=$b_collect['name']; 8 return false; 9 } 10 } 11 } 12 }); 13 return $item_a; 14}); 15var_dump($collection_c->all()); 16 17/* 18 19array(3) { 20 [0]=> 21 array(4) { 22 ["id"]=> 23 int(1) 24 ["value"]=> 25 int(2) 26 ["title"]=> 27 string(13) "タイトル1" 28 ["name"]=> 29 string(13) "アイテム2" 30 } 31 [1]=> 32 array(4) { 33 ["id"]=> 34 int(2) 35 ["value"]=> 36 int(1) 37 ["title"]=> 38 string(13) "タイトル2" 39 ["name"]=> 40 string(13) "アイテム1" 41 } 42 [2]=> 43 array(4) { 44 ["id"]=> 45 int(3) 46 ["value"]=> 47 int(3) 48 ["title"]=> 49 string(13) "タイトル3" 50 ["name"]=> 51 string(13) "アイテム3" 52 } 53} 54*/
下記のような記事も参考にしてみてもいいかもしれません。
投稿2019/02/13 05:20
編集2019/02/13 05:23総合スコア80888
ありがとうございます。頂いたコードを元に見直してみました。
$collection_c=$collection_a->map(function ($item_a,$key_a) use ($collection_b){
$find_collection = collect($collection_b->firstWhere('id','===',$item_a['id']));
$find_item = collect($find_collection->get('items'))->firstWhere('id','===',$item_a['value']);
$item_a['title'] = $find_collection->get('title');
$item_a['name'] = $find_item['name'];
return $item_a;
});
$collection_c->dump();
foreach,ifをcollectのfirstWhereに変更して見た目だけスッキリしました。
内部処理的はよりまわりくどく同じ事をしていますのでなんとも言えませんが・・・
構成の見直しのイメージになりました。ありがとうございます。
そうですね。Laravelっぽい作りになったと思います（別回答のほうが綺麗ですね。。）
あなたの回答
tips
プレビュー
