Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:120489
MIME-Version: 1.0
In-Reply-To: <CAF-mU4xMskp2cHtjZM3FB74EY6b1yvPshjQbO-PtQkuuD08=-g@mail.gmail.com>
References: <289E585B-EF8B-4B17-89BE-BE8295FD9FE1@gmail.com>
 <12bede25-36ad-037d-8802-6348da7a2e3d@alec.pl> <CAF-mU4xMskp2cHtjZM3FB74EY6b1yvPshjQbO-PtQkuuD08=-g@mail.gmail.com>
Date: Thu, 1 Jun 2023 07:07:13 +1000
Message-ID: <CAKVXxD-KCZJCN6yJkVhucgwb3TTtkfg8v-=2bHHPbHGkhZNHDQ@mail.gmail.com>
To: =?UTF-8?Q?Rokas_=C5=A0leinius?= <rokas.sleinius@gmail.com>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000035387905fd03b4e6"
Subject: Re: [PHP-DEV] [RFC] [Discussion] Add new function `array_group`
From: mickmackusa@gmail.com (mickmackusa)

--00000000000035387905fd03b4e6
Content-Type: text/plain; charset="UTF-8"

As an answerer and curator of many [php][arrays][grouping] tagged questions
on Stack Overflow for several years, I'd like to mention that developers'
non-SQL grouping needs are much more nuanced than merely defining group
qualification and creating subarrays.

Often devs will want to sum, count (increment a counter), concatenate
another column (or columns), or otherwise mutate the encountered row in
each group.

Have you considered the extendability of your pitched function? Without
these extra considerations, devs would need to make another pass over the
new nested structure.

mickmackusa

--00000000000035387905fd03b4e6--