Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:125813
Precedence: bulk
MIME-Version: 1.0
References: <ZydAaayECTRNG8Ackhb0tDgYjmdFKnkW4p0ISWlLFP5My4CsJBR1KKTylrD14Ags2GUyBLoJPf68N6Z6LoVzjG-tru8YAbmi74W5xzSkJ8I=@gpb.moe>
 <20340e0e-ddd0-44cd-b555-1402857eb20f@app.fastmail.com>
In-Reply-To: <20340e0e-ddd0-44cd-b555-1402857eb20f@app.fastmail.com>
Date: Sun, 20 Oct 2024 21:29:38 +0100
Message-ID: <CANNiG7i1mOpXnZGuh6GZBobKnABQYCvzBd1++EOyA8=JRNR6Mw@mail.gmail.com>
Subject: Re: [PHP-DEV] [RFC] Change behaviour of array sort functions to
 return a copy of the sorted array
To: Larry Garfield <larry@garfieldtech.com>
Cc: php internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000d49a7c0624ee6528"
From: bilge@scriptfusion.com (Bilge)

--000000000000d49a7c0624ee6528
Content-Type: text/plain; charset="UTF-8"

On Sun, 20 Oct 2024, 21:24 Larry Garfield, <larry@garfieldtech.com> wrote:

> On Sun, Oct 20, 2024, at 12:42 PM, Gina P. Banyard wrote:
> > Hello internals,
> >
> > I would like to propose a short RFC to make the return value of the
> > sort() and similar functions more useful:
> > https://wiki.php.net/rfc/array-sort-return-array
> >
> > I intend for the discussion to last 2 weeks and then open the vote.
> >
> > Best regards,
> >
> > Gina P. Banyard
>
> Like others, I'm skeptical of the behavior here not being all that clear.
>
> What several other languages do, and what I have in the plan for
> collections if/when they ever happen, is sort() sorts in place, while
> sorted() returns a new value.  There's similarly reverse() (in place) and
> reversed() (new value returned).
>
> *sorted($arr) seems like it would be a lot less confusing, and consistent
> with what other languages (and hopefully future PHP) do.  (I don't know
> what that means for array_walk(), but I don't know what that would even
> return anyway.)
>
> --Larry Garfield
>

I fail to see the logic of present tense = mutate, past tense = copy. At
least there is a pattern to it, but I would have to look it up several
times before it sunk in since it is not intuitive at all.

Cheers,
Bilge

>

--000000000000d49a7c0624ee6528
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"auto"><div><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D=
"gmail_attr">On Sun, 20 Oct 2024, 21:24 Larry Garfield, &lt;<a href=3D"mail=
to:larry@garfieldtech.com">larry@garfieldtech.com</a>&gt; wrote:<br></div><=
blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px=
 #ccc solid;padding-left:1ex">On Sun, Oct 20, 2024, at 12:42 PM, Gina P. Ba=
nyard wrote:<br>
&gt; Hello internals,<br>
&gt;<br>
&gt; I would like to propose a short RFC to make the return value of the <b=
r>
&gt; sort() and similar functions more useful:<br>
&gt; <a href=3D"https://wiki.php.net/rfc/array-sort-return-array" rel=3D"no=
referrer noreferrer" target=3D"_blank">https://wiki.php.net/rfc/array-sort-=
return-array</a><br>
&gt;<br>
&gt; I intend for the discussion to last 2 weeks and then open the vote.<br=
>
&gt;<br>
&gt; Best regards,<br>
&gt;<br>
&gt; Gina P. Banyard<br>
<br>
Like others, I&#39;m skeptical of the behavior here not being all that clea=
r.<br>
<br>
What several other languages do, and what I have in the plan for collection=
s if/when they ever happen, is sort() sorts in place, while sorted() return=
s a new value.=C2=A0 There&#39;s similarly reverse() (in place) and reverse=
d() (new value returned).=C2=A0 <br>
<br>
*sorted($arr) seems like it would be a lot less confusing, and consistent w=
ith what other languages (and hopefully future PHP) do.=C2=A0 (I don&#39;t =
know what that means for array_walk(), but I don&#39;t know what that would=
 even return anyway.)<br>
<br>
--Larry Garfield<br></blockquote></div></div><div dir=3D"auto"><br></div><d=
iv dir=3D"auto">I fail to see the logic of present tense =3D mutate, past t=
ense =3D copy. At least there is a pattern to it, but I would have to look =
it up several times before it sunk in since it is not intuitive at all.</di=
v><div dir=3D"auto"><br></div><div dir=3D"auto">Cheers,=C2=A0</div><div dir=
=3D"auto">Bilge=C2=A0</div><div dir=3D"auto"><div class=3D"gmail_quote"><bl=
ockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #=
ccc solid;padding-left:1ex">
</blockquote></div></div></div>

--000000000000d49a7c0624ee6528--