Prefer comma over || for match conditions

[fulldecent]

Above in this text it states:

match expression arms may contain multiple expressions separated by a comma. That is a logical OR, and is a short-hand for multiple match arms with the same right-hand side.

So in the code example below we prefer to use the comma instead of a logical OR.

[PhilDaiguille]

LGTM

[TimWolla]

Using the , instead of || is not quite equivalent, when the individual functions do not return a boolean. It works for str_contains(), but will not work for preg_match() for example:

https://3v4l.org/X7fQZ

I wonder if this example should perhaps be completely removed for that reason. Or alternatively, each of the cases should be cast to boolean in some way 🤔

[fulldecent]

Wow, so if uses a == true to evaluate truth. But match uses === true. I'd call that a bug.

Check out this https://3v4l.org/h78ue

[TimWolla]

Wow, so if uses a == true to evaluate truth. But match uses === true. I'd call that a bug.

That is an intentional design decision made in the RFC for the match() expression: https://wiki.php.net/rfc/match_expression_v2#no_type_coercion

[fulldecent]

That RFC relies on an old version of PHP (7) to make an argument.

PHP 8 fixed that issue: https://3v4l.org/PLkiG

---

Thanks for the references and education here.