PHP連想配列同士を特定の値でgroup byして集計したい

2020/09/29 01:19

[ "type"=> 001, "kbn"=> "AAAA", "tax"=> 1, "amount"=> 100 ] こうですか？

yambejp

2020/09/29 01:23

そういう感じです。「001」は文字列なら"001"です

2020/09/29 01:35

すいません、その辺実データ手直しして投稿しているため適当です。申し訳ないです。それでよろしくお願いします。

yambejp

2020/09/29 01:40

例示されたソースを書き直してください

2020/09/29 01:43

配列データはDBから取得したものではないのですか？

2020/09/29 01:48

phper.k 配列というかDB:selectで取ってきたデータです色々と事情があって望んだ結果を既存のデータマージで行うことになりました sqlで書くことができたら楽ですが

2020/09/29 01:48

ソース変えました

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

回答2件

foreachで愚直に取る方法しか思い付かず、スマートに何かlaravelらしく取得することは可能でしょうか？

DB::select() で取得している以上、戻り値は array になるので、「laravelらしく」Collection クラスでは処理できない。

「laravelらしく」にこだわるなら、Eloquent で取得してください。

投稿2020/09/29 02:50

総合スコア3923

2020/09/29 03:55

ありがとうございます！ Eloquentだと簡単にきれいにできますねけど今回はarrayで結構です。laravelらしくという表記は消しておきます

2020/09/29 04:01

「foreachで愚直に取る方法」が分かっているなら回答は不要だと考えています。

2020/09/29 04:13

それが嫌なので、array_sum、array_filter　mapなどを使ってもう少しきれいになる方法はないか、という質問です。 foreachで愚直取る方法の投稿はもちろん不要です

2020/09/29 04:15

「array_sum、array_filter、array_columnなどを使ってうまくできないか」これについて取り組んだことを記載してください。

2020/09/29 04:16

あと、サンプルデータの提示をちゃんとしてください。

2020/09/29 04:20

分からないのであれば回答は結構です！方法は大体そんなところを使えば取れるかもしれないな、で例で書いてあるので。要は愚直な方法ではないスマートな方法があれば教えてくださいということです。サンプルは発言をわかりやすくするためだけの、例です。こういった感じでできないかな？とピンとこないのであればスルーしてください。こちらもそこに時間が裂けないので質問してるだけです。

2020/09/29 04:34 編集

あのねーなに逆ギレしてんの？コピペで使えもしないサンプルデータ提示して、他人に回答させるその無神経な態度はいかがなものかともいますよ時間がないなら foreach で実装すればいいやん

m.ts10806

2020/09/29 06:01 編集

テーブル定義もコードも提示されてないからなんとも言えないけど、データ取得時に使いやすい形で取得できるならそれに越したことはないのです。Laravelに限らずPHPに限らず。取得してから整形なり加工なりしようとしてる時点でスマートさは０。フレームワークが無用の長物になっただけ。「分からないなら」というけど、回答できるだけの情報が提示されてないので無理。時間ないならお金払ってやってもらうべきですね。質問して回答待ってる時間があるのが不思議。答えてもらえる保証無いし。時間ないなら幾らかやりとりの必要が発生する質問サイト使うのが間違い。

2020/09/29 06:15

サンプルが雑でロジックが全くわからないけど、foreach でできるならいじらなくていいのに。

https://qiita.com/mikakane/items/76ae73990bf1ece9e7ae

2020/09/29 17:09

``` $s = collect($hoge)->where('type')->groupBy('kbn')->map(function ($v) { return [ 'type' => $v[0]->type, 'kbn' => $v[0]->kbn, 'tax' => $v->sum('tax'), 'amount' => $v->sum('amount'), ]; })->values()->all(); ``` laravelのcollection使いましたー https://qiita.com/mikakane/items/76ae73990bf1ece9e7ae

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

自己解決

Laravelのcollection使って以下のようにしました

  　　　　　$array = array_merge($hoge,$fuga);
           $sum = collect($hoge)->where('type')->groupBy('kbn')->map(function ($v) {
                return [
                    'type'  　=> $v[0]->type,
                    'kbn'     => $v[0]->kbn,
                    'tax'     => $v->sum('tax'),
                    'amount'  => $v->sum('amount'),
                ];
            })->values()->all();

投稿2020/09/29 17:11

編集2020/09/29 17:14

総合スコア292

2020/09/29 21:05

これで質問に掲載された結果になりますか？

2020/10/01 05:38

なりました！ type,kbnでtaxとamountをサマリーできました。 foreachでやるのはちょっとなと思っていたのでLaravelのcollection使いました。

2020/10/01 06:50 編集

質問に「掲載された」結果ですよ？うちで試したところ、文法エラーで動きすらしませんが。それに一行目で作った $array はどこへ行ったんです？

2020/10/01 06:39

上記は例なので文脈で理解していただけると・・一行目で作った$arrayは collect($hoge)　が間違いで　collect($array)ですね。でも$hogeでも動きますよ。例を突っ込んでコピペしただけでは動かないので、試してみたければご自分で考えてやってみてください。よろしくお願いします。

2020/10/01 06:43

文脈からは「この回答を使えば質問の結果が得られる」と読めます。他の意味があるんでしょうか？

2020/10/01 06:48 編集

ms5025 さん雰囲気でこの回答で良しとするなら、今後回答者は雰囲気で回答してもいいのですか？「こんな感じ」と雰囲気で回答して、まともに動かないコードで回答しても、あなたはそれで問題を解決できるのですか？

m.ts10806

2020/10/01 07:38 編集

>例を突っ込んでコピペしただけでは動かないので、同じような問題を抱えた人がここに辿り着いて参考にしようとしても地雷を踏ませたいわけですね。「teratailの回答は参考にならない」と各所で言われる一端です。「あの回答者が」ではなく「teratailが」という評価になるのをお忘れなく。回答もそうですがそのあとのコメントや対応が無責任すぎます。引用したQiitaの記事の筆者にも悪影響となりえます。ひとまず地雷であると、そのように回答の冒頭に書いておいてください。勘違いして大嘘を信じてしまうとあなたより将来有望な可能性のある若者への損失、邪魔にしかなりません。