Re: RFC [Discussion] array_column results grouping

Hendra Gunawan IMHO your suggestions is not related to array_group(), you are talking about a great change in PHP as mentioned by Larry, since we can have `Array($input, group(), reducer(), mapper(), filter() ..... and so forth);` or even any other way to handle the list comprehensions. but for array_group I think that we should keep it to the minimal, to only group array. ~Hassan Ahmed On Wed, Dec 8, 2021 at 1:26 AM Hendra Gunawan <the.liquid.metal@gmail.com> wrote: > > > > > And if you want to map, reduce, or filter without grouping? Then you can't > > really use this function. And as noted, the order in which you apply those > > operations may matter, and no order is necessarily more obvious or beneficial > > You can modify it slightly if you eager to: > > array_group( > ... > null|int|string|array $index_key, > ... > ): array > > If You provide null, it will operate on a bare array, no > grouping involved. > As a consequence, users have to provide reducer or > mapper. So virtually, > You design array_group to swallow > array_reduce and array_map. Though, > array_map is actually a different API compared to > array_reduce or > array_group. > > Interestingly, you will get a better quality of code if you include reducer: > > and mapper: consistently, whatever array level you work with. > > // array_reduce and > array_map alternative > array_group($array, null, reducer: $fnReducer); > array_group($array, null, mapper: $fnMapper); > > array_group($array, "level1", reducer: $fnReducer); > array_group($array, "level1", mapper: $fnMapper); > array_group($array, ["level1", "level2"], reducer: $fnReducer); > array_group($array, ["level1", "level2"], mapper: $fnMapper); > > I hope there is no bias between the proposed name (array_group) > and its > functionality. > > > > > Either of these approaches would be superior to cramming more functionality > > into a single-purpose function to turn it into a single-function swiss army > > knife: > > Not a swiss army knife, just a penknife. AS i wrote before, it is just a > shortcut. If you really mind about that, we can cancel filter > and sorter, > leaving only reducer and > mapper. Thought, we can filtering and sorting > before array_group, but i don't know whether it is as > accurate as embedding > them into array_group or not. If not, we have to iterate > manually, do > filtering and/or sorting, then reducing/mapping. > > > > > https://wiki.php.net/rfc/comprehensions > > https://wiki.php.net/rfc/pipe-operator-v2 > > Of course these are wonderful proposals. Yes, they are superior. To be honest, > I really hope they are landed in PHP. I already imagine what part of my code > will be replaced by them. Since I am not a C coder and totally blind about the > engine, I don't have my own opinion whether we lost good things or not, by not > adding them to PHP. > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: https://www.php.net/unsub.php >