Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112506
To: internals@lists.php.net
References: <b6e84a3b-012e-4ef3-8c2b-14c7fcf2d45d@www.fastmail.com>
Message-ID: <e2cff67d-f207-5faa-9501-15b392e63001@gmx.net>
Date: Mon, 14 Dec 2020 23:55:14 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.5.0
MIME-Version: 1.0
In-Reply-To: <b6e84a3b-012e-4ef3-8c2b-14c7fcf2d45d@www.fastmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: quoted-printable
Content-Language: en-US
Subject: Re: [PHP-DEV] [RFC] Short-match
From: a.leathley@gmx.net (Andreas Leathley)

On 14.12.20 18:33, Larry Garfield wrote:
> I present to Internals this tiny RFC to follow up on the match() express=
ion RFC from earlier in the year.  There was solidly positive support for =
this shortcut previously but it was removed for simplicity at the time, wi=
th the intent to bring it back later.  It's now later.
>
> https://wiki.php.net/rfc/short-match

I think it is a good addition that will make code clearer. If the main
counter-argument is that "if - elseif - else" does the same thing,
comparing the syntax to that and showing some real-world examples might
even convince some more people who think it is unnecessary, as the code
mostly speaks for itself, but the example currently in the RFC might be
a bit too simplistic to be convincing for the sceptics.

It would also be interesting how much switch(true) is used in open
source code currently, as it is the currently used "equivalent" to
match(true) - I know it is in some parts of the Symfony code, probably
because it is often easier to scan over when compared to huge if /
elseif blocks. match(true) is clearly an improvement to switch(true), so
while match has almost no use so far in PHP code, switch(true)
definitely has.