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.me...@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 > -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: https://www.php.net/unsub.php
